Received-SPF: pass (google.com: domain of linux-kernel+bounces-103637-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Reply-To: Sean Christopherson <seanjc@google.com>
Date: Thu, 14 Mar 2024 10:51:15 -0700
Precedence: bulk
Mime-Version: 1.0
Message-ID: <20240314175116.2366301-1-seanjc@google.com>
Subject: [PATCH] KVM: selftests: Fix __GUEST_ASSERT() format warnings in ARM's
 arch timer test
From: Sean Christopherson <seanjc@google.com>
To: Marc Zyngier <maz@kernel.org>, Oliver Upton <oliver.upton@linux.dev>
Cc: linux-arm-kernel@lists.infradead.org, kvmarm@lists.linux.dev, 
	linux-kernel@vger.kernel.org, Sean Christopherson <seanjc@google.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

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 =E2=80=98guest_run_stage=E2=80=99:
aarch64/arch_timer.c:138:33: warning: format =E2=80=98%lx=E2=80=99 expects =
argument of type =E2=80=98long unsigned int=E2=80=99,
                             but argument 6 has type =E2=80=98uint32_t=E2=
=80=99 {aka =E2=80=98unsigned int=E2=80=99} [-Wformat=3D]
  138 |                                 "config_iter + 1 =3D 0x%lx, irq_ite=
r =3D 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>
---

2024 Whack-a-Mole Champion.

Note, this is based on kvm/queue and probably won't apply to your tree as
the buggy commit went in via the RISC-V pull request.

 tools/testing/selftests/kvm/aarch64/arch_timer.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/testing/selftests/kvm/aarch64/arch_timer.c b/tools/testi=
ng/selftests/kvm/aarch64/arch_timer.c
index ddba2c2fb5de..93100b3f1312 100644
--- a/tools/testing/selftests/kvm/aarch64/arch_timer.c
+++ b/tools/testing/selftests/kvm/aarch64/arch_timer.c
@@ -135,7 +135,7 @@ static void guest_run_stage(struct test_vcpu_shared_dat=
a *shared_data,
=20
 		irq_iter =3D READ_ONCE(shared_data->nr_iter);
 		__GUEST_ASSERT(config_iter + 1 =3D=3D irq_iter,
-				"config_iter + 1 =3D 0x%lx, irq_iter =3D 0x%lx.\n"
+				"config_iter + 1 =3D 0x%x, irq_iter =3D 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);

base-commit: e9a2bba476c8332ed547fce485c158d03b0b9659
--=20
2.44.0.291.gc1ea87d7ee-goog