* Zhang, Chen (chen.zh...@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.zh...@intel.com> > > Sent: Monday, June 8, 2020 3:46 AM > > To: Dr . David Alan Gilbert <dgilb...@redhat.com>; qemu-dev <qemu- > > de...@nongnu.org> > > Cc: Zhang Chen <zhangc...@gmail.com>; Zhanghailiang > > <zhang.zhanghaili...@huawei.com>; Zhang, Chen <chen.zh...@intel.com> > > Subject: [PATCH V3 0/3] migration/colo: Optimize COLO framework code > > > > From: Zhang Chen <chen.zh...@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 / dgilb...@redhat.com / Manchester, UK
