Re: [Qemu-devel] [PATCH v6 00/13] Add migration support for VFIO device

[no-reply]

Patchew URL: https://patchew.org/QEMU/address@hidden/



Hi,

This series failed build test on s390x host. Please find the details below.

=== TEST SCRIPT BEGIN ===
#!/bin/bash
# Testing script will be invoked under the git checkout with
# HEAD pointing to a commit that has the patches applied on top of "base"
# branch
set -e

echo
echo "=== ENV ==="
env

echo
echo "=== PACKAGES ==="
rpm -qa

echo
echo "=== UNAME ==="
uname -a

CC=$HOME/bin/cc
INSTALL=$PWD/install
BUILD=$PWD/build
mkdir -p $BUILD $INSTALL
SRC=$PWD
cd $BUILD
$SRC/configure --cc=$CC --prefix=$INSTALL
make -j4
# XXX: we need reliable clean up
# make check -j4 V=1
make install
=== TEST SCRIPT END ===

  CC      hw/watchdog/trace.o
In file included from hw/vfio/trace.c:4:
hw/vfio/trace.h: In function ‘_nocheck__trace_vfio_save_pending’:
hw/vfio/trace.h:3655:18: error: format ‘%x’ expects argument of type ‘unsigned 
int’, but argument 7 has type ‘uint64_t’ {aka ‘long unsigned int’} 
[-Werror=format=]
 3655 |         qemu_log("%d@%zu.%06zu:vfio_save_pending " " (%s), precopy 
0x%"PRIx64" postcopy 0x%x"PRIx64" compatible 0x%"PRIx64 "\n",
      |                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
......
---
      |                                                                         
                   unsigned int
      |                                                                         
                  %lx
hw/vfio/trace.h: In function ‘_nocheck__trace_vfio_load_state_device_data’:
hw/vfio/trace.h:3770:61: error: expected ‘)’ before ‘PRIx6’
 3770 |         qemu_log("%d@%zu.%06zu:vfio_load_state_device_data " " (%s), 
Offset 0x%"PRIx64" size 0x%"PRIx6 "\n",
      |                                                             ^           
                         ~~~~~
      |                                                             )
hw/vfio/trace.h:3770:20: error: format ‘%d’ expects a matching ‘int’ argument 
[-Werror=format=]
 3770 |         qemu_log("%d@%zu.%06zu:vfio_load_state_device_data " " (%s), 
Offset 0x%"PRIx64" size 0x%"PRIx6 "\n",
      |                   ~^
      |                    |
      |                    int
hw/vfio/trace.h:3770:24: error: format ‘%zu’ expects a matching ‘size_t’ 
argument [-Werror=format=]
 3770 |         qemu_log("%d@%zu.%06zu:vfio_load_state_device_data " " (%s), 
Offset 0x%"PRIx64" size 0x%"PRIx6 "\n",
      |                      ~~^
      |                        |
      |                        long unsigned int
hw/vfio/trace.h:3770:30: error: format ‘%zu’ expects a matching ‘size_t’ 
argument [-Werror=format=]
 3770 |         qemu_log("%d@%zu.%06zu:vfio_load_state_device_data " " (%s), 
Offset 0x%"PRIx64" size 0x%"PRIx6 "\n",
      |                          ~~~~^
      |                              |
      |                              long unsigned int
hw/vfio/trace.h:3770:18: error: format ‘%s’ expects a matching ‘char *’ 
argument [-Werror=format=]
 3770 |         qemu_log("%d@%zu.%06zu:vfio_load_state_device_data " " (%s), 
Offset 0x%"PRIx64" size 0x%"PRIx6 "\n",
      |                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
hw/vfio/trace.h:3770:66: note: format string is defined here
---
      |                                                                 ~^
      |                                                                  |
      |                                                                  char *
hw/vfio/trace.h:3770:18: error: format ‘%lx’ expects a matching ‘long unsigned 
int’ argument [-Werror=format=]
 3770 |         qemu_log("%d@%zu.%06zu:vfio_load_state_device_data " " (%s), 
Offset 0x%"PRIx64" size 0x%"PRIx6 "\n",
      |                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from 
/var/tmp/patchew-tester-tmp-b93z8a_z/src/include/qemu/osdep.h:103,
---
/usr/include/inttypes.h:121:34: note: format string is defined here
  121 | # define PRIx64  __PRI64_PREFIX "x"
In file included from hw/vfio/trace.c:4:
hw/vfio/trace.h:3770:18: error: spurious trailing ‘%’ in format 
[-Werror=format=]
 3770 |         qemu_log("%d@%zu.%06zu:vfio_load_state_device_data " " (%s), 
Offset 0x%"PRIx64" size 0x%"PRIx6 "\n",
      |                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
hw/vfio/trace.h:3770:96: note: format string is defined here


The full log is available at
http://patchew.org/logs/address@hidden/testing.s390x/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to address@hidden