[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 16/47] Return path: Source handling of return
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH v4 16/47] Return path: Source handling of return path |
Date: |
Fri, 24 Oct 2014 12:04:25 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 |
On 10/23/2014 08:00 PM, Dr. David Alan Gilbert wrote:
>>> > > +static bool migration_already_active(MigrationState *ms)
>>> > > +{
>>> > > + switch (ms->state) {
>>> > > + case MIG_STATE_ACTIVE:
>>> > > + case MIG_STATE_SETUP:
>>> > > + return true;
>>> > > +
>>> > > + default:
>>> > > + return false;
>>> > > +
>>> > > + }
>>> > > +}
>> >
>> > Should CANCELLING also be considered active? It is on the source->dest
>> > path.
> Hmm, possibly - but my intention here was just to round up all of the
> places that already checked for ACTIVE+SETUP so that I could add
> POSTCOPY_ACTIVE;
> only one of those places also checked for CANCELLING, so I left it out.
Ok, I would need to check the callers... There may be bugs waiting to
be fixed. :) For now I guess it's ok as is.
Thanks for answering my comments!
Paolo
[Qemu-devel] [PATCH v4 15/47] Return path: Send responses from destination to source, Dr. David Alan Gilbert (git), 2014/10/03
[Qemu-devel] [PATCH v4 17/47] qemu_loadvm errors and debug, Dr. David Alan Gilbert (git), 2014/10/03
[Qemu-devel] [PATCH v4 18/47] ram_debug_dump_bitmap: Dump a migration bitmap as text, Dr. David Alan Gilbert (git), 2014/10/03
[Qemu-devel] [PATCH v4 19/47] Rework loadvm path for subloops, Dr. David Alan Gilbert (git), 2014/10/03