Re: [Qemu-devel] [PATCH V3 0/4] HMP: allow parsing for sub command

[Wenchao Xia <xiawenc@linux.vnet.ibm.com>]

Hi, Markus
  Any comments for this? There are other serials of patch need this
infrastructure extension, so hope it can be reviewed faster....

>    These patches enhance HMP to allow it parse 2nd level of commands, such
> as info sub command list, which means now foldered command with parameter
> is possible.
> 
> V2:
>    Follow the way supposed by Markus, which make the infrastructure knows
> there is possible a 2nd level of command exist, instead of a hack. In this
> way extention of command folder level is easy.
>    Moved some code and better doc according to comments.
>    Removed the patch about info snapshots, which will goto another serial.
> V3:
>    Move changelog position.
>    Split out code moving patch.
> 
> Wenchao Xia (4):
>    HMP: add QDict to info callback handler
>    HMP: add infrastructure for sub command
>    HMP: move define of mon_cmds
>    HMP: add sub command table to info
> 
>   hmp-commands.hx         |    3 +-
>   hmp.c                   |   36 ++++++++--------
>   hmp.h                   |   36 ++++++++--------
>   hw/i8259.c              |    4 +-
>   hw/lm32_pic.c           |    4 +-
>   hw/lm32_pic.h           |    4 +-
>   hw/loader.c             |    2 +-
>   hw/loader.h             |    3 +-
>   hw/pc.h                 |    4 +-
>   hw/pcmcia.h             |    2 +-
>   hw/qdev-monitor.c       |    4 +-
>   hw/qdev-monitor.h       |    4 +-
>   hw/sun4m.c              |    4 +-
>   hw/sun4m.h              |    4 +-
>   hw/usb.h                |    2 +-
>   hw/usb/bus.c            |    2 +-
>   hw/usb/host-bsd.c       |    2 +-
>   hw/usb/host-linux.c     |    2 +-
>   include/net/net.h       |    2 +-
>   include/net/slirp.h     |    2 +-
>   include/sysemu/sysemu.h |    4 +-
>   monitor.c               |  108 +++++++++++++++++++++-------------------------
>   net/net.c               |    2 +-
>   net/slirp.c             |    2 +-
>   savevm.c                |    2 +-
>   vl.c                    |    2 +-
>   26 files changed, 119 insertions(+), 127 deletions(-)
> 
> 


-- 
Best Regards

Wenchao Xia