Re: [PATCH V3 0/3] migration/colo: Optimize COLO framework code

[Dr. David Alan Gilbert]

* Zhang, Chen (chen.zhang@intel.com) wrote:
> Hi Dave,
> 

I think it's OK; I guess Zhanghailiang is going to do a pull for it?
One thought about the change of use of MIGRATION_STATUS_ACTIVE; I think
it's OK at the moment - but the migration states and the way they change
becomes part of the API - changing which states we use can confuse
management applications; since there's no libvirt support, I'll leave
it to Zhanghailiang to know if that's OK.

Dave


> 
> Thanks
> Zhang Chen
> 
> > -----Original Message-----
> > From: Zhang, Chen <chen.zhang@intel.com>
> > Sent: Monday, June 8, 2020 3:46 AM
> > To: Dr . David Alan Gilbert <dgilbert@redhat.com>; qemu-dev <qemu-
> > devel@nongnu.org>
> > Cc: Zhang Chen <zhangckid@gmail.com>; Zhanghailiang
> > <zhang.zhanghailiang@huawei.com>; Zhang, Chen <chen.zhang@intel.com>
> > Subject: [PATCH V3 0/3] migration/colo: Optimize COLO framework code
> > 
> > From: Zhang Chen <chen.zhang@intel.com>
> > 
> > This series optimize some code of COLO, please review.
> > 
> > Zhang Chen (3):
> >   migration/colo: Optimize COLO boot code path
> >   migration/colo: Update checkpoint time lately
> >   migration/migration.c: Remove MIGRATION_STATUS_ACTIVE in
> >     migration_iteration_finish
> > 
> >  migration/colo.c      |  7 ++-----
> >  migration/migration.c | 12 +++++-------
> >  2 files changed, 7 insertions(+), 12 deletions(-)
> > 
> > --
> > 2.17.1
> 
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK