On 2015/11/18 18:51, Juan Quintela wrote:
zhanghailiang <zhang.zhanghaili...@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.zhanghaili...@huawei.com>
Reviewed-by: Dr. David Alan Gilbert <dgilb...@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.
.
