On Wed, Jan 03, 2018 at 10:10:21AM +0100, Juan Quintela wrote: > Peter Xu <pet...@redhat.com> wrote: > > Introduce MigrationState.vm_down_start_time to replace the local > > variable "start_time" in migration_thread to avoid passing things around. > > > > Signed-off-by: Peter Xu <pet...@redhat.com> > > Reviewed-by: Juan Quintela <quint...@redhat.com> > > But I would suggest renaming the variable? > > Later, Juan. > > > > diff --git a/migration/migration.h b/migration/migration.h > > index 0f5df2367c..3ab5506233 100644 > > --- a/migration/migration.h > > +++ b/migration/migration.h > > @@ -107,6 +107,8 @@ struct MigrationState > > int64_t mig_start_time; > > /* Total time used by latest migration (ms) */ > > int64_t mig_total_time; > > + /* Timestamp when VM is down (ms) to migrate the last stuff */ > > + int64_t vm_down_start_time; > > downtime_start?
Will do. -- Peter Xu
