[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/6] Migration: Emit event at start of pass
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [Qemu-devel] [PATCH 4/6] Migration: Emit event at start of pass |
Date: |
Wed, 16 Dec 2015 10:01:43 +0000 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
* Eric Blake (address@hidden) wrote:
> On 12/15/2015 12:02 PM, Dr. David Alan Gilbert (git) wrote:
> > From: "Dr. David Alan Gilbert" <address@hidden>
> >
> > Emit an event each time we sync the dirty bitmap on the source;
> > this helps libvirt use postcopy by giving it a kick when it
> > might be a good idea to start the postcopy.
> >
> > Signed-off-by: Dr. David Alan Gilbert <address@hidden>
> > ---
> > migration/ram.c | 4 ++++
> > qapi/event.json | 11 +++++++++++
> > 2 files changed, 15 insertions(+)
>
> > +++ b/qapi/event.json
> > @@ -255,6 +255,17 @@
> > 'data': {'status': 'MigrationStatus'}}
> >
> > ##
> > +# @MIGRATION_PASS
> > +#
> > +# Emitted from the source side of a migration at the start of each pass
> > +# (when it syncs the dirty bitmap)
> > +#
> > +# Since: 2.6
>
> Missing documentation of the 'pass' member - is it a counter that says
> which pass this event is, incrementing each time through?
Thanks; yes, I added the pass parameter afterwards; I'll add:
# @pass: An incrementing count (starting at 1 on the first pass)
> Also missing docs in events.txt (although that file may go away soon,
> once we get Marc-Andre's patches in for automating documentation from
> .json files).
OK, I'll add the following (docs/qmp-events.txt):
MIGRATION_PASS
--------------
Emitted from the source side of a migration at the start of each pass
(when it syncs the dirty bitmap)
Data:
- "pass": An incrementing count (starting at 1 on the first pass)
Example:
{"timestamp": {"seconds": 1449669631, "microseconds": 239225},
"event": "MIGRATION_PASS", "data": {"pass": 2}}
Dave
>
> --
> Eric Blake eblake redhat com +1-919-301-3266
> Libvirt virtualization library http://libvirt.org
>
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK
- Re: [Qemu-devel] [PATCH 1/6] migration: Export migrate_set_state(), (continued)
- [Qemu-devel] [PATCH 3/6] Postcopy: Send events/change state on incoming side, Dr. David Alan Gilbert (git), 2015/12/15
- [Qemu-devel] [PATCH 2/6] migration: Add state records for migration incoming, Dr. David Alan Gilbert (git), 2015/12/15
- [Qemu-devel] [PATCH 5/6] Use qemu_get_buffer_in_place for xbzrle data, Dr. David Alan Gilbert (git), 2015/12/15
- [Qemu-devel] [PATCH 4/6] Migration: Emit event at start of pass, Dr. David Alan Gilbert (git), 2015/12/15
- Re: [Qemu-devel] [PATCH 4/6] Migration: Emit event at start of pass, Juan Quintela, 2015/12/16
- [Qemu-devel] [PATCH 6/6] multithread decompression: Avoid one copy, Dr. David Alan Gilbert (git), 2015/12/15