[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] migration: Fix possible bug for migrate cancel
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH] migration: Fix possible bug for migrate cancel |
Date: |
Mon, 24 Mar 2014 10:00:35 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.3.0 |
[adding libvirt]
On 03/24/2014 09:47 AM, Paolo Bonzini wrote:
> Il 24/03/2014 14:04, address@hidden ha scritto:
>> From: zengjunliang <address@hidden>
>>
>> Return error for migrate cancel, when migration status is not
>> MIG_STATE_SETUP or MIG_STATE_ACTIVE. Thus, libvirt can can
>> perceive the operation fails.
>>
>> Signed-off-by: zengjunliang <address@hidden>
>> Signed-off-by: Gonglei <address@hidden>
>
> I think this is done on purpose, because canceling migration is racy.
> Instead, libvirt should do "query-migrate" and check if the migration
> was completed or canceled.
Can you please give more details at how you are triggering the problem
with libvirt? I think Paolo is probably right - the bug is more likely
to be in libvirt not expecting the race and not recovering correctly
when the race occurs, than it is to be in changing qemu's state algorithm.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH] migration: Fix possible bug for migrate cancel, arei.gonglei, 2014/03/24
- Re: [Qemu-devel] [PATCH] migration: Fix possible bug for migrate cancel, Eric Blake, 2014/03/24
- Re: [Qemu-devel] [PATCH] migration: Fix possible bug for migrate cancel, Paolo Bonzini, 2014/03/24
- Re: [Qemu-devel] [PATCH] migration: Fix possible bug for migrate cancel,
Eric Blake <=
- Re: [Qemu-devel] [PATCH] migration: Fix possible bug for migrate cancel, Gonglei (Arei), 2014/03/25
- Re: [Qemu-devel] [PATCH] migration: Fix possible bug for migrate cancel, Gonglei (Arei), 2014/03/28
- Re: [Qemu-devel] [PATCH] migration: Fix possible bug for migrate cancel, Paolo Bonzini, 2014/03/28
- Re: [Qemu-devel] [PATCH] migration: Fix possible bug for migrate cancel, Dr. David Alan Gilbert, 2014/03/28
- Re: [Qemu-devel] [PATCH] migration: Fix possible bug for migrate cancel, Paolo Bonzini, 2014/03/28