On Tue, Jun 30, 2026 at 6:33 AM Cédric Le Goater <[email protected]> wrote: > > The following changes since commit 30e8a06b64aa58a3990ba39cb5d09531e7d265e0: > > Merge tag 'net-pull-request' of https://github.com/jasowang/qemu into > staging (2026-06-29 17:41:42 +0200) > > are available in the Git repository at: > > https://github.com/legoater/qemu/ tags/pull-vfio-20260630 > > for you to fetch changes up to afe79bb7bf7fe16f418d6e10522946e13da04384: > > vfio/listener: Fix translated_addr for non-identity-mapped RAM sections > (2026-06-30 05:35:56 +0200)
Hi Cedric, This pull request breaks the migration test in the ubuntu-24.04-s390x-all-system CI job: https://gitlab.com/qemu-project/qemu/-/jobs/15157007316#L3078 >>> RUST_BACKTRACE=1 >>> PYTHON=/home/gitlab-runner/builds/P3MFS4LUf/0/qemu-project/qemu/build/pyvenv/bin/python3 >>> QTEST_QEMU_IMG=./qemu-img MESON_TEST_ITERATION=1 >>> UBSAN_OPTIONS=halt_on_error=1:abort_on_error=1:print_summary=1:print_stacktrace=1 >>> QTEST_QEMU_VNC_BINARY=./tools/qemu-vnc/qemu-vnc MALLOC_PERTURB_=226 >>> MSAN_OPTIONS=halt_on_error=1:abort_on_error=1:print_summary=1:print_stacktrace=1 >>> >>> G_TEST_DBUS_DAEMON=/home/gitlab-runner/builds/P3MFS4LUf/0/qemu-project/qemu/tests/dbus-daemon.sh >>> QTEST_QEMU_BINARY=./qemu-system-s390x >>> ASAN_OPTIONS=halt_on_error=1:abort_on_error=1:print_summary=1 >>> QTEST_QEMU_STORAGE_DAEMON_BINARY=./storage-daemon/qemu-storage-daemon >>> /home/gitlab-runner/builds/P3MFS4LUf/0/qemu-project/qemu/build/tests/qtest/migration-test >>> --tap -k --full ――――――――――――――――――――――――――――――――――――― ✀ ――――――――――――――――――――――――――――――――――――― stderr: Memory content inconsistency at 48b7000 first_byte = b5 last_byte = b4 current = b5 hit_edge = 1 Memory content inconsistency at 48b8000 first_byte = b5 last_byte = b4 current = b5 hit_edge = 1 Memory content inconsistency at 48b9000 first_byte = b5 last_byte = b4 current = b5 hit_edge = 1 Memory content inconsistency at 48ba000 first_byte = b5 last_byte = b4 current = b5 hit_edge = 1 Memory content inconsistency at 48bb000 first_byte = b5 last_byte = b4 current = b5 hit_edge = 1 Memory content inconsistency at 48bc000 first_byte = b5 last_byte = b4 current = b5 hit_edge = 1 Memory content inconsistency at 48bd000 first_byte = b5 last_byte = b4 current = b5 hit_edge = 1 Memory content inconsistency at 48be000 first_byte = b5 last_byte = b4 current = b5 hit_edge = 1 Memory content inconsistency at 48bf000 first_byte = b5 last_byte = b4 current = b5 hit_edge = 1 Memory content inconsistency at 48c0000 first_byte = b5 last_byte = b4 current = b5 hit_edge = 1 and in another 6975 pages** ERROR:../tests/qtest/migration/framework.c:190:check_guests_ram: assertion failed: (bad == 0) (test program exited with status code -6) Please take a look and resend the pull request with a fix. Thanks, Stefan > > ---------------------------------------------------------------- > vfio queue: > > * Fixes ROM read issues in vfio/pci: information leak, error > propagation, and uninitialized state > * Validates VERSION replies in vfio-user and updates the spec > for DMA access mode bits > * Merges .dma_map_file() into .dma_map() in the iommufd backend > * Reworks switchover-ack to be re-usable and implements the > VFIO_PRECOPY_INFO_REINIT feature for additional pre-copy > iterations before switchover > * Adds ATS support for passthrough devices via iommufd > * Fixes translated_addr for non-identity-mapped RAM sections in > the VFIO listener > > ---------------------------------------------------------------- > Avihai Horon (14): > migration: Propagate errors in migration_completion_precopy() > migration/ram: Use migration_bitmap_sync_precopy() for postcopy discard > migration: Run final save_query_pending at switchover > migration: Log the approver in qemu_loadvm_approve_switchover() > migration: Replace switchover_ack_needed SaveVMHandler > migration: Rename switchover-ack code to legacy > migration: Make switchover-ack re-usable > migration: Fail migration if switchover-ack is requested after > switchover decision > vfio/migration: Extract VFIO_MIG_FLAG_DEV_INIT_DATA_SENT sending to > helper > vfio/migration: Add Error ** parameter to vfio_migration_init() > vfio/migration: Add new switchover-ack mechanism > vfio/migration: Implement VFIO_PRECOPY_INFO_REINIT feature > vfio/migration: Check VFIO_PRECOPY_INFO_REINIT during switchover > migration: Enable new switchover-ack > > Cédric Le Goater (3): > vfio/pci: Fix information leak in vfio_rom_read() > vfio/iommufd: Merge .dma_map_file() into .dma_map() > vfio/listener: Fix translated_addr for non-identity-mapped RAM sections > > GuoHan Zhao (1): > vfio-user: validate VERSION replies > > Mario Casquero (2): > vfio/pci: Initialize rom_read_failed in vfio_pci_load_rom() > vfio/pci: Propagate errors in vfio_pci_load_rom() using Error API > > Mattias Nissler (1): > docs: Update vfio-user spec to describe DMA access mode bits > > Nathan Chen (2): > iommufd: Introduce handler for device ATS support > vfio/pci: Add ats property > > docs/devel/migration/vfio.rst | 17 +++- > docs/interop/vfio-user.rst | 82 +++++++++------ > qapi/migration.json | 14 ++- > hw/vfio/pci.h | 1 + > hw/vfio/vfio-migration-internal.h | 2 + > include/hw/vfio/vfio-container.h | 15 --- > include/migration/client-options.h | 1 + > include/migration/misc.h | 2 + > include/migration/register.h | 56 +++++----- > include/system/host_iommu_device.h | 9 ++ > migration/migration.h | 34 ++++++- > migration/savevm.h | 9 +- > backends/iommufd.c | 8 ++ > hw/core/machine.c | 1 + > hw/s390x/s390-stattrib.c | 2 +- > hw/vfio-user/proxy.c | 21 ++-- > hw/vfio/container.c | 38 ------- > hw/vfio/iommufd.c | 50 ++++++--- > hw/vfio/listener.c | 5 +- > hw/vfio/migration.c | 204 > +++++++++++++++++++++++++++++-------- > hw/vfio/pci.c | 129 +++++++++++++++++++---- > migration/block-dirty-bitmap.c | 11 +- > migration/migration.c | 73 ++++++++++--- > migration/options.c | 9 ++ > migration/ram.c | 40 ++++---- > migration/savevm.c | 173 +++++++++++++++++++++---------- > hw/vfio/trace-events | 7 +- > migration/trace-events | 9 +- > 28 files changed, 709 insertions(+), 313 deletions(-) > >
