Re: [xen-unstable-smoke test] 175226: regressions - FAIL

[Julien Grall <julien@xxxxxxx>]

Hi,

On 15/12/2022 01:41, osstest service owner wrote:
> flight 175226 xen-unstable-smoke real [real]
> http://logs.test-lab.xenproject.org/osstest/logs/175226/
>
> Regressions :-(
>
> Tests which did not succeed and are blocking,
> including tests which could not be run:
>   build-armhf                   6 xen-build                fail REGR. vs. 175173
armhf/xen/tools/libs/light/../../../tools/config.h  -c -o libxl_genid.o 
libxl_genid.c
In file included from libxl_virtio.c:15:
libxl_virtio.c: In function 'libxl__set_xenstore_virtio':
libxl_internal.h:4388:51: error: format '%lu' expects argument of type 
'long unsigned int', but argument 3 has type 'uint64_t' {aka 'long long 
unsigned int'} [-Werror=format=]
 #define GCSPRINTF(fmt, ...) (libxl__sprintf((gc), (fmt), __VA_ARGS__))
                                                   ^~~~~
libxl_virtio.c:48:41: note: in expansion of macro 'GCSPRINTF'
     flexarray_append_pair(back, "base", GCSPRINTF("%lu", virtio->base));
                                         ^~~~~~~~~
libxl_internal.h:4388:51: error: format '%lu' expects argument of type 
'long unsigned int', but argument 3 has type 'uint64_t' {aka 'long long 
unsigned int'} [-Werror=format=]
 #define GCSPRINTF(fmt, ...) (libxl__sprintf((gc), (fmt), __VA_ARGS__))
                                                   ^~~~~
libxl_virtio.c:53:42: note: in expansion of macro 'GCSPRINTF'
     flexarray_append_pair(front, "base", GCSPRINTF("%lu", virtio->base));
                                          ^~~~~~~~~
cc1: all warnings being treated as errors
make[5]: *** 
[/home/osstest/build.175251.build-armhf/xen/tools/libs/light/../../../tools/Rules.mk:188: 
libxl_virtio.o] Error 1

This build breakage was introduced by "libxl: add support for generic 
virtio device". %lu will likely want to be switched to PRIx64.
Viresh, we need to unblock OSStest (our CI) as soon as possible. So can 
you look at it and confirm the rest of the tools build on arm32?
Cheers,

--
Julien Grall