Patchwork [v2,2/2] Reset system before loadvm

login
register
mail settings
Submitter Jan Kiszka
Date June 14, 2011, 4:29 p.m.
Message ID <4DF78C79.40606@siemens.com>
Download mbox | patch
Permalink /patch/100371/
State New
Headers show

Comments

Jan Kiszka - June 14, 2011, 4:29 p.m.
In case we load the vmstate during incoming migration, we start from a
clean, default machine state as we went through system reset before. But
if we load from a snapshot, the machine can be in any state. That can
cause troubles if loading an older image which does not carry all state
information the executing QEMU requires. Hardly any device takes care of
this scenario.

However, fixing this is trivial. We just need to issue a system reset
during loadvm as well.

Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
---
 savevm.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

Patch

diff --git a/savevm.c b/savevm.c
index 939845c..8139bc7 100644
--- a/savevm.c
+++ b/savevm.c
@@ -2073,6 +2073,7 @@  int load_vmstate(const char *name)
         return -EINVAL;
     }
 
+    qemu_system_reset(VMRESET_SILENT);
     ret = qemu_loadvm_state(f);
 
     qemu_fclose(f);