On (Tue) 18 Oct 2016 [20:10:10], zhanghailiang wrote: > For primary side, if COLO gets failover request from users. > To be exact, gets 'x_colo_lost_heartbeat' command. > COLO thread will exit the loop while the failover BH does the > cleanup work and resumes VM. > > Signed-off-by: zhanghailiang <zhang.zhanghaili...@huawei.com> > Signed-off-by: Li Zhijian <lizhij...@cn.fujitsu.com> > Reviewed-by: Dr. David Alan Gilbert <dgilb...@redhat.com>
> @@ -162,9 +196,20 @@ static int colo_do_checkpoint_transaction(MigrationState > *s, > @@ -280,8 +330,6 @@ out: > if (local_err) { > error_report_err(local_err); > } > - migrate_set_state(&s->state, MIGRATION_STATUS_COLO, > - MIGRATION_STATUS_COMPLETED); Yea, I guess this should not have been done in the previous patch to begin with? Amit