[libvirt] [PATCH v3 6/7] qemu: Skip starting NBD servers for offline migration
Jiri Denemark
jdenemar at redhat.com
Wed Nov 25 09:29:38 UTC 2015
NBD storage migration will not work with offline migration anyway and we
already checked that the user did not ask for it. Thus it doesn't make
sense to keep the code after 'done' label where we jump in case of
offline migration.
Signed-off-by: Jiri Denemark <jdenemar at redhat.com>
---
Notes:
ACKed in version 1
Version 3:
- no change
Version 2:
- no change
src/qemu/qemu_migration.c | 29 +++++++++++++----------------
1 file changed, 13 insertions(+), 16 deletions(-)
diff --git a/src/qemu/qemu_migration.c b/src/qemu/qemu_migration.c
index 9a4f19f..b6525df 100644
--- a/src/qemu/qemu_migration.c
+++ b/src/qemu/qemu_migration.c
@@ -3418,6 +3418,9 @@ qemuMigrationPrepareAny(virQEMUDriverPtr driver,
"make sense"));
goto cleanup;
}
+ cookieFlags = 0;
+ } else {
+ cookieFlags = QEMU_MIGRATION_COOKIE_GRAPHICS;
}
if (!(caps = virQEMUDriverGetCapabilities(driver, false)))
@@ -3572,6 +3575,16 @@ qemuMigrationPrepareAny(virQEMUDriverPtr driver,
QEMU_ASYNC_JOB_MIGRATION_IN) < 0)
goto stopjob;
+ if (mig->nbd &&
+ flags & (VIR_MIGRATE_NON_SHARED_DISK | VIR_MIGRATE_NON_SHARED_INC) &&
+ virQEMUCapsGet(priv->qemuCaps, QEMU_CAPS_NBD_SERVER)) {
+ if (qemuMigrationStartNBDServer(driver, vm, listenAddress,
+ nmigrate_disks, migrate_disks) < 0) {
+ goto stopjob;
+ }
+ cookieFlags |= QEMU_MIGRATION_COOKIE_NBD;
+ }
+
if (mig->lockState) {
VIR_DEBUG("Received lockstate %s", mig->lockState);
VIR_FREE(priv->lockState);
@@ -3582,22 +3595,6 @@ qemuMigrationPrepareAny(virQEMUDriverPtr driver,
}
done:
- if (flags & VIR_MIGRATE_OFFLINE)
- cookieFlags = 0;
- else
- cookieFlags = QEMU_MIGRATION_COOKIE_GRAPHICS;
-
- if (mig->nbd &&
- flags & (VIR_MIGRATE_NON_SHARED_DISK | VIR_MIGRATE_NON_SHARED_INC) &&
- virQEMUCapsGet(priv->qemuCaps, QEMU_CAPS_NBD_SERVER)) {
- if (qemuMigrationStartNBDServer(driver, vm, listenAddress,
- nmigrate_disks, migrate_disks) < 0) {
- /* error already reported */
- goto stopjob;
- }
- cookieFlags |= QEMU_MIGRATION_COOKIE_NBD;
- }
-
if (qemuMigrationBakeCookie(mig, driver, vm, cookieout,
cookieoutlen, cookieFlags) < 0) {
/* We could tear down the whole guest here, but
--
2.6.3
More information about the libvir-list
mailing list