Reviewed-by: zhanghailiang <[email protected]> Hmm, How much time it spends on preparing before COLO process ?
> -----Original Message----- > From: Zhang Chen [mailto:[email protected]] > Sent: Friday, May 15, 2020 12:28 PM > To: Dr . David Alan Gilbert <[email protected]>; Juan Quintela > <[email protected]>; Zhanghailiang <[email protected]>; > qemu-dev <[email protected]> > Cc: Zhang Chen <[email protected]>; Jason Wang > <[email protected]>; Zhang Chen <[email protected]> > Subject: [PATCH 2/3] migration/colo: Update checkpoint time lately > > From: Zhang Chen <[email protected]> > > Previous operation(like vm_start and replication_start_all) will consume > extra time for first forced synchronization, so reduce it in this patch. > > Signed-off-by: Zhang Chen <[email protected]> > --- > migration/colo.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/migration/colo.c b/migration/colo.c index > 5ef69b885d..d5bced22cb 100644 > --- a/migration/colo.c > +++ b/migration/colo.c > @@ -531,7 +531,6 @@ static void colo_process_checkpoint(MigrationState > *s) { > QIOChannelBuffer *bioc; > QEMUFile *fb = NULL; > - int64_t current_time = qemu_clock_get_ms(QEMU_CLOCK_HOST); > Error *local_err = NULL; > int ret; > > @@ -580,8 +579,8 @@ static void colo_process_checkpoint(MigrationState > *s) > qemu_mutex_unlock_iothread(); > trace_colo_vm_state_change("stop", "run"); > > - timer_mod(s->colo_delay_timer, > - current_time + s->parameters.x_checkpoint_delay); > + timer_mod(s->colo_delay_timer, > qemu_clock_get_ms(QEMU_CLOCK_HOST) + > + s->parameters.x_checkpoint_delay); > > while (s->state == MIGRATION_STATUS_COLO) { > if (failover_get_state() != FAILOVER_STATUS_NONE) { > -- > 2.17.1
