qemu-devel
[Top][All Lists]
Advanced

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

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


From: Wenchao Xia
Subject: Re: [Qemu-devel] [PATCH V3 0/4] HMP: allow parsing for sub command
Date: Fri, 04 Jan 2013 11:00:51 +0800
User-agent: Mozilla/5.0 (Windows NT 5.1; rv:16.0) Gecko/20121026 Thunderbird/16.0.2

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




reply via email to

[Prev in Thread] Current Thread [Next in Thread]