Re: [kvm-unit-tests PATCH v3 2/2] x86/vmx: (Re)Add a nVMX test to validate load/save of DEBUGCTL

[Sean Christopherson <seanjc@google.com>]

On Wed, May 27, 2026, Xiaoyao Li wrote:
> On 5/27/2026 11:12 PM, Sean Christopherson wrote:
> > +	for (i = 1; i <= supported_bits; i++) {
> > +		if ((i & supported_bits) != i)
> > +			continue;
> > +
> > +		for (j = 1; j <= supported_bits; j++) {
> > +			if ((j & supported_bits) != j)
> > +				continue;
> > +
> > +			__vmx_debugctl_test(i, j);
> > +		}
> > +	}
> 
> What a clever idea!
> 
> A minor flaw: Both i and j should start from 0, since 0 is a valid value.

Heh, I knew I should have added a comment.  I deliberately started at '1', because
'0' is used as a "base" value, to be able to detect changes.  I.e. '0' is getting
valiated on every test anyways.