]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
selftests/kvm: remove stale TODO in xapic_state_test
authorSukrut Heroorkar <hsukrut3@gmail.com>
Mon, 8 Sep 2025 21:05:46 +0000 (23:05 +0200)
committerSean Christopherson <seanjc@google.com>
Tue, 23 Sep 2025 15:39:01 +0000 (08:39 -0700)
The TODO about using the number of vCPUs instead of vcpu.id + 1
was already addressed by commit 376bc1b458c9 ("KVM: selftests: Don't
assume vcpu->id is '0' in xAPIC state test"). The comment is now
stale and can be removed.

Signed-off-by: Sukrut Heroorkar <hsukrut3@gmail.com>
Link: https://lore.kernel.org/r/20250908210547.12748-1-hsukrut3@gmail.com
Signed-off-by: Sean Christopherson <seanjc@google.com>
tools/testing/selftests/kvm/x86/xapic_state_test.c

index fdebff1165c7804a012a7cb02b22974c743c6268..3b4814c55722bc902a1c4bb0ace2ec558e7439a5 100644 (file)
@@ -120,8 +120,8 @@ static void test_icr(struct xapic_vcpu *x)
                __test_icr(x, icr | i);
 
        /*
-        * Send all flavors of IPIs to non-existent vCPUs.  TODO: use number of
-        * vCPUs, not vcpu.id + 1.  Arbitrarily use vector 0xff.
+        * Send all flavors of IPIs to non-existent vCPUs. Arbitrarily use
+        * vector 0xff.
         */
        icr = APIC_INT_ASSERT | 0xff;
        for (i = 0; i < 0xff; i++) {