[PATCH for-4.2 0/2] Fix bitmap migration

Vladimir Sementsov-Ogievskiy posted 2 patches 5 years, 11 months ago
Test docker-clang@ubuntu passed
Test docker-quick@centos7 passed
Test asan passed
Test checkpatch passed
Test FreeBSD passed
Test docker-mingw@fedora passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20191125125229.13531-1-vsementsov@virtuozzo.com
Maintainers: Max Reitz <mreitz@redhat.com>, Kevin Wolf <kwolf@redhat.com>
block/qcow2-bitmap.c       | 21 ++++++++++++++++++++-
tests/qemu-iotests/169     | 22 +++++++++++++++-------
tests/qemu-iotests/169.out |  4 ++--
3 files changed, 37 insertions(+), 10 deletions(-)
[PATCH for-4.2 0/2] Fix bitmap migration
Posted by Vladimir Sementsov-Ogievskiy 5 years, 11 months ago
Hi all!

We've faced a bug in rhev-2.12.0-33.el7-based Qemu.
In upstream, bug introduced in 4.0 by 74da6b943565c45
"block/dirty-bitmaps: implement inconsistent bit" commit.
At this commit we started to load inconsistent bitmap instead of
silently ignoring them, and it now I see that it breaks migration.

The fix is very simple, so I think it's OK for 4.2.. Still, it's not a
degradation, so we may postpone it to 5.0.

Vladimir Sementsov-Ogievskiy (2):
  block/qcow2-bitmap: fix bitmap migration
  iotests: add new test cases to bitmap migration

 block/qcow2-bitmap.c       | 21 ++++++++++++++++++++-
 tests/qemu-iotests/169     | 22 +++++++++++++++-------
 tests/qemu-iotests/169.out |  4 ++--
 3 files changed, 37 insertions(+), 10 deletions(-)

-- 
2.21.0


Re: [PATCH for-4.2 0/2] Fix bitmap migration
Posted by Max Reitz 5 years, 11 months ago
On 25.11.19 13:52, Vladimir Sementsov-Ogievskiy wrote:
> Hi all!
> 
> We've faced a bug in rhev-2.12.0-33.el7-based Qemu.
> In upstream, bug introduced in 4.0 by 74da6b943565c45
> "block/dirty-bitmaps: implement inconsistent bit" commit.
> At this commit we started to load inconsistent bitmap instead of
> silently ignoring them, and it now I see that it breaks migration.
> 
> The fix is very simple, so I think it's OK for 4.2.. Still, it's not a
> degradation, so we may postpone it to 5.0.
> 
> Vladimir Sementsov-Ogievskiy (2):
>   block/qcow2-bitmap: fix bitmap migration
>   iotests: add new test cases to bitmap migration
> 
>  block/qcow2-bitmap.c       | 21 ++++++++++++++++++++-
>  tests/qemu-iotests/169     | 22 +++++++++++++++-------
>  tests/qemu-iotests/169.out |  4 ++--
>  3 files changed, 37 insertions(+), 10 deletions(-)

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

Makes sense to me to put this into 4.2, but I don’t think it would
survive Peter’s check list. :?

(https://lists.nongnu.org/archive/html/qemu-block/2019-11/msg00807.html)

Max

Re: [PATCH for-4.2 0/2] Fix bitmap migration
Posted by Vladimir Sementsov-Ogievskiy 5 years, 11 months ago
25.11.2019 16:50, Max Reitz wrote:
> On 25.11.19 13:52, Vladimir Sementsov-Ogievskiy wrote:
>> Hi all!
>>
>> We've faced a bug in rhev-2.12.0-33.el7-based Qemu.
>> In upstream, bug introduced in 4.0 by 74da6b943565c45
>> "block/dirty-bitmaps: implement inconsistent bit" commit.
>> At this commit we started to load inconsistent bitmap instead of
>> silently ignoring them, and it now I see that it breaks migration.
>>
>> The fix is very simple, so I think it's OK for 4.2.. Still, it's not a
>> degradation, so we may postpone it to 5.0.
>>
>> Vladimir Sementsov-Ogievskiy (2):
>>    block/qcow2-bitmap: fix bitmap migration
>>    iotests: add new test cases to bitmap migration
>>
>>   block/qcow2-bitmap.c       | 21 ++++++++++++++++++++-
>>   tests/qemu-iotests/169     | 22 +++++++++++++++-------
>>   tests/qemu-iotests/169.out |  4 ++--
>>   3 files changed, 37 insertions(+), 10 deletions(-)
> 
> Reviewed-by: Max Reitz <mreitz@redhat.com>
> 
> Makes sense to me to put this into 4.2, but I don’t think it would
> survive Peter’s check list. :?

It depends on how much the fixed case is popular..

Actually, anyone who tries to migrate bitmaps with dirty-bitmaps capability
enabled and with shared storage will very possibly run into this bug.

But who is it, except for Virtuozzo, I don't know:) Still, I see that a lot
of that staff merged into Rhel Qemu we based on, so at least this should go
to qemu-stable and to Rhel update.

> 
> (https://lists.nongnu.org/archive/html/qemu-block/2019-11/msg00807.html)
> 
> Max
> 


-- 
Best regards,
Vladimir

Re: [PATCH for-4.2 0/2] Fix bitmap migration
Posted by Max Reitz 5 years, 11 months ago
On 25.11.19 13:52, Vladimir Sementsov-Ogievskiy wrote:
> Hi all!
> 
> We've faced a bug in rhev-2.12.0-33.el7-based Qemu.
> In upstream, bug introduced in 4.0 by 74da6b943565c45
> "block/dirty-bitmaps: implement inconsistent bit" commit.
> At this commit we started to load inconsistent bitmap instead of
> silently ignoring them, and it now I see that it breaks migration.
> 
> The fix is very simple, so I think it's OK for 4.2.. Still, it's not a
> degradation, so we may postpone it to 5.0.
> 
> Vladimir Sementsov-Ogievskiy (2):
>   block/qcow2-bitmap: fix bitmap migration
>   iotests: add new test cases to bitmap migration
> 
>  block/qcow2-bitmap.c       | 21 ++++++++++++++++++++-
>  tests/qemu-iotests/169     | 22 +++++++++++++++-------
>  tests/qemu-iotests/169.out |  4 ++--
>  3 files changed, 37 insertions(+), 10 deletions(-)

Thanks, applied to my block branch:

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

Max