Re: [Qemu-devel] [PATCH v4 00/19] block: bdrv_reopen() patches

[Kevin Wolf <kwolf@redhat.com>]

Am 20.09.2012 21:13, schrieb Jeff Cody:
> These patches are based off Supriya Kannery's original bdrv_reopen()
> patches as part of the hostcache series.
> 
> This provides support for safe reopen of a single image, or transactional
> reopening of multiple images atomically.
> 
> These changes are all reflected in my github repo:
> 
>     git://github.com/codyprime/qemu-kvm-jtc.git  branch: jtc-live-commit-1.3-v7
> 

> Jeff Cody (19):
>   block: correctly set the keep_read_only flag
>   block: make bdrv_set_enable_write_cache() modify open_flags
>   block: Framework for reopening files safely
>   block: move aio initialization into a helper function
>   block: move open flag parsing in raw block drivers to helper
>     functions
>   block: do not parse BDRV_O_CACHE_WB in block drivers
>   block: use BDRV_O_NOCACHE instead of s->aligned_buf in raw-posix.c
>   block: purge s->aligned_buf and s->aligned_buf_size from raw-posix.c
>   block: raw-posix image file reopen
>   block: raw image file reopen
>   block: qed image file reopen
>   block: qcow2 image file reopen
>   block: qcow image file reopen
>   block: vmdk image file reopen
>   block: raw-win32 driver reopen support
>   block: vdi image file reopen
>   block: vpc image file reopen
>   block: convert bdrv_commit() to use bdrv_reopen()
>   block: remove keep_read_only flag from BlockDriverState struct
> 
>  block.c           | 299 +++++++++++++++++++++++++++++++++++++++++++++---------
>  block.h           |  18 ++++
>  block/iscsi.c     |   4 -
>  block/qcow.c      |  10 ++
>  block/qcow2.c     |  10 ++
>  block/qed.c       |   9 ++
>  block/raw-posix.c | 225 ++++++++++++++++++++++++++++++----------
>  block/raw-win32.c | 145 ++++++++++++++++++++++----
>  block/raw.c       |  10 ++
>  block/rbd.c       |   6 --
>  block/sheepdog.c  |  14 ++-
>  block/vdi.c       |   7 ++
>  block/vmdk.c      |  35 +++++++
>  block/vpc.c       |   7 ++
>  block_int.h       |   9 +-
>  15 files changed, 666 insertions(+), 142 deletions(-)

Thanks, applied all to the block branch, except for patch 15
(raw-win32), which I think can safely be applied on top when we've come
to a conclusion.

Kevin