Re: [Qemu-devel] [PATCH 3/4] monitor, console: add QEVENT_SCREEN_DUMP_COMPLETE

[Luiz Capitulino]

On Tue, 28 Feb 2012 22:51:44 +0200
Alon Levy <address@hidden> wrote:

> On Tue, Feb 28, 2012 at 05:01:13PM -0300, Luiz Capitulino wrote:
> > On Fri, 24 Feb 2012 23:22:04 +0200
> > Alon Levy <address@hidden> wrote:
> > 
> > > Signed-off-by: Alon Levy <address@hidden>
> > > ---
> > >  QMP/qmp-events.txt |   14 ++++++++++++++
> > >  console.c          |   18 ++++++++++++++++++
> > >  console.h          |    1 +
> > >  monitor.c          |    2 ++
> > >  monitor.h          |    1 +
> > >  5 files changed, 36 insertions(+), 0 deletions(-)
> > > 
> > > diff --git a/QMP/qmp-events.txt b/QMP/qmp-events.txt
> > > index 9286af5..0d1c9f4 100644
> > > --- a/QMP/qmp-events.txt
> > > +++ b/QMP/qmp-events.txt
> > > @@ -335,3 +335,17 @@ Example:
> > >                 "len": 10737418240, "offset": 134217728,
> > >                 "speed": 0 },
> > >       "timestamp": { "seconds": 1267061043, "microseconds": 959568 } }
> > > +
> > > +SCREEN_DUMP_COMPLETE
> > > +--------------------
> > > +
> > > +Emitted when screen-dump-async completes.
> > > +
> > > +Data:
> > > +
> > > +- "filename": Name of file containing screen dump (json-string)
> > > +
> > > +Example:
> > > +
> > > +{ "event": "SCREEN_DUMP_COMPLETE",
> > > +     "data": { "filename": "/tmp/a.ppm" } }
> > 
> > This example is incomplete, a QMP event contains more fields.
> 
> timestamp, right?

Yes.