Re: [PATCH v1 2/3] KVM: selftests: Check dirty-ring size before enabling

From: Sean Christopherson

Date: Thu Oct 01 2026 - 19:18:57 EST


On Thu, Oct 01, 2026, Leonardo Bras wrote:
> On Wed, Sep 30, 2026 at 01:31:25PM -0700, Sean Christopherson wrote:
> > > "Dirty-ring not supported in this kernel\n");
> >
> > "this kernel" could be misleading, some architectures simply don't support the
> > dirty ring.

Hmm, the counter argument is that this is actually accurate if those architectures
were to gain support in the future, but someone ran this on an older kernel...

> So, do you suggest keeping it simple by printing "Dirty-ring not supported"?

Yeah, unless you have a better idea. Actually, even though Kconfig isn't user-
controllable, it's still there in the .config. So maybe something like this?

TEST_ASSERT(cap, "Dirty-ring not supported, check CONFIG_HAVE_KVM_DIRTY_RING?");

That would at least give the user a breadcrumb to start with.