Re: [Qemu-devel] [PATCH v7 04/11] hmp: disable monitor in preconfig state

2018-06-01 Thread Dr. David Alan Gilbert
* Igor Mammedov (imamm...@redhat.com) wrote: > On Fri, 25 May 2018 16:39:34 -0300 > Eduardo Habkost wrote: > > > On Fri, May 25, 2018 at 08:05:30AM +0200, Markus Armbruster wrote: > > > Eduardo Habkost writes: > > > > > > > On Thu, May 24, 2018 at 08:16:20PM +0200, Markus Armbruster wrote:

Re: [Qemu-devel] [PATCH v7 04/11] hmp: disable monitor in preconfig state

2018-05-28 Thread Igor Mammedov
On Fri, 25 May 2018 16:39:34 -0300 Eduardo Habkost wrote: > On Fri, May 25, 2018 at 08:05:30AM +0200, Markus Armbruster wrote: > > Eduardo Habkost writes: > > > > > On Thu, May 24, 2018 at 08:16:20PM +0200, Markus Armbruster wrote: > > >> Markus Armbruster writes: > > >> > > >> > Igor M

Re: [Qemu-devel] [PATCH v7 04/11] hmp: disable monitor in preconfig state

2018-05-25 Thread Eduardo Habkost
On Fri, May 25, 2018 at 08:05:30AM +0200, Markus Armbruster wrote: > Eduardo Habkost writes: > > > On Thu, May 24, 2018 at 08:16:20PM +0200, Markus Armbruster wrote: > >> Markus Armbruster writes: > >> > >> > Igor Mammedov writes: > >> > > >> >> Ban it for now, if someone would need it to work

Re: [Qemu-devel] [PATCH v7 04/11] hmp: disable monitor in preconfig state

2018-05-24 Thread Markus Armbruster
Eduardo Habkost writes: > On Thu, May 24, 2018 at 08:16:20PM +0200, Markus Armbruster wrote: >> Markus Armbruster writes: >> >> > Igor Mammedov writes: >> > >> >> Ban it for now, if someone would need it to work early, >> >> one would have to implement checks if HMP command is valid >> >> at p

Re: [Qemu-devel] [PATCH v7 04/11] hmp: disable monitor in preconfig state

2018-05-24 Thread Eduardo Habkost
On Thu, May 24, 2018 at 08:16:20PM +0200, Markus Armbruster wrote: > Markus Armbruster writes: > > > Igor Mammedov writes: > > > >> 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-of

Re: [Qemu-devel] [PATCH v7 04/11] hmp: disable monitor in preconfig state

2018-05-24 Thread Markus Armbruster
Markus Armbruster writes: > Igor Mammedov writes: > >> 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 >> Reviewed-by: Eric Blake >> --- >> v5: >> * add 'use QMP in

Re: [Qemu-devel] [PATCH v7 04/11] hmp: disable monitor in preconfig state

2018-05-24 Thread Markus Armbruster
Igor Mammedov writes: > 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 > Reviewed-by: Eric Blake > --- > v5: > * add 'use QMP instead" to error message, suggesting user

[Qemu-devel] [PATCH v7 04/11] hmp: disable monitor in preconfig state

2018-05-04 Thread Igor Mammedov
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 Reviewed-by: Eric Blake --- v5: * add 'use QMP instead" to error message, suggesting user the right interface to use v4: * v