Re: [Qemu-devel] [PATCH v3 4/9] HMP: disable monitor in preconfig state

[Igor Mammedov <imammedo@redhat.com>]

On Wed, 7 Mar 2018 14:01:27 +0000
"Dr. David Alan Gilbert" <dgilbert@redhat.com> wrote:

> * Igor Mammedov (imammedo@redhat.com) wrote:
> > Ban it for now, if someone would need it to work early,
> > one would have to implement checks if HMP command is valid
> > at preconfig state.
> > 
> > Signed-off-by: Igor Mammedov <imammedo@redhat.com>
> > ---
> > ---
> >  monitor.c | 4 ++++
> >  1 file changed, 4 insertions(+)
> > 
> > diff --git a/monitor.c b/monitor.c
> > index f499250..fcb5386 100644
> > --- a/monitor.c
> > +++ b/monitor.c
> > @@ -3097,6 +3097,10 @@ static void handle_hmp_command(Monitor *mon, const char *cmdline)
> >  
> >      trace_handle_hmp_command(mon, cmdline);
> >  
> > +    if (runstate_check(RUN_STATE_PRECONFIG)) {
> > +        monitor_printf(mon, "HMP not available in precofig state\n");
> > +    }  
> 
> But you've not returned, so the command will still get parsed?
yep, it's bug. I've already fixed in v4.
Plan to post simplified v4 this Friday if nothing goes south.

> 
> Dave
> 
> >      cmd = monitor_parse_command(mon, cmdline, &cmdline, mon->cmd_table);
> >      if (!cmd) {
> >          return;
> > -- 
> > 2.7.4
> > 
> >   
> --
> Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK