On 2016/10/26 13:58, Amit Shah wrote:
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?


Yes, I should not add this in patch 3, i will fix this in next version.

Thanks,
Hailiang

                Amit

.



Reply via email to