qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Fam Zheng <famz@redhat.com>
To: qemu-devel@nongnu.org
Cc: kwolf@redhat.com, famz@redhat.com, jcody@redhat.com,
	stefanha@redhat.com, pbonzini@redhat.com
Subject: [Qemu-devel] [PATCH v3 0/2] block: allow commit active as top
Date: Thu, 15 Aug 2013 16:14:05 +0800	[thread overview]
Message-ID: <1376554447-28638-1-git-send-email-famz@redhat.com> (raw)

Previously live commit of active block device is not supported, this series
implements it and updates corresponding qemu-iotests cases.

Please see commit messages for implementation details.

v3: [addressing Stefan's comments]
    - Sleep in dirty map setup loop.
    - Handle error of bdrv_co_is_allocated_above().
    - Fix order of dirty map reset and commit_populate().
    - Use bdrv_co_flush().
    - Put s->overlay check in one if block.
    - Don't interleave orig_overlay_flags and orig_base_flags saving
      stanzas.

v2: report ready when all blocks commited for the first time, leave it to the
    user to complete the job (upon user's complete command, also commit writes
    since ready and flushes before reporting completion) 


Fam Zheng (2):
  block: allow live commit of active image
  qemu-iotests: update test cases for commit active

 block.c                | 102 +++++++----------------
 block/commit.c         | 215 ++++++++++++++++++++++++++++++-------------------
 include/block/block.h  |   5 +-
 tests/qemu-iotests/040 |  73 +++++++----------
 4 files changed, 195 insertions(+), 200 deletions(-)

-- 
1.8.3.1

             reply	other threads:[~2013-08-15  8:14 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-15  8:14 Fam Zheng [this message]
2013-08-15  8:14 ` [Qemu-devel] [PATCH v3 1/2] block: allow live commit of active image Fam Zheng
2013-09-04 12:35   ` Stefan Hajnoczi
2013-09-18  3:32     ` Fam Zheng
2013-09-18  9:36       ` Stefan Hajnoczi
2013-09-18 11:46     ` Paolo Bonzini
2013-08-15  8:14 ` [Qemu-devel] [PATCH v3 2/2] qemu-iotests: update test cases for commit active Fam Zheng

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=1376554447-28638-1-git-send-email-famz@redhat.com \
    --to=famz@redhat.com \
    --cc=jcody@redhat.com \
    --cc=kwolf@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@redhat.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).