* Amit Shah (amit.s...@redhat.com) wrote:
> On (Tue) 16 Jun 2015 [11:26:36], Dr. David Alan Gilbert (git) wrote:
>
> > - if (s->state == MIGRATION_STATUS_ACTIVE ||
> > - s->state == MIGRATION_STATUS_SETUP) {
> > + if (migration_already_active(s)) {
>
> (I know, not introduced here, but:)
>
> A better name is migration_is_active()
Done.
>
> > +bool migration_postcopy_phase(MigrationState *s)
> > +{
> > + return (s->state == MIGRATION_STATUS_POSTCOPY_ACTIVE);
> > +}
>
> And this is better named migration_in_postcopy()
Done
>
> otherwise,
>
> Reviewed-by: Amit Shah <amit.s...@redhat.com>
>
>
>
> Amit
--
Dr. David Alan Gilbert / dgilb...@redhat.com / Manchester, UK
