KVM: selftests: Fix __GUEST_ASSERT() format warnings in ARM's arch timer test
authorSean Christopherson <seanjc@google.com>
Thu, 14 Mar 2024 17:51:15 +0000 (10:51 -0700)
committerOliver Upton <oliver.upton@linux.dev>
Tue, 26 Mar 2024 08:51:44 +0000 (01:51 -0700)
Use %x instead of %lx when printing uint32_t variables to fix format
warnings in ARM's arch timer test.

aarch64/arch_timer.c: In function ‘guest_run_stage’:
aarch64/arch_timer.c:138:33: warning: format ‘%lx’ expects argument of type ‘long unsigned int’,
                             but argument 6 has type ‘uint32_t’ {aka ‘unsigned int’} [-Wformat=]
  138 |                                 "config_iter + 1 = 0x%lx, irq_iter = 0x%lx.\n"
      |                                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
......
  141 |                                 config_iter + 1, irq_iter);
      |                                 ~~~~~~~~~~~~~~~
      |                                             |
      |                                             uint32_t {aka unsigned int}

Fixes: d1dafd065a23 ("KVM: arm64: selftests: Enable tuning of error margin in arch_timer test")
Signed-off-by: Sean Christopherson <seanjc@google.com>
Link: https://lore.kernel.org/r/20240314175116.2366301-1-seanjc@google.com
Signed-off-by: Oliver Upton <oliver.upton@linux.dev>
tools/testing/selftests/kvm/aarch64/arch_timer.c

index ddba2c2fb5deb1b2ec2ab02db352bada7e70feee..93100b3f1312aa330f2d25edc91ebd3aa41d4399 100644 (file)
@@ -135,7 +135,7 @@ static void guest_run_stage(struct test_vcpu_shared_data *shared_data,
 
                irq_iter = READ_ONCE(shared_data->nr_iter);
                __GUEST_ASSERT(config_iter + 1 == irq_iter,
-                               "config_iter + 1 = 0x%lx, irq_iter = 0x%lx.\n"
+                               "config_iter + 1 = 0x%x, irq_iter = 0x%x.\n"
                                "  Guest timer interrupt was not trigged within the specified\n"
                                "  interval, try to increase the error margin by [-e] option.\n",
                                config_iter + 1, irq_iter);