Re: [PATCH v5 00/50] Make HMP optional (and later standalone)

["Dr. David Alan Gilbert" <dave@treblig.org>]

* Philippe Mathieu-Daudé (philmd@oss.qualcomm.com) wrote:
> On 28/8/26 14:03, Marc-André Lureau wrote:
> > Hi,
> > 
> > This series makes HMP compile-time optional via --disable-hmp. The
> > resulting binary speaks only QMP - reducing the attack surface, binary
> > size and enforcing a separation between machine-readable interface and
> > interactive debugging console.
> > 
> > QMP offers almost all the functionality HMP could provide, so a separate
> > standalone "HMP-like" console could be implemented on top of it. The
> > rough list of missing commands (+complexity estimation) after this
> > series:
> 
> 
> > Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
> > ---
> > Changes in v5:
> > - renamed local variable hmp_mon -> hmp in patch for consistency
> > - renamed error_vprintf_mon/error_printf_mon -> error_vprintf_hmp/error_printf_hmp
> > - simplified default_monitor init with IS_ENABLED(CONFIG_HMP)
> > - added early NULL return in qemu_vprintf/qemu_vfprintf
> > - reworked !opts->has_mode path to use explicit error_setg() when HMP is disabled
> > - split device-introspect-test patch
> > - rebased, collected tags, all reviewed, ready-to-merge
> 
> Thanks for the final polishing. Everything LGTM!

Yeh better merge it before anything moves!
(Assume Marc-André is sending the pull).

Dave

-- 
 -----Open up your eyes, open up your mind, open up your code -------   
/ Dr. David Alan Gilbert    |       Running GNU/Linux       | Happy  \ 
\        dave @ treblig.org |                               | In Hex /
 \ _________________________|_____ http://www.treblig.org   |_______/