Skip to content

Commit 8cdc71f

Browse files
committed
KVM: selftests: Print timer ctl register in ISTATUS assertion
Zenghui noted that the test assertion for the ISTATUS bit is printing the current timer value instead of the control register in the case of failure. While the assertion is sound, printing CNT isn't informative. Change things around to actually print the CTL register value instead. Reported-by: Zenghui Yu <[email protected]> Closes: https://lore.kernel.org/kvmarm/[email protected]/ Reviewed-by: Zenghui Yu <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Oliver Upton <[email protected]>
1 parent 06fdd89 commit 8cdc71f

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

tools/testing/selftests/kvm/aarch64/arch_timer.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,7 @@ static void guest_validate_irq(unsigned int intid,
160160
__GUEST_ASSERT(xcnt >= cval,
161161
"xcnt = 0x%lx, cval = 0x%lx, xcnt_diff_us = 0x%lx",
162162
xcnt, cval, xcnt_diff_us);
163-
__GUEST_ASSERT(xctl & CTL_ISTATUS, "xcnt = 0x%lx", xcnt);
163+
__GUEST_ASSERT(xctl & CTL_ISTATUS, "xctl = 0x%lx", xctl);
164164

165165
WRITE_ONCE(shared_data->nr_iter, shared_data->nr_iter + 1);
166166
}

0 commit comments

Comments
 (0)