[PATCH v12 0/3] qcow2: advanced compression options

Andrey Shinkevich posted 3 patches 4 years, 3 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/1575288906-551879-1-git-send-email-andrey.shinkevich@virtuozzo.com
Test asan passed
Test checkpatch passed
Test FreeBSD passed
Test docker-mingw@fedora passed
Test docker-clang@ubuntu failed
Test docker-quick@centos7 passed
Maintainers: Markus Armbruster <armbru@redhat.com>, Max Reitz <mreitz@redhat.com>, Eric Blake <eblake@redhat.com>, Kevin Wolf <kwolf@redhat.com>
block/Makefile.objs        |   1 +
block/filter-compress.c    | 168 +++++++++++++++++++++++++++++++++++++++++++++
block/qcow2.c              | 102 +++++++++++++++++++--------
qapi/block-core.json       |  10 +--
tests/qemu-iotests/214     |  43 ++++++++++++
tests/qemu-iotests/214.out |  14 ++++
6 files changed, 307 insertions(+), 31 deletions(-)
create mode 100644 block/filter-compress.c
[PATCH v12 0/3] qcow2: advanced compression options
Posted by Andrey Shinkevich 4 years, 3 months ago
The compression filter driver is introduced as suggested by Max.
A sample usage of the filter can be found in the test #214.
Now, multiple clusters can be written compressed.
It is useful for the backup job.

v12:
  01: Missed to change the driver interface .bdrv_co_block_status
      from _status_from_backing to _status_from_file (noticed by
      Vladimir).

Andrey Shinkevich (3):
  block: introduce compress filter driver
  qcow2: Allow writing compressed data of multiple clusters
  tests/qemu-iotests: add case to write compressed data of multiple
    clusters

 block/Makefile.objs        |   1 +
 block/filter-compress.c    | 168 +++++++++++++++++++++++++++++++++++++++++++++
 block/qcow2.c              | 102 +++++++++++++++++++--------
 qapi/block-core.json       |  10 +--
 tests/qemu-iotests/214     |  43 ++++++++++++
 tests/qemu-iotests/214.out |  14 ++++
 6 files changed, 307 insertions(+), 31 deletions(-)
 create mode 100644 block/filter-compress.c

-- 
1.8.3.1


Re: [PATCH v12 0/3] qcow2: advanced compression options
Posted by Max Reitz 4 years, 3 months ago
On 02.12.19 13:15, Andrey Shinkevich wrote:
> The compression filter driver is introduced as suggested by Max.
> A sample usage of the filter can be found in the test #214.
> Now, multiple clusters can be written compressed.
> It is useful for the backup job.
> 
> v12:
>   01: Missed to change the driver interface .bdrv_co_block_status
>       from _status_from_backing to _status_from_file (noticed by
>       Vladimir).
> 
> Andrey Shinkevich (3):
>   block: introduce compress filter driver
>   qcow2: Allow writing compressed data of multiple clusters
>   tests/qemu-iotests: add case to write compressed data of multiple
>     clusters
> 
>  block/Makefile.objs        |   1 +
>  block/filter-compress.c    | 168 +++++++++++++++++++++++++++++++++++++++++++++
>  block/qcow2.c              | 102 +++++++++++++++++++--------
>  qapi/block-core.json       |  10 +--
>  tests/qemu-iotests/214     |  43 ++++++++++++
>  tests/qemu-iotests/214.out |  14 ++++
>  6 files changed, 307 insertions(+), 31 deletions(-)
>  create mode 100644 block/filter-compress.c

Thanks, fixed patch 1 and applied to my block branch:

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

Max

Re: [PATCH v12 0/3] qcow2: advanced compression options
Posted by Andrey Shinkevich 4 years, 3 months ago
Pinging...
(please)

On 02/12/2019 15:15, Andrey Shinkevich wrote:
> The compression filter driver is introduced as suggested by Max.
> A sample usage of the filter can be found in the test #214.
> Now, multiple clusters can be written compressed.
> It is useful for the backup job.
> 
> v12:
>    01: Missed to change the driver interface .bdrv_co_block_status
>        from _status_from_backing to _status_from_file (noticed by
>        Vladimir).
> 
> Andrey Shinkevich (3):
>    block: introduce compress filter driver
>    qcow2: Allow writing compressed data of multiple clusters
>    tests/qemu-iotests: add case to write compressed data of multiple
>      clusters
> 
>   block/Makefile.objs        |   1 +
>   block/filter-compress.c    | 168 +++++++++++++++++++++++++++++++++++++++++++++
>   block/qcow2.c              | 102 +++++++++++++++++++--------
>   qapi/block-core.json       |  10 +--
>   tests/qemu-iotests/214     |  43 ++++++++++++
>   tests/qemu-iotests/214.out |  14 ++++
>   6 files changed, 307 insertions(+), 31 deletions(-)
>   create mode 100644 block/filter-compress.c
> 

-- 
With the best regards,
Andrey Shinkevich