[libvirt] [PATCH 0/8] qemu: monitor: Clean up some more HMP cruft

Jiri Denemark jdenemar at redhat.com
Thu Sep 19 16:34:15 UTC 2019


On Thu, Sep 19, 2019 at 18:10:15 +0200, Peter Krempa wrote:
> There was quite a bit of legacy HMP cruft. We can remove it as we use
> HMP commands via QMP.
> 
> Peter Krempa (8):
>   qemu: monitor: Remove legacy monitor commands for FD manipulation
>   qemu: monitor: Remove qemuMonitorHMPCommand macro
>   qemu: monitor: Remove support for HMP commands with fds
>   qemu: monitor: Don't escape HMP commands just to unescape them right
>     away
>   qemu: monitor: Remove HMP command (un)escaping infrastructure
>   qemu: monitor: Don't include text monitor in json monitor
>   qemu: monitor: Don't handle HMP in qemuMonitorJSONArbitraryCommand
>   qemu: monitor: Remove qemuMonitorHMPCommand in favor of
>     qemuMonitorJSONHumanCommand
> 
>  src/qemu/qemu_monitor.c      | 178 +----------------------------------
>  src/qemu/qemu_monitor.h      |  13 +--
>  src/qemu/qemu_monitor_json.c | 104 +++-----------------
>  src/qemu/qemu_monitor_json.h |  13 +--
>  src/qemu/qemu_monitor_text.c |  39 +++-----
>  tests/qemumonitortestutils.c |   6 +-
>  6 files changed, 32 insertions(+), 321 deletions(-)

Nice.

Reviewed-by: Jiri Denemark <jdenemar at redhat.com>




More information about the libvir-list mailing list