[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 0/6] Obtain dirty bitmap via VM logging

From: ChenLiang
Subject: Re: [Qemu-devel] [PATCH 0/6] Obtain dirty bitmap via VM logging
Date: Thu, 22 May 2014 20:57:57 +0800
User-agent: Mozilla/5.0 (Windows NT 6.1; rv:11.0) Gecko/20120327 Thunderbird/11.0.1

On 2014/5/22 19:21, Sanidhya Kashyap wrote:

> On Wed, May 21, 2014 at 12:15 PM, ChenLiang <address@hidden> wrote:
>> On 2014/5/21 12:56, Sanidhya Kashyap wrote:
>>> On Wed, May 21, 2014 at 9:43 AM, ChenLiang <address@hidden> wrote:
>>>> Hi,
>>>> Nice job. We should avoid running migration_thread and 
>>>> bitmap_logging_thread simultaneously.
>>> Any particular suggestion to avoid running simultaneous execution of
>>> the threads?
>> We can do it like this:
>> https://lists.gnu.org/archive/html/qemu-devel/2014-04/msg02185.html
> Hi Chen,
> I have some questions which I wanted  to get clarified before
> implementing the above part of avoiding the simultaneous execution.
> As the migration process is going on, the RUN_STATE_RUNNING state is
> being used, which is the same case in bitmap dump process. In order to
> use the concept of RUN_STATE_* states, should I create two new states
> RUN_STATE_OUTMIGRATE state will have all the transitions that
> RUN_STATE_RUNNING supports except the ones being used by migration.
> Similarly, RUN_STATE_MIGRATE will support all the transitions of
> We might also have to update the runstate_is_running function, since
> RUN_STATE_LOGBITMAP and RUN_STATE_MIGRATE are almost similar in nature
> to the RUN_STATE_RUNNING. What is your opinion about that?
> Is it the only approach to do it or are there any simple and efficient
> approaches?
> Thanks,
> Sanidhya

Hmm, I think we need the two flags. Although it is little hard to do it. 
Because the vm can't
do something like hotplug vcpu or device during migrate. But for now qemu don't 
guarantee it(in my reading).
So we should change the vm state  RUN_STATE_RUNNING to RUN_STATE_MIGRATE when 
BTW, it is not your patches problem.

cc:  Juan

Best regards

reply via email to

[Prev in Thread] Current Thread [Next in Thread]