From: Wenchao Xia <xiawenc@linux.vnet.ibm.com>
To: qemu-devel@nongnu.org
Cc: aliguori@us.ibm.com, phrdina@redhat.com, stefanha@gmail.com,
armbru@redhat.com, lcapitulino@redhat.com, pbonzini@redhat.com,
Wenchao Xia <xiawenc@linux.vnet.ibm.com>
Subject: [Qemu-devel] [PATCH V2 00/11] add qmp/hmp interfaces for snapshot info
Date: Wed, 9 Jan 2013 13:04:32 +0800 [thread overview]
Message-ID: <1357707883-9182-1-git-send-email-xiawenc@linux.vnet.ibm.com> (raw)
This serial of patches does two things: merge some info code
in qemu-img, and add following interfaces:
1) qmp: query-images
2) qmp: query-snapshots
3) hmp: show snapshot info on a single block device
These patches follows the rule that use qmp to retieve information,
hmp layer just do a translation from qmp object it got, so almost
every hmp interface may have a correlated qmp interface.
To make code graceful, snapshot retrieving code in qemu and qemu-img
are merged into block.c, and some function name was adjusted to make it
tips better. Now it works as:
qemu qemu-img
dump_monitor dump_stdout
|--------------|
|
qmp
|
block
Note:
Last two patches need previous sent patches which extend hmp sub command, at:
http://lists.nongnu.org/archive/html/qemu-devel/2012-12/msg03487.html
v2:
Rename and adjusted qmp interface according to comments from Eric.
Spelling fix.
Information retrieving function in block layer goes to seperated patch.
Free qmp object after usage in hmp.
Wenchao Xia (11):
qemu-img: remove unused parameter in collect_image_info()
block: add bdrv_get_filename() function
block: add snapshot and image info query function
qemu-img: switch image retrieving function
block: rename bdrv_query_info to bdrv_query_block_info
qmp: add interface query-images.
block: export function bdrv_find_snapshot()
qmp: add interface query-snapshots
hmp: export function hmp_handle_error()
hmp: retrieve info from qmp for snapshot info
hmp: show snapshot on single block device
block.c | 200 ++++++++++++++++++++++++++++++++++++++++++++++++-
hmp.c | 2 +-
hmp.h | 2 +
include/block/block.h | 14 +++-
monitor.c | 6 +-
qapi-schema.json | 40 ++++++++++
qemu-img.c | 87 +---------------------
qmp-commands.hx | 129 +++++++++++++++++++++++++++++++
savevm.c | 137 +++++++++++++++++----------------
9 files changed, 458 insertions(+), 159 deletions(-)
next reply other threads:[~2013-01-09 5:07 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-01-09 5:04 Wenchao Xia [this message]
-- strict thread matches above, loose matches on Subject: below --
2013-01-09 5:12 [Qemu-devel] [PATCH V2 00/11] add qmp/hmp interfaces for snapshot info Wenchao Xia
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=1357707883-9182-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=lcapitulino@redhat.com \
--cc=pbonzini@redhat.com \
--cc=phrdina@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@gmail.com \
/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).