Skip to content

Commit ff86b48

Browse files
SukrutHerookarsean-jc
authored andcommitted
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 376bc1b ("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>
1 parent c435978 commit ff86b48

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -120,8 +120,8 @@ static void test_icr(struct xapic_vcpu *x)
120120
__test_icr(x, icr | i);
121121

122122
/*
123-
* Send all flavors of IPIs to non-existent vCPUs. TODO: use number of
124-
* vCPUs, not vcpu.id + 1. Arbitrarily use vector 0xff.
123+
* Send all flavors of IPIs to non-existent vCPUs. Arbitrarily use
124+
* vector 0xff.
125125
*/
126126
icr = APIC_INT_ASSERT | 0xff;
127127
for (i = 0; i < 0xff; i++) {

0 commit comments

Comments
 (0)