From: Pavel Hrdina <phrdina@redhat.com>
To: qemu-devel@nongnu.org
Cc: lcapitulino@redhat.com
Subject: Re: [Qemu-devel] [PATCH 00/13] convert savevm to use qapi and introduce qmp command
Date: Wed, 09 Jan 2013 16:34:16 +0100 [thread overview]
Message-ID: <1357745656.13501.0.camel@localhost> (raw)
In-Reply-To: <cover.1357741229.git.phrdina@redhat.com>
Please remove from CC the bad address with c0m.
On Wed, 2013-01-09 at 16:17 +0100, Pavel Hrdina wrote:
> This patch series convert the savevm command into qapi and introduce QMP
> command vm-snapshot-save.
> It also rewrite error report for functions used by this command.
>
> Last patch introduce new functionality of savevm and vm-snapshot-save
> that you cannot override existing snapshot without using 'force' parameter
> and for QMP you have to always provide name parameter.
>
> Pavel Hrdina (13):
> block: add error parameter to bdrv_snapshot_create() and related
> functions
> block: add error parameter to del_existing_snapshots()
> savevm: add error parameter to qemu_savevm_state_begin()
> savevm: add error parameter to qemu_savevm_state_iterate()
> savevm: add error parameter to qemu_savevm_state_complete()
> savevm: add error parameter to qemu_savevm_state()
> qapi: Convert savevm
> qemu-img: introduce qemu_img_handle_error
> block: update return value from bdrv_snapshot_create
> savevm: update return value from qemu_savevm_state_begin
> savevm: update return value from qemu_savevm_state_complete
> savevm: update return value from qemu_savevm_state
> vm-snapshot-save: add force parameter
>
> block.c | 26 ++++++----
> block/qcow2-snapshot.c | 14 ++++--
> block/qcow2.h | 4 +-
> block/rbd.c | 15 ++++--
> block/sheepdog.c | 21 ++++----
> hmp-commands.hx | 17 +++----
> hmp.c | 32 +++++++++++++
> hmp.h | 1 +
> include/block/block.h | 3 +-
> include/block/block_int.h | 3 +-
> include/sysemu/sysemu.h | 8 ++--
> migration.c | 6 +--
> qapi-schema.json | 21 ++++++++
> qemu-img.c | 18 +++++--
> qmp-commands.hx | 32 +++++++++++++
> savevm.c | 120 +++++++++++++++++++++++++---------------------
> 16 files changed, 237 insertions(+), 104 deletions(-)
>
prev parent reply other threads:[~2013-01-09 15:34 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-01-09 15:17 [Qemu-devel] [PATCH 00/13] convert savevm to use qapi and introduce qmp command Pavel Hrdina
2013-01-09 15:17 ` [Qemu-devel] [PATCH 01/13] block: add error parameter to bdrv_snapshot_create() and related functions Pavel Hrdina
2013-01-09 15:17 ` [Qemu-devel] [PATCH 02/13] block: add error parameter to del_existing_snapshots() Pavel Hrdina
2013-01-09 15:17 ` [Qemu-devel] [PATCH 03/13] savevm: add error parameter to qemu_savevm_state_begin() Pavel Hrdina
2013-01-09 15:17 ` [Qemu-devel] [PATCH 04/13] savevm: add error parameter to qemu_savevm_state_iterate() Pavel Hrdina
2013-01-09 15:17 ` [Qemu-devel] [PATCH 05/13] savevm: add error parameter to qemu_savevm_state_complete() Pavel Hrdina
2013-01-09 15:18 ` [Qemu-devel] [PATCH 06/13] savevm: add error parameter to qemu_savevm_state() Pavel Hrdina
2013-01-09 15:18 ` [Qemu-devel] [PATCH 07/13] qapi: Convert savevm Pavel Hrdina
2013-01-09 22:32 ` Eric Blake
2013-01-10 5:50 ` Wenchao Xia
2013-01-09 15:18 ` [Qemu-devel] [PATCH 08/13] qemu-img: introduce qemu_img_handle_error Pavel Hrdina
2013-01-09 15:18 ` [Qemu-devel] [PATCH 09/13] block: update return value from bdrv_snapshot_create Pavel Hrdina
2013-01-09 15:18 ` [Qemu-devel] [PATCH 10/13] savevm: update return value from qemu_savevm_state_begin Pavel Hrdina
2013-01-09 15:18 ` [Qemu-devel] [PATCH 11/13] savevm: update return value from qemu_savevm_state_complete Pavel Hrdina
2013-01-09 15:18 ` [Qemu-devel] [PATCH 12/13] savevm: update return value from qemu_savevm_state Pavel Hrdina
2013-01-09 22:39 ` Eric Blake
2013-01-10 10:35 ` Pavel Hrdina
2013-01-09 15:18 ` [Qemu-devel] [PATCH 13/13] vm-snapshot-save: add force parameter Pavel Hrdina
2013-01-09 22:47 ` Eric Blake
2013-01-10 11:54 ` Pavel Hrdina
2013-01-09 15:34 ` Pavel Hrdina [this message]
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=1357745656.13501.0.camel@localhost \
--to=phrdina@redhat.com \
--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).