Re: [PATCH 0/5] Stop adding HMP-only commands, allow QMP for all

[Paolo Bonzini]

On 08/09/21 17:09, Markus Armbruster wrote:
> This is 77 out of 170 HMP commands.  I was hoping for fewer.

There are several that are not quite 1:1, but still not HMP-specific.

>      exit_preconfig

This is x-exit-preconfig.

>      migrate_set_capability
>      migrate_set_parameter

These are migrate-set-{capabilities,parameters}

>      mouse_button
>      mouse_move
>      mouse_set
>      sendkey

These are input-send-event, but they are not implemented in terms of it.

>      nmi

This is inject-nmi.

>      loadvm
>      savevm

These are snapshot-{load,save}.

>      watchdog_action

This is set-action, but it is not implemented in terms of it.

Paolo

> >                                          and then perhaps have
> > a task to track their conversion to QMP. This could possibly
> > be a useful task for newbies if we make it clear that they
> > wouldn't be required to undertake complex QAPI modelling in
> > doing this conversion.
>
> Yes.
>
> Thanks for tackling this!