public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
* [PATCH-SERIES qemu/qemu-server v2 0/5] fix #6424: avoid timeout issue for QMP 'quit' for bulk suspend
@ 2026-06-29 13:54 Fiona Ebner
  2026-06-29 13:54 ` [PATCH qemu v2 1/5] async snapshot: allow skipping VM start after successful completion Fiona Ebner
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: Fiona Ebner @ 2026-06-29 13:54 UTC (permalink / raw)
  To: pve-devel

Changes in v2:
* rebase on current master
* use qemu-stable v11.0.2

I was able to reproduce the issue for a VM without IO thread busy
with IO on a heavily rate-limited storage.

The fix has two independent parts:

1) increase the QMP timeout:

The 'quit' QMP command itself is fast, since it only records the
shutdown request and notifies the main loop, but getting the response
can take a while. The reason is that qmp_dispatch() yields and must be
woken after executing the command and at that stage, QEMU is already
busy with teardown too.

2) skip VM start after successfully saving state:

When creating a snapshot of the VM state for hibernation, the VM must
not be started again if the snapshot operation was completed
successfully. The VM should remain stopped, so that the QMP 'quit' is
issued without further activity from the VM.

The latter addresses a more fundamental issue, but it still makes
sense to bump the timeout for 'quit' regardless, because the
explanation in 1) still applies.

I also added the rebase to QEMU 11.0.1 as part of this series, since
using the new parameter for 2) needs to be somehow version-guarded.
There would be other ways like extending QMP 'query-proxmox-support',
but since the opportunity with 11.0.1 is there, I felt like it is
easier in this case.

qemu:

Fiona Ebner (2):
  async snapshot: allow skipping VM start after successful completion
  update submodule and patches to QEMU 11.0.2

 ...d-support-for-sync-bitmap-mode-never.patch |   6 +-
 ...-support-for-conditional-and-always-.patch |   2 +-
 ...-to-bdrv_dirty_bitmap_merge_internal.patch |   2 +-
 .../0006-mirror-move-some-checks-to-qmp.patch |   2 +-
 ...oid-idle-event-loop-being-accounted.patch} |   0
 ...ial-deadlock-when-draining-during-tr.patch | 100 --------------
 ...-fix-regression-with-block-device-e.patch} |  15 +--
 ...k-to-bounce-buffer-if-BLKZEROOUT-is-.patch |  36 -----
 ...fix-decoding-of-MOVBE-and-CRC32-in-1.patch |  84 ------------
 ...-accidentally-autofree-existing-virg.patch |  59 --------
 ...store-IRQ-polling-for-non-kernel-irq.patch |  47 -------
 ...strList-leak-in-x86_cpu_get_unavaila.patch |  36 -----
 ...-missing-PF_INSTR-in-SIGSEGV-context.patch |  43 ------
 ...e_save_state_v-fix-double-error_setg.patch |  46 -------
 ...e-fix-regression-when-mmap-ing-expor.patch |  94 -------------
 ...issing-VIRTIO_BLK_T_SCSI_CMD-size-ch.patch |  48 -------
 ...-use-after-free-of-cancelled-request.patch |  82 -----------
 ...ar-tag-byte-when-processing-messages.patch |  40 ------
 ...very-bitmask-with-modified-xAPIC-ids.patch |  63 ---------
 ...aio-bound-ioq_submit-recursion-depth.patch | 127 ------------------
 ...reject-zero-DMA-page-size-capability.patch |  46 -------
 ...-zero-migration-page-size-capability.patch |  44 ------
 ...estrict-dma_map_file-to-shared-RAM-o.patch |  88 ------------
 ...async-for-background-state-snapshots.patch |  49 ++++---
 ...add-optional-buffer-size-to-QEMUFile.patch |   6 +-
 ...add-the-zeroinit-block-driver-filter.patch |   4 +-
 ...le-posix-make-locking-optiono-on-cre.patch |   6 +-
 ...VE-Backup-add-vma-backup-format-code.patch |   6 +-
 ...ckup-Proxmox-backup-patches-for-QEMU.patch |   8 +-
 ...estore-new-command-to-restore-from-p.patch |   4 +-
 ...k-driver-to-map-backup-archives-into.patch |   8 +-
 ...ct-stderr-to-journal-when-daemonized.patch |   6 +-
 ...igrate-dirty-bitmap-state-via-savevm.patch |   4 +-
 .../0037-block-add-alloc-track-driver.patch   |   4 +-
 .../0038-PVE-backup-add-fleecing-option.patch |   2 +-
 ...ment-backup-access-setup-and-teardow.patch |   2 +-
 ...se-migration-blocker-check-for-snaps.patch |   6 +-
 debian/patches/series                         |  21 +--
 qemu                                          |   2 +-
 39 files changed, 83 insertions(+), 1165 deletions(-)
 rename debian/patches/extra/{0004-fdmon-io_uring-avoid-idle-event-loop-being-accounted.patch => 0002-fdmon-io_uring-avoid-idle-event-loop-being-accounted.patch} (100%)
 delete mode 100644 debian/patches/extra/0002-ide-avoid-potential-deadlock-when-draining-during-tr.patch
 rename debian/patches/extra/{0012-block-export-fuse-fix-regression-with-block-device-e.patch => 0003-block-export-fuse-fix-regression-with-block-device-e.patch} (71%)
 delete mode 100644 debian/patches/extra/0003-block-io-fallback-to-bounce-buffer-if-BLKZEROOUT-is-.patch
 delete mode 100644 debian/patches/extra/0005-target-i386-tcg-fix-decoding-of-MOVBE-and-CRC32-in-1.patch
 delete mode 100644 debian/patches/extra/0006-hw-display-don-t-accidentally-autofree-existing-virg.patch
 delete mode 100644 debian/patches/extra/0007-hw-i386-vapic-restore-IRQ-polling-for-non-kernel-irq.patch
 delete mode 100644 debian/patches/extra/0008-target-i386-fix-strList-leak-in-x86_cpu_get_unavaila.patch
 delete mode 100644 debian/patches/extra/0009-target-i386-fix-missing-PF_INSTR-in-SIGSEGV-context.patch
 delete mode 100644 debian/patches/extra/0010-migration-vmstate_save_state_v-fix-double-error_setg.patch
 delete mode 100644 debian/patches/extra/0011-block-export-fuse-fix-regression-when-mmap-ing-expor.patch
 delete mode 100644 debian/patches/extra/0013-virtio-blk-add-missing-VIRTIO_BLK_T_SCSI_CMD-size-ch.patch
 delete mode 100644 debian/patches/extra/0014-lsi53c895a-fix-use-after-free-of-cancelled-request.patch
 delete mode 100644 debian/patches/extra/0015-lsi53c895a-clear-tag-byte-when-processing-messages.patch
 delete mode 100644 debian/patches/extra/0016-apic-fix-delivery-bitmask-with-modified-xAPIC-ids.patch
 delete mode 100644 debian/patches/extra/0017-block-linux-aio-bound-ioq_submit-recursion-depth.patch
 delete mode 100644 debian/patches/extra/0018-vfio-user-reject-zero-DMA-page-size-capability.patch
 delete mode 100644 debian/patches/extra/0019-vfio-user-reject-zero-migration-page-size-capability.patch
 delete mode 100644 debian/patches/extra/0020-vfio-container-Restrict-dma_map_file-to-shared-RAM-o.patch


qemu-server:

Fiona Ebner (3):
  fix #6424: increase timeout for QMP 'quit' to 60s to avoid issue with
    bulk suspend
  run state: use v5.36 and signatures in module
  suspend: skip VM start after successfully saving state

 src/PVE/QMPClient.pm           |  6 ++++++
 src/PVE/QemuServer/RunState.pm | 32 ++++++++++++++++----------------
 2 files changed, 22 insertions(+), 16 deletions(-)


Summary over all repositories:
  41 files changed, 105 insertions(+), 1181 deletions(-)

-- 
Generated by git-murpp 0.5.0




^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2026-06-29 13:55 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-06-29 13:54 [PATCH-SERIES qemu/qemu-server v2 0/5] fix #6424: avoid timeout issue for QMP 'quit' for bulk suspend Fiona Ebner
2026-06-29 13:54 ` [PATCH qemu v2 1/5] async snapshot: allow skipping VM start after successful completion Fiona Ebner
2026-06-29 13:54 ` [PATCH qemu v2 2/5] update submodule and patches to QEMU 11.0.2 Fiona Ebner
2026-06-29 13:54 ` [PATCH qemu-server v2 3/5] fix #6424: increase timeout for QMP 'quit' to 60s to avoid issue with bulk suspend Fiona Ebner
2026-06-29 13:54 ` [PATCH qemu-server v2 4/5] run state: use v5.36 and signatures in module Fiona Ebner
2026-06-29 13:54 ` [PATCH qemu-server v2 5/5] suspend: skip VM start after successfully saving state Fiona Ebner

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal