Re: [RESEND][PATCH] migration: fix xml file residual during vm crash with migration

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 7/27/20 5:28 AM, zhengchuan wrote:
From 935ec812b822ca978631e72bb9b9a5d00df24a42 Mon Sep 17 00:00:00 2001
From: Zheng Chuan <zhengchuan@xxxxxxxxxx>
Date: Mon, 27 Jul 2020 14:39:05 +0800
Subject: [PATCH] migration: fix xml file residual during vm crash with
  migration

I think the wording here would be 'fix residual xml file during ...'


when migration is cancelled (such as kill -9 vmpid in Src, etc), it could

Capital W in 'When'


do virDomainSaveStatus() to save xml file after qemuProcessStop(), which results
in xml residulal.

residulal -> residual


Fix it by that do not do virDomainSaveStatus() if vm is not active.

'Fix that by not doing virDomainSaveStatus() if vm is not active' is clearer.



Signed-off-by: Zheng Chuan <zhengchuan@xxxxxxxxxx>
---
  src/qemu/qemu_migration.c | 3 +++
  1 file changed, 3 insertions(+)

diff --git a/src/qemu/qemu_migration.c b/src/qemu/qemu_migration.c
index 2c7bf34..d2804ab 100644
--- a/src/qemu/qemu_migration.c
+++ b/src/qemu/qemu_migration.c
@@ -3073,6 +3073,9 @@ qemuMigrationSrcConfirmPhase(virQEMUDriverPtr driver,
          qemuMigrationParamsReset(driver, vm, QEMU_ASYNC_JOB_MIGRATION_OUT,
                                   jobPriv->migParams, priv->job.apiFlags);
+ if (!virDomainObjIsActive(vm))
+            goto done;
+

This patch breaks build as is because this function does not have a 'done' label
any more. I changed it here to 'return 0' and it passed the tests. Not sure if
returning 0 here is what you want though.


Thanks,


DHB


          if (virDomainObjSave(vm, driver->xmlopt, cfg->stateDir) < 0)
              VIR_WARN("Failed to save status on vm %s", vm->def->name);
      }





[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux