[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/1] migration: announce VM's new home just befo
From: |
Jason Wang |
Subject: |
Re: [Qemu-devel] [PATCH 1/1] migration: announce VM's new home just before VM is runnable |
Date: |
Thu, 15 Oct 2015 09:50:50 +0800 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 10/14/2015 08:07 PM, Amit Shah wrote:
> 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. */
Acked-by: Jason Wang <address@hidden>
- Re: [Qemu-devel] [PATCH 1/1] migration: announce VM's new home just before VM is runnable, (continued)
- 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
Re: [Qemu-devel] [PATCH 1/1] migration: announce VM's new home just before VM is runnable,
Jason Wang <=