[Qemu-devel] [PATCH v3 0/2] block: Fix permission during reopen

Fam Zheng posted 2 patches 6 years, 1 month ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20180313142003.12124-1-famz@redhat.com
Test checkpatch passed
Test docker-build@min-glib passed
Test docker-mingw@fedora passed
Test docker-quick@centos6 passed
Test s390x passed
block.c                    |  7 +++++++
tests/qemu-iotests/153     | 12 ++++++++++++
tests/qemu-iotests/153.out |  5 +++++
3 files changed, 24 insertions(+)
[Qemu-devel] [PATCH v3 0/2] block: Fix permission during reopen
Posted by Fam Zheng 6 years, 1 month ago
v3: Fix test case. [Max]

v2: Use update_flags_from_options. [Kevin]

We write open the whole backing chain during reopen. It is not necessary and
will cause image locking problems if the backing image is shared.

Fam Zheng (2):
  block: Fix flags in reopen queue
  iotests: Add regression test for commit base locking

 block.c                    |  7 +++++++
 tests/qemu-iotests/153     | 12 ++++++++++++
 tests/qemu-iotests/153.out |  5 +++++
 3 files changed, 24 insertions(+)

-- 
2.14.3


Re: [Qemu-devel] [PATCH v3 0/2] block: Fix permission during reopen
Posted by Max Reitz 6 years, 1 month ago
On 2018-03-13 15:20, Fam Zheng wrote:
> v3: Fix test case. [Max]
> 
> v2: Use update_flags_from_options. [Kevin]
> 
> We write open the whole backing chain during reopen. It is not necessary and
> will cause image locking problems if the backing image is shared.
> 
> Fam Zheng (2):
>   block: Fix flags in reopen queue
>   iotests: Add regression test for commit base locking

Reviewed-by: Max Reitz <mreitz@redhat.com>

Re: [Qemu-devel] [PATCH v3 0/2] block: Fix permission during reopen
Posted by Kevin Wolf 6 years, 1 month ago
Am 13.03.2018 um 15:20 hat Fam Zheng geschrieben:
> v3: Fix test case. [Max]
> 
> v2: Use update_flags_from_options. [Kevin]
> 
> We write open the whole backing chain during reopen. It is not necessary and
> will cause image locking problems if the backing image is shared.

Thanks, applied to the block branch.

Kevin