Fix for write sharing on luks raw images

[PATCH 0/2] Fix for write sharing on luks raw images

Maxim Levitsky posted 2 patches 5 years, 2 months ago

Download series mbox

Test checkpatch failed

Test docker-mingw@fedora failed

Test FreeBSD passed

Test docker-quick@centos7 failed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200719122059.59843-1-mlevitsk@redhat.com

Maintainers: Max Reitz <mreitz@redhat.com>, Kevin Wolf <kwolf@redhat.com>

block/crypto.c             |  2 +-
tests/qemu-iotests/296     | 44 +++++++++++++++++++++++++++++++++++++-
tests/qemu-iotests/296.out | 12 +++++++++--
3 files changed, 54 insertions(+), 4 deletions(-)

Expand all Fold all

[PATCH 0/2] Fix for write sharing on luks raw images

Posted by Maxim Levitsky 5 years, 2 months ago

A rebase gone wrong, and I ended up allowing a luks image
to be opened at the same time by two VMs without any warnings/overrides.

Fix that and also add an iotest to prevent this from happening.

Best regards,
	Maxim Levisky

Maxim Levitsky (2):
  block/crypto: disallow write sharing by default
  qemu-iotests: add testcase for bz #1857490

 block/crypto.c             |  2 +-
 tests/qemu-iotests/296     | 44 +++++++++++++++++++++++++++++++++++++-
 tests/qemu-iotests/296.out | 12 +++++++++--
 3 files changed, 54 insertions(+), 4 deletions(-)

-- 
2.26.2

Re: [PATCH 0/2] Fix for write sharing on luks raw images

Posted by no-reply@patchew.org 5 years, 2 months ago

Patchew URL: https://patchew.org/QEMU/20200719122059.59843-1-mlevitsk@redhat.com/



Hi,

This series failed the docker-quick@centos7 build test. Please find the testing commands and
their output below. If you have Docker installed, you can probably reproduce it
locally.

=== TEST SCRIPT BEGIN ===
#!/bin/bash
make docker-image-centos7 V=1 NETWORK=1
time make docker-test-quick@centos7 SHOW_ENV=1 J=14 NETWORK=1
=== TEST SCRIPT END ===




The full log is available at
http://patchew.org/logs/20200719122059.59843-1-mlevitsk@redhat.com/testing.docker-quick@centos7/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com

Re: [PATCH 0/2] Fix for write sharing on luks raw images

Posted by Max Reitz 5 years, 2 months ago

On 19.07.20 14:20, Maxim Levitsky wrote:
> A rebase gone wrong, and I ended up allowing a luks image
> to be opened at the same time by two VMs without any warnings/overrides.
> 
> Fix that and also add an iotest to prevent this from happening.
> 
> Best regards,
> 	Maxim Levisky
> 
> Maxim Levitsky (2):
>   block/crypto: disallow write sharing by default
>   qemu-iotests: add testcase for bz #1857490

Thanks, applied to my block branch:

https://git.xanclic.moe/XanClic/qemu/commits/branch/block