[PATCH v2 0/6] parallels: load bitmap extension

Vladimir Sementsov-Ogievskiy posted 6 patches 4 years, 8 months ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20210224104707.88430-1-vsementsov@virtuozzo.com
Maintainers: John Snow <jsnow@redhat.com>, Eric Blake <eblake@redhat.com>, Max Reitz <mreitz@redhat.com>, Kevin Wolf <kwolf@redhat.com>, "Denis V. Lunev" <den@openvz.org>, Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>, Stefan Hajnoczi <stefanha@redhat.com>
docs/interop/parallels.txt                    |  27 +-
block/parallels.h                             |   7 +-
include/block/dirty-bitmap.h                  |   2 +
block/dirty-bitmap.c                          |  13 +
block/parallels-ext.c                         | 300 ++++++++++++++++++
block/parallels.c                             |  26 +-
block/qcow2-bitmap.c                          |  16 +-
block/meson.build                             |   3 +-
tests/qemu-iotests/iotests.py                 |  10 +
.../sample_images/parallels-with-bitmap.bz2   | Bin 0 -> 203 bytes
.../sample_images/parallels-with-bitmap.sh    |  51 +++
.../qemu-iotests/tests/parallels-read-bitmap  |  55 ++++
.../tests/parallels-read-bitmap.out           |   6 +
13 files changed, 484 insertions(+), 32 deletions(-)
create mode 100644 block/parallels-ext.c
create mode 100644 tests/qemu-iotests/sample_images/parallels-with-bitmap.bz2
create mode 100755 tests/qemu-iotests/sample_images/parallels-with-bitmap.sh
create mode 100755 tests/qemu-iotests/tests/parallels-read-bitmap
create mode 100644 tests/qemu-iotests/tests/parallels-read-bitmap.out
[PATCH v2 0/6] parallels: load bitmap extension
Posted by Vladimir Sementsov-Ogievskiy 4 years, 8 months ago
Hi all!

We need to load bitmaps from parallels image in our product.
So here is a feature.

v2:
01: tweak commit message, add r-bs by Eric and Denis
02: tweak wording by Denis's suggestions
03: new, suggested by Denis
04: rebase on 03, add several error checking,
    convert l1 table to cpu from small-endian,
    allow loading several bitmaps
06: add copyright, drop "/work/mega"

Vladimir Sementsov-Ogievskiy (6):
  qcow2-bitmap: make bytes_covered_by_bitmap_cluster() public
  parallels.txt: fix bitmap L1 table description
  block/parallels: BDRVParallelsState: add cluster_size field
  parallels: support bitmap extension for read-only mode
  iotests.py: add unarchive_sample_image() helper
  iotests: add parallels-read-bitmap test

 docs/interop/parallels.txt                    |  27 +-
 block/parallels.h                             |   7 +-
 include/block/dirty-bitmap.h                  |   2 +
 block/dirty-bitmap.c                          |  13 +
 block/parallels-ext.c                         | 300 ++++++++++++++++++
 block/parallels.c                             |  26 +-
 block/qcow2-bitmap.c                          |  16 +-
 block/meson.build                             |   3 +-
 tests/qemu-iotests/iotests.py                 |  10 +
 .../sample_images/parallels-with-bitmap.bz2   | Bin 0 -> 203 bytes
 .../sample_images/parallels-with-bitmap.sh    |  51 +++
 .../qemu-iotests/tests/parallels-read-bitmap  |  55 ++++
 .../tests/parallels-read-bitmap.out           |   6 +
 13 files changed, 484 insertions(+), 32 deletions(-)
 create mode 100644 block/parallels-ext.c
 create mode 100644 tests/qemu-iotests/sample_images/parallels-with-bitmap.bz2
 create mode 100755 tests/qemu-iotests/sample_images/parallels-with-bitmap.sh
 create mode 100755 tests/qemu-iotests/tests/parallels-read-bitmap
 create mode 100644 tests/qemu-iotests/tests/parallels-read-bitmap.out

-- 
2.29.2


Re: [PATCH v2 0/6] parallels: load bitmap extension
Posted by Kevin Wolf 4 years, 8 months ago
Am 24.02.2021 um 11:47 hat Vladimir Sementsov-Ogievskiy geschrieben:
> Hi all!
> 
> We need to load bitmaps from parallels image in our product.
> So here is a feature.

Thanks, applied to the block branch.

I changed some sentences in patch 2 as suggested in my reply to it.
Please let me know if you're happy with these changes or if you would
like to tweak them a bit more.

Kevin