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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxx>

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux