qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [RFC PATCH 08/22] Monitor commands


From: Kirill Batuzov
Subject: Re: [Qemu-devel] [RFC PATCH 08/22] Monitor commands
Date: Wed, 2 Jul 2014 14:54:31 +0400 (MSK)
User-agent: Alpine 2.02 (DEB 1266 2009-07-14)


On Wed, 2 Jul 2014, Pavel Dovgaluk wrote:

> > -----Original Message-----
> > From: Eric Blake [mailto:address@hidden
> > Sent: Tuesday, July 01, 2014 9:33 PM
> > To: Pavel Dovgaluk; 'QEMU Developers'
> > Subject: Re: [Qemu-devel] [RFC PATCH 08/22] Monitor commands
> > 
> > On 07/01/2014 05:23 AM, Pavel Dovgaluk wrote:
> > > This patch adds commands to control replay from the monitor.
> > >
> > > Signed-off-by: Pavel Dovgalyuk <address@hidden>
> > > ---
> > 
> > What are the counterpart QMP commands?  It is seldom right to add an HMP
> > command unless the same thing can be done via QMP.
> 
> We haven't controlled qemu through QMP yet, thus haven't implemented QMP 
> support
> for replay commands.
> 
> Today I started adding this support and noticed, that QMP in current master 
> branch is
> broken - it does not reply anything except information at start.
>

On Windows hosts only. I've sent a fix for this problem yesterday:
https://lists.gnu.org/archive/html/qemu-devel/2014-07/msg00169.html

> > > @@ -2596,6 +2734,7 @@ int monitor_handle_fd_param2(Monitor *mon, const 
> > > char *fdname, Error
> > **errp)
> > >      return fd;
> > >  }
> > >
> > > +
> > >  /* Please update hmp-commands.hx when adding or changing commands */
> > >  static mon_cmd_t info_cmds[] = {
> > 
> > Spurious newline addition?
> 
> Thanks, forgot to undo this.
> 
> Pavel Dovgaluk
> 
> 
>
-- 
Kirill



reply via email to

[Prev in Thread] Current Thread [Next in Thread]