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? > int64_t downtime; > int64_t expected_downtime; > bool enabled_capabilities[MIGRATION_CAPABILITY__MAX];
