[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 1/1] migration: announce VM's new home just before V
From: |
Amit Shah |
Subject: |
[Qemu-devel] [PATCH 1/1] migration: announce VM's new home just before VM is runnable |
Date: |
Wed, 14 Oct 2015 17:37:19 +0530 |
We were announcing the dest host's IP as our new IP a bit too soon -- if
there were errors detected after this announcement was done, the
migration is failed and the VM could continue running on the src host --
causing problems later.
Move around the qemu_announce_self() call so it's done just before the
VM is runnable.
Signed-off-by: Amit Shah <address@hidden>
---
migration/migration.c | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/migration/migration.c b/migration/migration.c
index b7de9b7..ca21ba8 100644
--- a/migration/migration.c
+++ b/migration/migration.c
@@ -295,7 +295,6 @@ static void process_incoming_migration_co(void *opaque)
exit(EXIT_FAILURE);
}
migrate_generate_event(MIGRATION_STATUS_COMPLETED);
- qemu_announce_self();
/* Make sure all file formats flush their mutable metadata */
bdrv_invalidate_cache_all(&local_err);
@@ -305,6 +304,12 @@ static void process_incoming_migration_co(void *opaque)
exit(EXIT_FAILURE);
}
+ /*
+ * This must happen after all error conditions are dealt with and
+ * we're sure the VM is going to be running on this host.
+ */
+ qemu_announce_self();
+
/* If global state section was not received or we are in running
state, we need to obey autostart. Any other state is set with
runstate_set. */
--
2.4.3
- [Qemu-devel] [PATCH 1/1] migration: announce VM's new home just before VM is runnable,
Amit Shah <=
- Re: [Qemu-devel] [PATCH 1/1] migration: announce VM's new home just before VM is runnable, Michael S. Tsirkin, 2015/10/14
- Re: [Qemu-devel] [PATCH 1/1] migration: announce VM's new home just before VM is runnable, Dr. David Alan Gilbert, 2015/10/14
- Re: [Qemu-devel] [PATCH 1/1] migration: announce VM's new home just before VM is runnable, Juan Quintela, 2015/10/14
- Re: [Qemu-devel] [PATCH 1/1] migration: announce VM's new home just before VM is runnable, Michael S. Tsirkin, 2015/10/14
- Re: [Qemu-devel] [PATCH 1/1] migration: announce VM's new home just before VM is runnable, Jason Wang, 2015/10/15
- Re: [Qemu-devel] [PATCH 1/1] migration: announce VM's new home just before VM is runnable, Amit Shah, 2015/10/15
- Re: [Qemu-devel] [PATCH 1/1] migration: announce VM's new home just before VM is runnable, Juan Quintela, 2015/10/15
- Re: [Qemu-devel] [PATCH 1/1] migration: announce VM's new home just before VM is runnable, Amit Shah, 2015/10/15
- Re: [Qemu-devel] [PATCH 1/1] migration: announce VM's new home just before VM is runnable, Michael S. Tsirkin, 2015/10/15