Re: [PATCH 1/2] KVM: SVM: don't check have_run_cpus in sev_writeback_caches()

[Yury Norov <yury.norov@gmail.com>]

On Mon, Aug 11, 2025 at 01:50:15PM -0700, Sean Christopherson wrote:
> On Mon, Aug 11, 2025, Yury Norov wrote:
> > From: Yury Norov (NVIDIA) <yury.norov@gmail.com>
> > 
> > Before calling wbnoinvd_on_cpus_mask(), the function checks the cpumask
> > for emptiness. It's useless, as the following wbnoinvd_on_cpus_mask()
> > ends up with smp_call_function_many_cond(), which handles empty cpumask
> > correctly.
> 
> I don't agree that it's useless.  The early check avoids disabling/enabling
> preemption (which is cheap, but still), and IMO it makes the KVM code more obviously
> correct.  E.g. it takes quite a bit of digging to understand that invoking
> wbnoinvd_on_cpus_mask() with an empty mask is ok/fine.
> 
> I'm not completely opposed to this change, but I also don't see the point.

So, there's a tradeoff between useless preemption cycling, which is
O(1) and cpumask_empty(), which is O(N).

I have no measurements that can support one vs another. But the
original patch doesn't discuss it anyhow, as well. So, with the
lack of any information on performance impact, I'd stick with the 
version that brings less code.

Agree?