qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Wenchao Xia <xiawenc@linux.vnet.ibm.com>
To: qemu-devel@nongnu.org
Cc: Wenchao Xia <xiawenc@linux.vnet.ibm.com>,
	aliguori@us.ibm.com, armbru@redhat.com, chenwj@iis.sinica.edu.tw,
	lcapitulino@redhat.com
Subject: [Qemu-devel] [PATCH V6 0/5] HMP: allow parsing for sub command
Date: Mon, 14 Jan 2013 14:06:24 +0800	[thread overview]
Message-ID: <1358143589-5913-1-git-send-email-xiawenc@linux.vnet.ibm.com> (raw)

  These patches enhance HMP to allow it parse 2nd level of commands, such
as info sub command list, which means foldered command with parameter is
possible now.

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 function declaration and better doc according to comments.
  Removed the patch about info snapshots, which will goto another serial.
V3:
  Split out code moving patch.
V4:
  Removed change of qmp_find_cmd().
  Removed name change of monitor_parse_command().
v5:
  Eliminate 'info' in mhandler for that it have same format with 'cmd' and info
is not a special case but a sub-command now.
  Split out patch that checking for space before check for sub-command.
  Better comments for monitor_parse_command().
  Add parameter start for better error tips in sub-command case.
  Add comments about how sub_table and mhandler interact.
  Better commit message and tips that "info <unknowns>" show error now.
v6:
  Filter out space before checking of sub command in montior_parse_command(),
and discard *p1 in it.
  Merged patch "filter out space" into "infrastructure" patch.

Wenchao Xia (5):
  HMP: add QDict to info callback handler
  HMP: delete info 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               |  200 ++++++++++++++++++++++++----------------------
 net/net.c               |    2 +-
 net/slirp.c             |    2 +-
 savevm.c                |    2 +-
 vl.c                    |    2 +-
 26 files changed, 174 insertions(+), 164 deletions(-)

             reply	other threads:[~2013-01-14  6:08 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-14  6:06 Wenchao Xia [this message]
2013-01-14  6:06 ` [Qemu-devel] [PATCH V6 1/5] HMP: add QDict to info callback handler Wenchao Xia
2013-01-14  6:06 ` [Qemu-devel] [PATCH V6 2/5] HMP: delete info handler Wenchao Xia
2013-01-14  6:06 ` [Qemu-devel] [PATCH V6 3/5] HMP: add infrastructure for sub command Wenchao Xia
2013-01-14  6:06 ` [Qemu-devel] [PATCH V6 4/5] HMP: move define of mon_cmds Wenchao Xia
2013-01-14  6:06 ` [Qemu-devel] [PATCH V6 5/5] HMP: add sub command table to info Wenchao Xia
2013-01-14 16:44 ` [Qemu-devel] [PATCH V6 0/5] HMP: allow parsing for sub command Luiz Capitulino

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1358143589-5913-1-git-send-email-xiawenc@linux.vnet.ibm.com \
    --to=xiawenc@linux.vnet.ibm.com \
    --cc=aliguori@us.ibm.com \
    --cc=armbru@redhat.com \
    --cc=chenwj@iis.sinica.edu.tw \
    --cc=lcapitulino@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).