[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v2 for-2.10 0/8] RBD reopen, read_only cleanup
From: |
Jeff Cody |
Subject: |
Re: [Qemu-block] [PATCH v2 for-2.10 0/8] RBD reopen, read_only cleanup |
Date: |
Thu, 20 Apr 2017 11:05:20 -0400 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Fri, Apr 07, 2017 at 04:55:24PM -0400, Jeff Cody wrote:
>
> Changes from v1:
>
> Patch 2: Has v1 patch 8 (do not blindly xset bs->read_only) squashed into it
> (thanks Stefan)
> COW -> "copy-on-read" (Thanks John)
> Drop unneeded call in vvfat, and bypass enable_write_target (Stefan)
>
> Patch 5: Rename bdrv_try_... to bdrv_can_set_read_only() (Thanks John, Stefan)
>
> Patch 6: Use "reopen_state->flags" not "reopen_state->bs->open_flags"
> (Thanks John)
>
>
>
> This series does two things:
>
> 1.) Cleans up some of the logic behind setting the read_only flag
> for a BDS in the block layer, so that it is done consistently
> (and rules are applied consistently), and
>
> 2.) Adds .bdrv_reopen_prepare() implementation for RBD, so that block
> jobs can be run on backing chains that have rbd protocol nodes.
>
> Jeff Cody (8):
> block: add bdrv_set_read_only() helper function
> block: do not set BDS read_only if copy_on_read enabled
> block: honor BDRV_O_ALLOW_RDWR when clearing bs->read_only
> block: code movement
> block: introduce bdrv_can_set_read_only()
> block: use bdrv_can_set_read_only() during reopen
> block/rbd - update variable names to more apt names
> block/rbd: Add support for reopen()
>
> block.c | 56 +++++++++++++++++++++++++++++++++++---------
> block/bochs.c | 5 +++-
> block/cloop.c | 5 +++-
> block/dmg.c | 6 ++++-
> block/rbd.c | 65
> +++++++++++++++++++++++++++++++++++++--------------
> block/vvfat.c | 19 +++++++++++----
> include/block/block.h | 2 ++
> 7 files changed, 123 insertions(+), 35 deletions(-)
>
> --
> 2.9.3
>
Thanks,
Applied to my block branch:
git://github.com/codyprime/qemu-kvm-jtc block
-Jeff
- [Qemu-block] [PATCH v2 for-2.10 1/8] block: add bdrv_set_read_only() helper function, (continued)
- [Qemu-block] [PATCH v2 for-2.10 1/8] block: add bdrv_set_read_only() helper function, Jeff Cody, 2017/04/07
- [Qemu-block] [PATCH v2 for-2.10 3/8] block: honor BDRV_O_ALLOW_RDWR when clearing bs->read_only, Jeff Cody, 2017/04/07
- [Qemu-block] [PATCH v2 for-2.10 2/8] block: do not set BDS read_only if copy_on_read enabled, Jeff Cody, 2017/04/07
- [Qemu-block] [PATCH v2 for-2.10 4/8] block: code movement, Jeff Cody, 2017/04/07
- [Qemu-block] [PATCH v2 for-2.10 5/8] block: introduce bdrv_can_set_read_only(), Jeff Cody, 2017/04/07
- [Qemu-block] [PATCH v2 for-2.10 6/8] block: use bdrv_can_set_read_only() during reopen, Jeff Cody, 2017/04/07
- [Qemu-block] [PATCH v2 for-2.10 8/8] block/rbd: Add support for reopen(), Jeff Cody, 2017/04/07
- [Qemu-block] [PATCH v2 for-2.10 7/8] block/rbd - update variable names to more apt names, Jeff Cody, 2017/04/07
- Re: [Qemu-block] [PATCH v2 for-2.10 0/8] RBD reopen, read_only cleanup, Stefan Hajnoczi, 2017/04/10
- Re: [Qemu-block] [Qemu-devel] [PATCH v2 for-2.10 0/8] RBD reopen, read_only cleanup, John Snow, 2017/04/10
- Re: [Qemu-block] [PATCH v2 for-2.10 0/8] RBD reopen, read_only cleanup,
Jeff Cody <=