a583c50c4e
- kvm-migration-Add-helper-to-get-target-runstate.patch [RHEL-54670] - kvm-qmp-cont-Only-activate-disks-if-migration-completed.patch [RHEL-54670] - kvm-migration-block-Make-late-block-active-the-default.patch [RHEL-54670] - kvm-migration-block-Apply-late-block-active-behavior-to-.patch [RHEL-54670] - kvm-migration-block-Fix-possible-race-with-block_inactiv.patch [RHEL-54670] - kvm-migration-block-Rewrite-disk-activation.patch [RHEL-54670] - kvm-block-Add-active-field-to-BlockDeviceInfo.patch [RHEL-54670] - kvm-block-Allow-inactivating-already-inactive-nodes.patch [RHEL-54670] - kvm-block-Inactivate-external-snapshot-overlays-when-nec.patch [RHEL-54670] - kvm-migration-block-active-Remove-global-active-flag.patch [RHEL-54670] - kvm-block-Don-t-attach-inactive-child-to-active-node.patch [RHEL-54670] - kvm-block-Fix-crash-on-block_resize-on-inactive-node.patch [RHEL-54670] - kvm-block-Add-option-to-create-inactive-nodes.patch [RHEL-54670] - kvm-block-Add-blockdev-set-active-QMP-command.patch [RHEL-54670] - kvm-block-Support-inactive-nodes-in-blk_insert_bs.patch [RHEL-54670] - kvm-block-export-Don-t-ignore-image-activation-error-in-.patch [RHEL-54670] - kvm-block-Drain-nodes-before-inactivating-them.patch [RHEL-54670] - kvm-block-export-Add-option-to-allow-export-of-inactive-.patch [RHEL-54670] - kvm-nbd-server-Support-inactive-nodes.patch [RHEL-54670] - kvm-iotests-Add-filter_qtest.patch [RHEL-54670] - kvm-iotests-Add-qsd-migrate-case.patch [RHEL-54670] - kvm-iotests-Add-NBD-based-tests-for-inactive-nodes.patch [RHEL-54670] - Resolves: RHEL-54670 (Provide QMP command for block device reactivation after migration [rhel-10.0])
74 lines
2.6 KiB
Diff
74 lines
2.6 KiB
Diff
From fd5603e42b6287c849bfed700d34c817b4b93891 Mon Sep 17 00:00:00 2001
|
|
From: Peter Xu <peterx@redhat.com>
|
|
Date: Fri, 6 Dec 2024 18:08:34 -0500
|
|
Subject: [PATCH 02/22] qmp/cont: Only activate disks if migration completed
|
|
|
|
RH-Author: Kevin Wolf <kwolf@redhat.com>
|
|
RH-MergeRequest: 340: QMP command for block device reactivation after migration
|
|
RH-Jira: RHEL-54670
|
|
RH-Acked-by: Eric Blake <eblake@redhat.com>
|
|
RH-Acked-by: Stefan Hajnoczi <stefanha@redhat.com>
|
|
RH-Commit: [2/22] 554508370a344aa69e0e8888c0567d353cbcfe94 (kmwolf/centos-qemu-kvm)
|
|
|
|
As the comment says, the activation of disks is for the case where
|
|
migration has completed, rather than when QEMU is still during
|
|
migration (RUN_STATE_INMIGRATE).
|
|
|
|
Move the code over to reflect what the comment is describing.
|
|
|
|
Cc: Kevin Wolf <kwolf@redhat.com>
|
|
Cc: Markus Armbruster <armbru@redhat.com>
|
|
Signed-off-by: Peter Xu <peterx@redhat.com>
|
|
Reviewed-by: Fabiano Rosas <farosas@suse.de>
|
|
Message-Id: <20241206230838.1111496-3-peterx@redhat.com>
|
|
Signed-off-by: Fabiano Rosas <farosas@suse.de>
|
|
(cherry picked from commit e4e5e89bbd8e731e86735d9d25b7b5f49e8f08b6)
|
|
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
|
|
---
|
|
monitor/qmp-cmds.c | 26 ++++++++++++++------------
|
|
1 file changed, 14 insertions(+), 12 deletions(-)
|
|
|
|
diff --git a/monitor/qmp-cmds.c b/monitor/qmp-cmds.c
|
|
index f84a0dc523..76f21e8af3 100644
|
|
--- a/monitor/qmp-cmds.c
|
|
+++ b/monitor/qmp-cmds.c
|
|
@@ -96,21 +96,23 @@ void qmp_cont(Error **errp)
|
|
}
|
|
}
|
|
|
|
- /* Continuing after completed migration. Images have been inactivated to
|
|
- * allow the destination to take control. Need to get control back now.
|
|
- *
|
|
- * If there are no inactive block nodes (e.g. because the VM was just
|
|
- * paused rather than completing a migration), bdrv_inactivate_all() simply
|
|
- * doesn't do anything. */
|
|
- bdrv_activate_all(&local_err);
|
|
- if (local_err) {
|
|
- error_propagate(errp, local_err);
|
|
- return;
|
|
- }
|
|
-
|
|
if (runstate_check(RUN_STATE_INMIGRATE)) {
|
|
autostart = 1;
|
|
} else {
|
|
+ /*
|
|
+ * Continuing after completed migration. Images have been
|
|
+ * inactivated to allow the destination to take control. Need to
|
|
+ * get control back now.
|
|
+ *
|
|
+ * If there are no inactive block nodes (e.g. because the VM was
|
|
+ * just paused rather than completing a migration),
|
|
+ * bdrv_inactivate_all() simply doesn't do anything.
|
|
+ */
|
|
+ bdrv_activate_all(&local_err);
|
|
+ if (local_err) {
|
|
+ error_propagate(errp, local_err);
|
|
+ return;
|
|
+ }
|
|
vm_start();
|
|
}
|
|
}
|
|
--
|
|
2.39.3
|
|
|