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, jcody@redhat.com, hbrock@redhat.com,
	rjones@redhat.com, armbru@redhat.com, imain@redhat.com,
	stefanha@redhat.com, pbonzini@redhat.com
Subject: [Qemu-devel] [PATCH v20 00/15] Drop in_use from BlockDriverState and enable point-in-time snapshot exporting over NBD
Date: Tue, 20 May 2014 14:04:25 +0800	[thread overview]
Message-ID: <1400565880-13409-1-git-send-email-famz@redhat.com> (raw)

Thanks for Eric's and Markus' reviewing!

v20: Rebase to master, with minor improvements as listed below:

    [02/15] block: Introduce op_blockers to BlockDriverState
            Better error message in bdrv_op_is_blocked. (Markus)

    [03/15] block: Replace in_use with operation blocker
            Accurate commit message: why use bdrv_op_blocker_is_empty in
            block_job_create. (Markus)

    [06/15] block: Add backing_blocker in BlockDriverState
            Drop error_is_set().
            Call error_free() unconditionally.
            

    [08/15] block: Support dropping active in bdrv_drop_intermediate
            Fix list index in function comment. (Eric)

    [13/15] block: Add blockdev-backup to transaction
            Drop error_is_set().

    [14/15] qemu-iotests: Test blockdev-backup in 055
            Bump year in file header. (Eric)
 

v19: Rebase to master, resolve several contextual conflicts.
     Patch 05 has the only code change difference from v18 to preserve.
        assert(bs->backing_hd == NULL);

v18: Address reviewing comments from Jeff and Eric. Rebased to current master.
     Side by side diff from v17: http://bit.ly/1oO2Fvt

    [01/15] block: Add BlockOpType enum
            Add Jeff's reviewed-by.
            
    [02/15] block: Introduce op_blockers to BlockDriverState
            Add Jeff's reviewed-by.
            
    [03/15] block: Replace in_use with operation blocker
            Add Jeff's reviewed-by.
            
    [04/15] block: Move op_blocker check from block_job_create to its caller
            Add Jeff's reviewed-by.
            
    [05/15] block: Add bdrv_set_backing_hd()
            Don't unset bs->backing_file and bs->backing_format when
            backing_hd==NULL, because qcow2_close() will save these into image
            header.
            
    [08/15] block: Support dropping active in bdrv_drop_intermediate
            Swap parameters for bdrv_swap:
                bdrv_swap(active, base); -> bdrv_swap(base, active);
            Use bdrv_set_backing_hd().

    [10/15] commit: Use bdrv_drop_intermediate
            New. (Jeff)

    [11/15] qmp: Add command 'blockdev-backup'
            Since 2.0 -> Since 2.1. (Eric)

    [13/15] block: Add blockdev-backup to transaction
            Comment "Since 2.1" for blockdev-backup. (Eric)

    [15/15] qemu-iotests: Image fleecing test case 089
            Case number 083 -> 089.


Fam Zheng (15):
  block: Add BlockOpType enum
  block: Introduce op_blockers to BlockDriverState
  block: Replace in_use with operation blocker
  block: Move op_blocker check from block_job_create to its caller
  block: Add bdrv_set_backing_hd()
  block: Add backing_blocker in BlockDriverState
  block: Parse "backing" option to reference existing BDS
  block: Support dropping active in bdrv_drop_intermediate
  stream: Use bdrv_drop_intermediate and drop close_unused_images
  commit: Use bdrv_drop_intermediate
  qmp: Add command 'blockdev-backup'
  block: Allow backup on referenced named BlockDriverState
  block: Add blockdev-backup to transaction
  qemu-iotests: Test blockdev-backup in 055
  qemu-iotests: Image fleecing test case 089

 block-migration.c               |   7 +-
 block.c                         | 310 +++++++++++++++++++++++++++-------------
 block/backup.c                  |  26 ++++
 block/commit.c                  |   2 +-
 block/mirror.c                  |   9 +-
 block/stream.c                  |  42 +-----
 blockdev.c                      | 122 ++++++++++++++--
 blockjob.c                      |  14 +-
 hw/block/dataplane/virtio-blk.c |  18 ++-
 include/block/block.h           |  29 +++-
 include/block/block_int.h       |   9 +-
 include/block/blockjob.h        |   3 +
 qapi-schema.json                |  52 +++++++
 qmp-commands.hx                 |  44 ++++++
 tests/qemu-iotests/055          | 277 +++++++++++++++++++++++++++++------
 tests/qemu-iotests/055.out      |   4 +-
 tests/qemu-iotests/089          |  99 +++++++++++++
 tests/qemu-iotests/089.out      |   5 +
 tests/qemu-iotests/group        |   1 +
 19 files changed, 856 insertions(+), 217 deletions(-)
 create mode 100755 tests/qemu-iotests/089
 create mode 100644 tests/qemu-iotests/089.out

