[PATCH 0/2] block: Fix multiple blockdev-snapshot calls

Kevin Wolf posted 2 patches 4 years, 4 months ago
Failed in applying to current master (apply log)
block.c                    |   2 +
tests/qemu-iotests/273     |  76 +++++++++
tests/qemu-iotests/273.out | 337 +++++++++++++++++++++++++++++++++++++
tests/qemu-iotests/group   |   1 +
4 files changed, 416 insertions(+)
create mode 100755 tests/qemu-iotests/273
create mode 100644 tests/qemu-iotests/273.out
[PATCH 0/2] block: Fix multiple blockdev-snapshot calls
Posted by Kevin Wolf 4 years, 4 months ago
Kevin Wolf (2):
  block: Remove 'backing': null from bs->{explicit_,}options
  iotests: Test multiple blockdev-snapshot calls

 block.c                    |   2 +
 tests/qemu-iotests/273     |  76 +++++++++
 tests/qemu-iotests/273.out | 337 +++++++++++++++++++++++++++++++++++++
 tests/qemu-iotests/group   |   1 +
 4 files changed, 416 insertions(+)
 create mode 100755 tests/qemu-iotests/273
 create mode 100644 tests/qemu-iotests/273.out

-- 
2.20.1


Re: [PATCH 0/2] block: Fix multiple blockdev-snapshot calls
Posted by Peter Krempa 4 years, 4 months ago
On Fri, Nov 08, 2019 at 09:53:10 +0100, Kevin Wolf wrote:
> Kevin Wolf (2):
>   block: Remove 'backing': null from bs->{explicit_,}options
>   iotests: Test multiple blockdev-snapshot calls

Hi,

I'm not sure how the freeze rules for qemu are at this point thus:

Will this patchset make it into 4.2. I argue it's a pretty important fix
as it ends up in image corruption.

If it won't make into we will need to add a QMP feature flag to detect
presence of the fix so that I can gate libvirt's support of blockdev
with it. (This can also be done if we make it into 4.2 but I'll be okay
infering it from other fixes present in 4.2).

Thanks

Peter
Re: [PATCH 0/2] block: Fix multiple blockdev-snapshot calls
Posted by Kevin Wolf 4 years, 4 months ago
Am 18.11.2019 um 17:26 hat Peter Krempa geschrieben:
> On Fri, Nov 08, 2019 at 09:53:10 +0100, Kevin Wolf wrote:
> > Kevin Wolf (2):
> >   block: Remove 'backing': null from bs->{explicit_,}options
> >   iotests: Test multiple blockdev-snapshot calls
> 
> Hi,
> 
> I'm not sure how the freeze rules for qemu are at this point thus:
> 
> Will this patchset make it into 4.2. I argue it's a pretty important fix
> as it ends up in image corruption.
> 
> If it won't make into we will need to add a QMP feature flag to detect
> presence of the fix so that I can gate libvirt's support of blockdev
> with it. (This can also be done if we make it into 4.2 but I'll be okay
> infering it from other fixes present in 4.2).

Yes, this is planned for 4.2. I'm in the process of preparing a pull
request right now.

Kevin