From 71f8fd5a644f6e2c4d443d8b564a8cc60b928ccb Mon Sep 17 00:00:00 2001 From: Luca Fancellu Date: Fri, 22 May 2026 08:20:44 +0200 Subject: [PATCH] tools/tests: Fix paging-mempool xen_pfn_t format strings xen_pfn_t is not an unsigned long on all architectures. In particular, Arm32 builds see it as a 64-bit type, so printing it with %lx triggers a -Werror=format build failure. Use PRI_xen_pfn for the GFN diagnostics instead. Fixes: 66c982a5d3614 ("tests/paging-mempool: Extend to test P2M relocation") Signed-off-by: Luca Fancellu Reviewed-by: Andrew Cooper --- tools/tests/paging-mempool/test-paging-mempool.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/tests/paging-mempool/test-paging-mempool.c b/tools/tests/paging-mempool/test-paging-mempool.c index 97f5b77209..0e03e2e4a0 100644 --- a/tools/tests/paging-mempool/test-paging-mempool.c +++ b/tools/tests/paging-mempool/test-paging-mempool.c @@ -208,7 +208,7 @@ static int check_guest_marks(xen_pfn_t gfn, uint32_t mark_start, size_t count) if ( errs[i] ) { rc = -1; - fail(" Fail: check mark unable to map gfn %05lx: %d\n", + fail(" Fail: check mark unable to map gfn %05"PRI_xen_pfn": %d\n", gfns[i], errs[i]); continue; } @@ -216,7 +216,7 @@ static int check_guest_marks(xen_pfn_t gfn, uint32_t mark_start, size_t count) if ( *mark == exp ) continue; - fail(" Fail: check mark: gfn %05lx expecting %08x (%u), got %08x (%u)\n", + fail(" Fail: check mark: gfn %05"PRI_xen_pfn" expecting %08x (%u), got %08x (%u)\n", gfns[i], exp, ~exp, *mark, ~*mark); rc = -1; }