-- 
1.9.2

             reply	other threads:[~2014-05-20  6:04 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-20  6:04 Fam Zheng [this message]
2014-05-20  6:04 ` [Qemu-devel] [PATCH v20 01/15] block: Add BlockOpType enum Fam Zheng
2014-05-21 12:28   ` Stefan Hajnoczi
2014-05-20  6:04 ` [Qemu-devel] [PATCH v20 02/15] block: Introduce op_blockers to BlockDriverState Fam Zheng
2014-05-21 12:28   ` Stefan Hajnoczi
2014-05-20  6:04 ` [Qemu-devel] [PATCH v20 03/15] block: Replace in_use with operation blocker Fam Zheng
2014-05-21 12:46   ` Stefan Hajnoczi
2014-05-20  6:04 ` [Qemu-devel] [PATCH v20 04/15] block: Move op_blocker check from block_job_create to its caller Fam Zheng
2014-05-20 11:43   ` Jeff Cody
2014-05-21  1:36     ` Fam Zheng
2014-05-21  4:34       ` Jeff Cody
2014-05-21  6:08         ` Fam Zheng
2014-05-21 13:13           ` Stefan Hajnoczi
2014-05-21 13:17             ` Jeff Cody
2014-05-21 14:04           ` Jeff Cody
2014-05-21 13:20   ` Stefan Hajnoczi
2014-05-20  6:04 ` [Qemu-devel] [PATCH v20 05/15] block: Add bdrv_set_backing_hd() Fam Zheng
2014-05-21 13:56   ` Stefan Hajnoczi
2014-05-21 14:17   ` Jeff Cody
2014-05-20  6:04 ` [Qemu-devel] [PATCH v20 06/15] block: Add backing_blocker in BlockDriverState Fam Zheng
2014-05-21 14:03   ` Stefan Hajnoczi
2014-05-21 14:24     ` Jeff Cody
2014-05-21 14:37       ` Fam Zheng
2014-05-22 16:57         ` Jeff Cody
2014-05-21 14:06   ` Stefan Hajnoczi
2014-05-21 14:49     ` Markus Armbruster
2014-05-20  6:04 ` [Qemu-devel] [PATCH v20 07/15] block: Parse "backing" option to reference existing BDS Fam Zheng
2014-05-20  6:04 ` [Qemu-devel] [PATCH v20 08/15] block: Support dropping active in bdrv_drop_intermediate Fam Zheng
2014-05-20  6:04 ` [Qemu-devel] [PATCH v20 09/15] stream: Use bdrv_drop_intermediate and drop close_unused_images Fam Zheng
2014-05-20  6:04 ` [Qemu-devel] [PATCH v20 10/15] commit: Use bdrv_drop_intermediate Fam Zheng
2014-05-20  6:04 ` [Qemu-devel] [PATCH v20 11/15] qmp: Add command 'blockdev-backup' Fam Zheng
2014-05-20  6:04 ` [Qemu-devel] [PATCH v20 12/15] block: Allow backup on referenced named BlockDriverState Fam Zheng
2014-05-20  6:04 ` [Qemu-devel] [PATCH v20 13/15] block: Add blockdev-backup to transaction Fam Zheng
2014-05-20  6:04 ` [Qemu-devel] [PATCH v20 14/15] qemu-iotests: Test blockdev-backup in 055 Fam Zheng
2014-05-20  6:04 ` [Qemu-devel] [PATCH v20 15/15] qemu-iotests: Image fleecing test case 089 Fam Zheng
2018-06-29 18:00 ` [Qemu-devel] [PATCH v20 00/15] Drop in_use from BlockDriverState and enable point-in-time snapshot exporting over NBD Eric Blake
2018-06-29 18:34   ` John Snow

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=1400565880-13409-1-git-send-email-famz@redhat.com \
    --to=famz@redhat.com \
    --cc=armbru@redhat.com \
    --cc=hbrock@redhat.com \
    --cc=imain@redhat.com \
    --cc=jcody@redhat.com \
    --cc=kwolf@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=rjones@redhat.com \
    --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).