[v2] backup-top failure path fix

[PATCH v2 0/2] backup-top failure path fix

Vladimir Sementsov-Ogievskiy posted 2 patches 4 years, 3 months ago

Diff against v1
Download series mbox

Failed in applying to current master (apply log)

block/backup-top.c         | 21 +++++----
tests/qemu-iotests/283     | 92 ++++++++++++++++++++++++++++++++++++++
tests/qemu-iotests/283.out |  8 ++++
tests/qemu-iotests/group   |  1 +
4 files changed, 113 insertions(+), 9 deletions(-)
create mode 100644 tests/qemu-iotests/283
create mode 100644 tests/qemu-iotests/283.out

Expand all Fold all

[PATCH v2 0/2] backup-top failure path fix

Posted by Vladimir Sementsov-Ogievskiy 4 years, 3 months ago

Hi all!

Here is a small crash fix.

v2: rebase on Max's block tree
 01: add Max's r-b
 02: improve wording

Vladimir Sementsov-Ogievskiy (2):
  block/backup-top: fix failure path
  iotests: add test for backup-top failure on permission activation

 block/backup-top.c         | 21 +++++----
 tests/qemu-iotests/283     | 92 ++++++++++++++++++++++++++++++++++++++
 tests/qemu-iotests/283.out |  8 ++++
 tests/qemu-iotests/group   |  1 +
 4 files changed, 113 insertions(+), 9 deletions(-)
 create mode 100644 tests/qemu-iotests/283
 create mode 100644 tests/qemu-iotests/283.out

-- 
2.21.0

Re: [PATCH v2 0/2] backup-top failure path fix

Posted by Max Reitz 4 years, 3 months ago

On 21.01.20 15:28, Vladimir Sementsov-Ogievskiy wrote:
> Hi all!
> 
> Here is a small crash fix.

Thanks, applied to my block branch:

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

Max

Re: [PATCH v2 0/2] backup-top failure path fix

Posted by Vladimir Sementsov-Ogievskiy 4 years, 3 months ago

21.01.2020 19:32, Max Reitz wrote:
> On 21.01.20 15:28, Vladimir Sementsov-Ogievskiy wrote:
>> Hi all!
>>
>> Here is a small crash fix.
> 
> Thanks, applied to my block branch:
> 
> https://git.xanclic.moe/XanClic/qemu/commits/branch/block
> 

Thank you Max!


-- 
Best regards,
Vladimir