[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 00/13] Add migration support for VFIO device
|
From: |
no-reply |
|
Subject: |
Re: [Qemu-devel] [PATCH v5 00/13] Add migration support for VFIO device |
|
Date: |
Mon, 8 Jul 2019 13:49:19 -0700 (PDT) |
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/virtio/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:126: error: stray ‘\’ in program
3655 | qemu_log("%d@%zu.%06zu:vfio_save_pending " " (%s), precopy
0x%"PRIx64" postcopy 0x%x"PRIx64" compatible 0x%""PRIx64 "\n",
|
^
hw/vfio/trace.h:3655:51: error: expected ‘)’ before ‘n’
3655 | qemu_log("%d@%zu.%06zu:vfio_save_pending " " (%s), precopy
0x%"PRIx64" postcopy 0x%x"PRIx64" compatible 0x%""PRIx64 "\n",
| ^
~
| )
hw/vfio/trace.h:3655:128: error: missing terminating " character [-Werror]
3655 | qemu_log("%d@%zu.%06zu:vfio_save_pending " " (%s), precopy
0x%"PRIx64" postcopy 0x%x"PRIx64" compatible 0x%""PRIx64 "\n",
|
^
hw/vfio/trace.h:3655:128: error: missing terminating " character
3655 | qemu_log("%d@%zu.%06zu:vfio_save_pending " " (%s), precopy
0x%"PRIx64" postcopy 0x%x"PRIx64" compatible 0x%""PRIx64 "\n",
|
^~
hw/vfio/trace.h:3655:20: error: format ‘%d’ expects a matching ‘int’ argument
[-Werror=format=]
3655 | qemu_log("%d@%zu.%06zu:vfio_save_pending " " (%s), precopy
0x%"PRIx64" postcopy 0x%x"PRIx64" compatible 0x%""PRIx64 "\n",
| ~^
| |
| int
hw/vfio/trace.h:3655:24: error: format ‘%zu’ expects a matching ‘size_t’
argument [-Werror=format=]
3655 | qemu_log("%d@%zu.%06zu:vfio_save_pending " " (%s), precopy
0x%"PRIx64" postcopy 0x%x"PRIx64" compatible 0x%""PRIx64 "\n",
| ~~^
| |
| long unsigned int
hw/vfio/trace.h:3655:30: error: format ‘%zu’ expects a matching ‘size_t’
argument [-Werror=format=]
3655 | qemu_log("%d@%zu.%06zu:vfio_save_pending " " (%s), precopy
0x%"PRIx64" postcopy 0x%x"PRIx64" compatible 0x%""PRIx64 "\n",
| ~~~~^
| |
| long unsigned int
hw/vfio/trace.h:3655:18: error: format ‘%s’ expects a matching ‘char *’
argument [-Werror=format=]
3655 | qemu_log("%d@%zu.%06zu:vfio_save_pending " " (%s), precopy
0x%"PRIx64" postcopy 0x%x"PRIx64" compatible 0x%""PRIx64 "\n",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
hw/vfio/trace.h:3655:56: note: format string is defined here
---
| ~^
| |
| char *
hw/vfio/trace.h:3655:18: error: format ‘%lx’ expects a matching ‘long unsigned
int’ argument [-Werror=format=]
3655 | qemu_log("%d@%zu.%06zu:vfio_save_pending " " (%s), precopy
0x%"PRIx64" postcopy 0x%x"PRIx64" compatible 0x%""PRIx64 "\n",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from
/var/tmp/patchew-tester-tmp-gaz1sa3c/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:3655:18: error: format ‘%x’ expects a matching ‘unsigned int’
argument [-Werror=format=]
3655 | qemu_log("%d@%zu.%06zu:vfio_save_pending " " (%s), precopy
0x%"PRIx64" postcopy 0x%x"PRIx64" compatible 0x%""PRIx64 "\n",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
hw/vfio/trace.h:3655:92: note: format string is defined here
---
|
~^
|
|
|
unsigned int
hw/vfio/trace.h:3655:18: error: unknown conversion type character ‘P’ in format
[-Werror=format=]
3655 | qemu_log("%d@%zu.%06zu:vfio_save_pending " " (%s), precopy
0x%"PRIx64" postcopy 0x%x"PRIx64" compatible 0x%""PRIx64 "\n",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
hw/vfio/trace.h:3655:118: note: format string is defined here
3655 | qemu_log("%d@%zu.%06zu:vfio_save_pending " " (%s), precopy
0x%"PRIx64" postcopy 0x%x"PRIx64" compatible 0x%""PRIx64 "\n",
|
^
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-gaz1sa3c/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
- [Qemu-devel] [PATCH v5 04/13] vfio: Add save and load functions for VFIO PCI devices, (continued)
- [Qemu-devel] [PATCH v5 04/13] vfio: Add save and load functions for VFIO PCI devices, Kirti Wankhede, 2019/07/08
- [Qemu-devel] [PATCH v5 06/13] vfio: Add VM state change handler to know state of VM, Kirti Wankhede, 2019/07/08
- [Qemu-devel] [PATCH v5 07/13] vfio: Add migration state change notifier, Kirti Wankhede, 2019/07/08
- [Qemu-devel] [PATCH v5 08/13] vfio: Register SaveVMHandlers for VFIO device, Kirti Wankhede, 2019/07/08
- [Qemu-devel] [PATCH v5 05/13] vfio: Add migration region initialization and finalize function, Kirti Wankhede, 2019/07/08
- [Qemu-devel] [PATCH v5 09/13] vfio: Add save state functions to SaveVMHandlers, Kirti Wankhede, 2019/07/08
- [Qemu-devel] [PATCH v5 10/13] vfio: Add load state functions to SaveVMHandlers, Kirti Wankhede, 2019/07/08
- [Qemu-devel] [PATCH v5 11/13] vfio: Add function to get dirty page list, Kirti Wankhede, 2019/07/08
- [Qemu-devel] [PATCH v5 12/13] vfio: Add vfio_listerner_log_sync to mark dirty pages, Kirti Wankhede, 2019/07/08
- [Qemu-devel] [PATCH v5 13/13] vfio: Make vfio-pci device migration capable., Kirti Wankhede, 2019/07/08
- Re: [Qemu-devel] [PATCH v5 00/13] Add migration support for VFIO device,
no-reply <=
- Re: [Qemu-devel] [PATCH v5 00/13] Add migration support for VFIO device, no-reply, 2019/07/08