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.