On 6/6/2025 3:22 AM, Sean Christopherson wrote: > Use set_bit() in vmx_posted_interrupts_test_worker(), as the usage doesn't > actually look at the original value. > > No functional change intended. > > Signed-off-by: Sean Christopherson <seanjc@xxxxxxxxxx> > --- > x86/vmx_tests.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/x86/vmx_tests.c b/x86/vmx_tests.c > index 2f178227..53706000 100644 > --- a/x86/vmx_tests.c > +++ b/x86/vmx_tests.c > @@ -11122,8 +11122,8 @@ static void vmx_posted_interrupts_test_worker(void *data) > while (!args->in_guest) > pause(); > > - test_and_set_bit(args->nr, args->pi_desc); > - test_and_set_bit(256, args->pi_desc); > + set_bit(args->nr, args->pi_desc); > + set_bit(256, args->pi_desc); > apic_icr_write(PI_VECTOR, args->dest); > } > Reviewed-by: Dapeng Mi <dapeng1.mi@xxxxxxxxxxxxxxx>
