Commit ff86b48d authored by Sukrut Heroorkar's avatar Sukrut Heroorkar Committed by Sean Christopherson
Browse files

selftests/kvm: remove stale TODO in xapic_state_test 



The TODO about using the number of vCPUs instead of vcpu.id + 1
was already addressed by commit 376bc1b4 ("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: default avatarSukrut Heroorkar <hsukrut3@gmail.com>
Link: https://lore.kernel.org/r/20250908210547.12748-1-hsukrut3@gmail.com


Signed-off-by: default avatarSean Christopherson <seanjc@google.com>
parent c435978e

tools/testing/selftests/kvm/x86/xapic_state_test.c

+2 −2
Original line number Diff line number Diff line
@@ -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++) {