Re: [Qemu-devel] [PATCH] migration: Add state records for migration incoming

[zhanghailiang <zhang.zhanghailiang@huawei.com>]

On 2015/11/18 18:51, Juan Quintela wrote:
> zhanghailiang <zhang.zhanghailiang@huawei.com> wrote:
>> For migration destination, sometimes we need to know its state,
>> and it is also useful for tracing migration incoming process.
>>
>> Here we add a new member 'state' for MigrationIncomingState,
>> and also use migrate_set_state() to modify its value.
>> We fix the first parameter of migrate_set_state(), and make it
>> public.
>>
>> Signed-off-by: zhanghailiang <zhang.zhanghailiang@huawei.com>
>> Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
>
Hi Juan,

Thanks for your reply.

> 1st: split the patch about the change in migrate_set_state prototype,
> and the rest.
>

OK, i will do it in next version.

> Once there, if we are going to do this, at least show it on info migrate
> on destination?
>

I don't know if there is any sense to export migration info in destination.
In actual application, we use libvirt to issue migration command and get
migration info in source side. So it seems unnecessary to export them.

For COLO, we only use the state internally, to indicate whether we are in COLO state.

Thanks,
zhanghailiang

> If we are going this way, I think it is going to be better to reuse
> MigrationState?  That way, we could make the info migrate statistics
> easier to understand?
>
> Thanks, Juan.
>
> .
>