All of lore.kernel.org
 help / color / mirror / Atom feed
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(-)
> 

      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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.