* Zhang, Chen ([email protected]) 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 <[email protected]> > > Sent: Monday, June 8, 2020 3:46 AM > > To: Dr . David Alan Gilbert <[email protected]>; qemu-dev <qemu- > > [email protected]> > > Cc: Zhang Chen <[email protected]>; Zhanghailiang > > <[email protected]>; Zhang, Chen <[email protected]> > > Subject: [PATCH V3 0/3] migration/colo: Optimize COLO framework code > > > > From: Zhang Chen <[email protected]> > > > > 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 / [email protected] / Manchester, UK
