Re: [Qemu-devel] [PULL 01/16] migration: introduce postcopy-blocktime capability

["Dr. David Alan Gilbert" <dgilbert@redhat.com>]

* Eric Blake (eblake@redhat.com) wrote:
> On 04/25/2018 03:03 PM, Dr. David Alan Gilbert (git) wrote:
> > From: Alexey Perevalov <a.perevalov@samsung.com>
> > 
> > Right now it could be used on destination side to
> > enable vCPU blocktime calculation for postcopy live migration.
> > vCPU blocktime - it's time since vCPU thread was put into
> > interruptible sleep, till memory page was copied and thread awake.
> > 
> > Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> > Signed-off-by: Alexey Perevalov <a.perevalov@samsung.com>
> > Reviewed-by: Juan Quintela <quintela@redhat.com>
> > Signed-off-by: Juan Quintela <quintela@redhat.com>
> > Message-Id: <1521742647-25550-2-git-send-email-a.perevalov@samsung.com>
> > Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> > ---
> 
> > +++ b/qapi/migration.json
> > @@ -354,16 +354,20 @@
> >  #
> >  # @x-multifd: Use more than one fd for migration (since 2.11)
> >  #
> > +#
> >  # @dirty-bitmaps: If enabled, QEMU will migrate named dirty bitmaps.
> 
> Why is the extra blank line added here?  Doesn't hold up the pull
> request, but might be worth a trivial followup patch.

I think it's just a merge artifact; this patch has been around for a few
versions so has been bumped down that list a few times.

OK, I'll do that.

Dave

> >  #                 (since 2.12)
> >  #
> > +# @postcopy-blocktime: Calculate downtime for postcopy live migration
> > +#                     (since 2.13)
> > +#
> >  # Since: 1.2
> >  ##
> >  { 'enum': 'MigrationCapability',
> >    'data': ['xbzrle', 'rdma-pin-all', 'auto-converge', 'zero-blocks',
> >             'compress', 'events', 'postcopy-ram', 'x-colo', 'release-ram',
> >             'block', 'return-path', 'pause-before-switchover', 'x-multifd',
> > -           'dirty-bitmaps' ] }
> > +           'dirty-bitmaps', 'postcopy-blocktime' ] }
> >  
> >  ##
> >  # @MigrationCapabilityStatus:
> > 
> 
> -- 
> Eric Blake, Principal Software Engineer
> Red Hat, Inc.           +1-919-301-3266
> Virtualization:  qemu.org | libvirt.org
> 



--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK