[PATCH v21 3/4] qcow2: add zstd cluster compression

2020-04-28 Thread Denis Plotnikov
size in both cases: 1.4G Signed-off-by: Denis Plotnikov QAPI part: Acked-by: Markus Armbruster --- docs/interop/qcow2.txt | 1 + configure | 2 +- qapi/block-core.json | 3 +- block/qcow2-threads.c | 167 + block/qcow2.c | 7

[PATCH v21 1/4] qcow2: introduce compression type feature

2020-04-28 Thread Denis Plotnikov
feature compression type backing_file_offset += 56 (8 + 48 -> header_change + feature_table_change) * add "compression type" for test output matching when it isn't filtered affected tests: 049, 060, 061, 065, 144, 182, 242, 255 Signed-off-by: Denis Plotnikov Reviewed

[PATCH v21 4/4] iotests: 287: add qcow2 compression type test

2020-04-28 Thread Denis Plotnikov
The test checks fulfilling qcow2 requirements for the compression type feature and zstd compression type operability. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy Tested-by: Vladimir Sementsov-Ogievskiy --- slirp | 2 +- tests/qemu-iotests

[PATCH v21 2/4] qcow2: rework the cluster compression routine

2020-04-28 Thread Denis Plotnikov
The patch enables processing the image compression type defined for the image and chooses an appropriate method for image clusters (de)compression. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Alberto Garcia Reviewed-by: Max Reitz --- block/qcow2

[PATCH v21 0/4] implement zstd cluster compression method

2020-04-28 Thread Denis Plotnikov
ayout in qcow2 spec [Vladimir] * fix wording [Eric, Vladimir] * fix compression type filtering in iotests [Eric] v1: the initial series Denis Plotnikov (4): qcow2: introduce compression type feature qcow2: rework the cluster compression routine qcow2: add zstd cluster compressio

Re: [PATCH v20 4/4] iotests: 287: add qcow2 compression type test

2020-04-28 Thread Denis Plotnikov
On 27.04.2020 16:29, Max Reitz wrote: On 21.04.20 10:11, Denis Plotnikov wrote: The test checks fulfilling qcow2 requirements for the compression type feature and zstd compression type operability. Signed-off-by: Denis Plotnikov --- tests/qemu-iotests/287 | 146

Re: [PATCH v20 3/4] qcow2: add zstd cluster compression

2020-04-28 Thread Denis Plotnikov
On 28.04.2020 09:16, Max Reitz wrote: On 27.04.20 21:26, Denis Plotnikov wrote: On 27.04.2020 15:35, Max Reitz wrote: On 21.04.20 10:11, Denis Plotnikov wrote: zstd significantly reduces cluster compression time. It provides better compression performance maintaining the same level

Re: [RFC patch v1 2/3] qemu-file: add buffered mode

2020-04-28 Thread Denis Plotnikov
On 27.04.2020 15:14, Dr. David Alan Gilbert wrote: * Denis Plotnikov (dplotni...@virtuozzo.com) wrote: The patch adds ability to qemu-file to write the data asynchronously to improve the performance on writing. Before, only synchronous writing was supported. Enabling of the asyncronous mode

Re: [PATCH v20 3/4] qcow2: add zstd cluster compression

2020-04-27 Thread Denis Plotnikov
On 27.04.2020 15:35, Max Reitz wrote: On 21.04.20 10:11, Denis Plotnikov wrote: zstd significantly reduces cluster compression time. It provides better compression performance maintaining the same level of the compression ratio in comparison with zlib, which, at the moment, is the only

Re: [RFC patch v1 2/3] qemu-file: add buffered mode

2020-04-27 Thread Denis Plotnikov
On 25.04.2020 12:10, Vladimir Sementsov-Ogievskiy wrote: 13.04.2020 14:12, Denis Plotnikov wrote: The patch adds ability to qemu-file to write the data asynchronously to improve the performance on writing. Before, only synchronous writing was supported. Enabling of the asyncronous mode

Re: [RFC patch v1 2/3] qemu-file: add buffered mode

2020-04-27 Thread Denis Plotnikov
On 25.04.2020 00:25, Eric Blake wrote: On 4/13/20 6:12 AM, Denis Plotnikov wrote: The patch adds ability to qemu-file to write the data asynchronously to improve the performance on writing. Before, only synchronous writing was supported. Enabling of the asyncronous mode is managed by new

[PATCH v20 3/4] qcow2: add zstd cluster compression

2020-04-21 Thread Denis Plotnikov
size in both cases: 1.4G Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Alberto Garcia QAPI part: Acked-by: Markus Armbruster --- docs/interop/qcow2.txt | 1 + configure | 2 +- qapi/block-core.json | 3 +- block/qcow2-threads.c | 157

Re: [RFC patch v1 0/3] qemu-file writing performance improving

2020-04-21 Thread Denis Plotnikov
Ping! On 13.04.2020 14:12, Denis Plotnikov wrote: Problem description: qcow2 internal snapshot saving time is too big on HDD ~ 25 sec When a qcow2 image is placed on a regular HDD and the image is openned with O_DIRECT the snapshot saving time is around 26 sec. The snapshot saving time can

[PATCH v20 1/4] qcow2: introduce compression type feature

2020-04-21 Thread Denis Plotnikov
feature compression type backing_file_offset += 56 (8 + 48 -> header_change + feature_table_change) * add "compression type" for test output matching when it isn't filtered affected tests: 049, 060, 061, 065, 144, 182, 242, 255 Signed-off-by: Denis Plotnikov Reviewed

[PATCH v20 4/4] iotests: 287: add qcow2 compression type test

2020-04-21 Thread Denis Plotnikov
The test checks fulfilling qcow2 requirements for the compression type feature and zstd compression type operability. Signed-off-by: Denis Plotnikov --- tests/qemu-iotests/287 | 146 + tests/qemu-iotests/287.out | 67 + tests/qemu-iotests

[PATCH v20 0/4] qcow2: Implement zstd cluster compression methodi

2020-04-21 Thread Denis Plotnikov
Eric] * remove rudundant checks [Eric] * fix compressed cluster layout in qcow2 spec [Vladimir] * fix wording [Eric, Vladimir] * fix compression type filtering in iotests [Eric] v1: the initial series Denis Plotnikov (4): qcow2: introduce compression type feature qcow2: rework

[PATCH v20 2/4] qcow2: rework the cluster compression routine

2020-04-21 Thread Denis Plotnikov
The patch enables processing the image compression type defined for the image and chooses an appropriate method for image clusters (de)compression. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Alberto Garcia --- block/qcow2-threads.c | 71

Re: [PATCH v19 4/4] iotests: 287: add qcow2 compression type test

2020-04-20 Thread Denis Plotnikov
On 20.04.2020 17:47, Vladimir Sementsov-Ogievskiy wrote: 20.04.2020 17:30, Denis Plotnikov wrote: On 20.04.2020 15:56, Vladimir Sementsov-Ogievskiy wrote: 20.04.2020 12:17, Denis Plotnikov wrote: The test checks fulfilling qcow2 requirements for the compression type feature and zstd

Re: [PATCH v19 4/4] iotests: 287: add qcow2 compression type test

2020-04-20 Thread Denis Plotnikov
On 20.04.2020 15:56, Vladimir Sementsov-Ogievskiy wrote: 20.04.2020 12:17, Denis Plotnikov wrote: The test checks fulfilling qcow2 requirements for the compression type feature and zstd compression type operability. Signed-off-by: Denis Plotnikov ---   tests/qemu-iotests/287 | 153

[PATCH v19 3/4] qcow2: add zstd cluster compression

2020-04-20 Thread Denis Plotnikov
size in both cases: 1.4G Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Alberto Garcia QAPI part: Acked-by: Markus Armbruster --- docs/interop/qcow2.txt | 1 + configure | 2 +- qapi/block-core.json | 3 +- block/qcow2-threads.c | 157

[PATCH v19 0/4] qcow2: Implement zstd cluster compression method

2020-04-20 Thread Denis Plotnikov
[Vladimir, Eric] * fix zstd availability checking in zstd iotest [Vladimir] * remove unnecessry casting [Eric] * remove rudundant checks [Eric] * fix compressed cluster layout in qcow2 spec [Vladimir] * fix wording [Eric, Vladimir] * fix compression type filtering in iotests [Eric] v

[PATCH v19 2/4] qcow2: rework the cluster compression routine

2020-04-20 Thread Denis Plotnikov
The patch enables processing the image compression type defined for the image and chooses an appropriate method for image clusters (de)compression. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Alberto Garcia --- block/qcow2-threads.c | 71

[PATCH v19 4/4] iotests: 287: add qcow2 compression type test

2020-04-20 Thread Denis Plotnikov
The test checks fulfilling qcow2 requirements for the compression type feature and zstd compression type operability. Signed-off-by: Denis Plotnikov --- tests/qemu-iotests/287 | 153 + tests/qemu-iotests/287.out | 67 tests/qemu-iotests

[PATCH v19 1/4] qcow2: introduce compression type feature

2020-04-20 Thread Denis Plotnikov
feature compression type backing_file_offset += 56 (8 + 48 -> header_change + feature_table_change) * add "compression type" for test output matching when it isn't filtered affected tests: 049, 060, 061, 065, 144, 182, 242, 255 Signed-off-by: Denis Plotnikov Reviewed

Re: [PATCH v18 3/4] qcow2: add zstd cluster compression

2020-04-16 Thread Denis Plotnikov
On 16.04.2020 15:55, Alberto Garcia wrote: On Thu 02 Apr 2020 08:36:44 AM CEST, Denis Plotnikov wrote: +static ssize_t qcow2_zstd_compress(void *dest, size_t dest_size, + const void *src, size_t src_size) +{ +ssize_t ret; +ZSTD_outBuffer output

Re: [PATCH v18 0/4] qcow2: Implement zstd cluster compression method

2020-04-13 Thread Denis Plotnikov
Ping! Is there something to be fixed in the series? Thanks, Denis On 02.04.2020 09:36, Denis Plotnikov wrote: v18: * 04: add quotes to all file name variables [Vladimir] * 04: add Vladimir's comment according to "qemu-io write -s" option issue. v17: *

[RFC patch v1 2/3] qemu-file: add buffered mode

2020-04-13 Thread Denis Plotnikov
The patch adds ability to qemu-file to write the data asynchronously to improve the performance on writing. Before, only synchronous writing was supported. Enabling of the asyncronous mode is managed by new "enabled_buffered" callback. Signed-off-by: Denis Plotnikov --- include/qemu/

[RFC patch v1 1/3] qemu-file: introduce current buffer

2020-04-13 Thread Denis Plotnikov
is currently filled with data. This patch doesn't add any features to qemu-file and doesn't change any qemu-file behavior. Signed-off-by: Denis Plotnikov --- include/qemu/typedefs.h | 1 + migration/qemu-file.c | 156 +--- 2 files changed, 95 insertions

[RFC patch v1 3/3] migration/savevm: use qemu-file buffered mode for non-cached bdrv

2020-04-13 Thread Denis Plotnikov
with data. Signed-off-by: Denis Plotnikov --- migration/savevm.c | 38 -- 1 file changed, 36 insertions(+), 2 deletions(-) diff --git a/migration/savevm.c b/migration/savevm.c index c00a680..db0cac9 100644 --- a/migration/savevm.c +++ b/migration/savevm.c

[RFC patch v1 0/3] qemu-file writing performance improving

2020-04-13 Thread Denis Plotnikov
anything related to qemu-file using in other places, like migration. All comments are *VERY* appriciated! Thanks, Denis Denis Plotnikov (3): qemu-file: introduce current buffer qemu-file: add buffered mode migration/savevm: use qemu-file buffered mode for non-cached bdrv include/qemu

[PATCH v18 1/4] qcow2: introduce compression type feature

2020-04-02 Thread Denis Plotnikov
feature compression type backing_file_offset += 56 (8 + 48 -> header_change + feature_table_change) * add "compression type" for test output matching when it isn't filtered affected tests: 049, 060, 061, 065, 144, 182, 242, 255 Signed-off-by: Denis Plotnikov Reviewed

[PATCH v18 2/4] qcow2: rework the cluster compression routine

2020-04-02 Thread Denis Plotnikov
The patch enables processing the image compression type defined for the image and chooses an appropriate method for image clusters (de)compression. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Alberto Garcia --- block/qcow2-threads.c | 71

[PATCH v18 4/4] iotests: 287: add qcow2 compression type test

2020-04-02 Thread Denis Plotnikov
The test checks fulfilling qcow2 requiriements for the compression type feature and zstd compression type operability. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy --- tests/qemu-iotests/287 | 167 + tests/qemu-iotests/287.out

[PATCH v18 0/4] qcow2: Implement zstd cluster compression method

2020-04-02 Thread Denis Plotnikov
pe introduction patch [Vladimir, Eric] * fix zstd availability checking in zstd iotest [Vladimir] * remove unnecessry casting [Eric] * remove rudundant checks [Eric] * fix compressed cluster layout in qcow2 spec [Vladimir] * fix wording [Eric, Vladimir] * fix compression type filteri

[PATCH v18 3/4] qcow2: add zstd cluster compression

2020-04-02 Thread Denis Plotnikov
size in both cases: 1.4G Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy QAPI part: Acked-by: Markus Armbruster --- docs/interop/qcow2.txt | 1 + configure | 2 +- qapi/block-core.json | 3 +- block/qcow2-threads.c | 157

Re: [PATCH v16 3/4] qcow2: add zstd cluster compression

2020-04-01 Thread Denis Plotnikov
On 01.04.2020 18:36, Vladimir Sementsov-Ogievskiy wrote: 01.04.2020 17:37, Denis Plotnikov wrote: zstd significantly reduces cluster compression time. It provides better compression performance maintaining the same level of the compression ratio in comparison with zlib, which, at the moment

[PATCH v17 1/4] qcow2: introduce compression type feature

2020-04-01 Thread Denis Plotnikov
feature compression type backing_file_offset += 56 (8 + 48 -> header_change + feature_table_change) * add "compression type" for test output matching when it isn't filtered affected tests: 049, 060, 061, 065, 144, 182, 242, 255 Signed-off-by: Denis Plotnikov Reviewed

[PATCH v17 3/4] qcow2: add zstd cluster compression

2020-04-01 Thread Denis Plotnikov
size in both cases: 1.4G Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy QAPI part: Acked-by: Markus Armbruster --- docs/interop/qcow2.txt | 1 + configure | 2 +- qapi/block-core.json | 3 +- block/qcow2-threads.c | 157

[PATCH v17 0/4] qcow2: Implement zstd cluster compression method

2020-04-01 Thread Denis Plotnikov
ecks [Eric] * fix compressed cluster layout in qcow2 spec [Vladimir] * fix wording [Eric, Vladimir] * fix compression type filtering in iotests [Eric] v1: the initial series Denis Plotnikov (4): qcow2: introduce compression type feature qcow2: rework the cluster compression rou

[PATCH v17 2/4] qcow2: rework the cluster compression routine

2020-04-01 Thread Denis Plotnikov
The patch enables processing the image compression type defined for the image and chooses an appropriate method for image clusters (de)compression. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Alberto Garcia --- block/qcow2-threads.c | 71

[PATCH v17 4/4] iotests: 287: add qcow2 compression type test

2020-04-01 Thread Denis Plotnikov
The test checks fulfilling qcow2 requiriements for the compression type feature and zstd compression type operability. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy --- tests/qemu-iotests/287 | 162 + tests/qemu-iotests/287.out

[PATCH v16 4/4] iotests: 287: add qcow2 compression type test

2020-04-01 Thread Denis Plotnikov
The test checks fulfilling qcow2 requiriements for the compression type feature and zstd compression type operability. Signed-off-by: Denis Plotnikov --- tests/qemu-iotests/287 | 162 + tests/qemu-iotests/287.out | 70 tests/qemu-iotests

[PATCH v16 1/4] qcow2: introduce compression type feature

2020-04-01 Thread Denis Plotnikov
feature compression type backing_file_offset += 56 (8 + 48 -> header_change + feature_table_change) * add "compression type" for test output matching when it isn't filtered affected tests: 049, 060, 061, 065, 144, 182, 242, 255 Signed-off-by: Denis Plotnikov Reviewed

[PATCH v16 3/4] qcow2: add zstd cluster compression

2020-04-01 Thread Denis Plotnikov
size in both cases: 1.4G Signed-off-by: Denis Plotnikov QAPI part: Acked-by: Markus Armbruster --- docs/interop/qcow2.txt | 1 + configure | 2 +- qapi/block-core.json | 3 +- block/qcow2-threads.c | 163 + block/qcow2.c | 7

[PATCH v16 0/4] qcow2: Implement zstd cluster compression method

2020-04-01 Thread Denis Plotnikov
ecks [Eric] * fix compressed cluster layout in qcow2 spec [Vladimir] * fix wording [Eric, Vladimir] * fix compression type filtering in iotests [Eric] v1: the initial series Denis Plotnikov (4): qcow2: introduce compression type feature qcow2: rework the cluster compression r

[PATCH v16 2/4] qcow2: rework the cluster compression routine

2020-04-01 Thread Denis Plotnikov
The patch enables processing the image compression type defined for the image and chooses an appropriate method for image clusters (de)compression. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Alberto Garcia --- block/qcow2-threads.c | 71

[PATCH v15 4/4] iotests: 287: add qcow2 compression type test

2020-04-01 Thread Denis Plotnikov
The test checks fulfilling qcow2 requiriements for the compression type feature and zstd compression type operability. Signed-off-by: Denis Plotnikov --- tests/qemu-iotests/287 | 159 + tests/qemu-iotests/287.out | 70 tests/qemu-iotests

[PATCH v15 0/4] qcow2: Implement zstd cluster compression method

2020-04-01 Thread Denis Plotnikov
he compression type introduction patch [Vladimir, Eric] * fix zstd availability checking in zstd iotest [Vladimir] * remove unnecessry casting [Eric] * remove rudundant checks [Eric] * fix compressed cluster layout in qcow2 spec [Vladimir] * fix wording [Eric, Vladimir] * fix compressi

[PATCH v15 1/4] qcow2: introduce compression type feature

2020-04-01 Thread Denis Plotnikov
feature compression type backing_file_offset += 56 (8 + 48 -> header_change + feature_table_change) * add "compression type" for test output matching when it isn't filtered affected tests: 049, 060, 061, 065, 144, 182, 242, 255 Signed-off-by: Denis Plotnikov Reviewed

[PATCH v15 3/4] qcow2: add zstd cluster compression

2020-04-01 Thread Denis Plotnikov
size in both cases: 1.4G Signed-off-by: Denis Plotnikov QAPI part: Acked-by: Markus Armbruster --- docs/interop/qcow2.txt | 1 + configure | 2 +- qapi/block-core.json | 3 +- block/qcow2-threads.c | 162 + block/qcow2.c | 7

[PATCH v15 2/4] qcow2: rework the cluster compression routine

2020-04-01 Thread Denis Plotnikov
The patch enables processing the image compression type defined for the image and chooses an appropriate method for image clusters (de)compression. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Alberto Garcia --- block/qcow2-threads.c | 71

Re: [PATCH v14 4/4] iotests: 287: add qcow2 compression type test

2020-04-01 Thread Denis Plotnikov
On 31.03.2020 21:43, Eric Blake wrote: On 3/31/20 12:44 PM, Denis Plotnikov wrote: The test checks fulfilling qcow2 requiriements for the compression type feature and zstd compression type operability. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy ---   tests

Re: [PATCH v14 3/4] qcow2: add zstd cluster compression

2020-04-01 Thread Denis Plotnikov
On 01.04.2020 08:49, Vladimir Sementsov-Ogievskiy wrote: 31.03.2020 20:44, Denis Plotnikov wrote: zstd significantly reduces cluster compression time. It provides better compression performance maintaining the same level of the compression ratio in comparison with zlib, which, at the moment

[PATCH v14 1/4] qcow2: introduce compression type feature

2020-03-31 Thread Denis Plotnikov
feature compression type backing_file_offset += 56 (8 + 48 -> header_change + feature_table_change) * add "compression type" for test output matching when it isn't filtered affected tests: 049, 060, 061, 065, 144, 182, 242, 255 Signed-off-by: Denis Plotnikov Reviewed

[PATCH v14 3/4] qcow2: add zstd cluster compression

2020-03-31 Thread Denis Plotnikov
size in both cases: 1.4G Signed-off-by: Denis Plotnikov QAPI part: Acked-by: Markus Armbruster --- docs/interop/qcow2.txt | 1 + configure | 2 +- qapi/block-core.json | 3 +- block/qcow2-threads.c | 162 + block/qcow2.c | 7

[PATCH v14 4/4] iotests: 287: add qcow2 compression type test

2020-03-31 Thread Denis Plotnikov
The test checks fulfilling qcow2 requiriements for the compression type feature and zstd compression type operability. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy --- tests/qemu-iotests/287 | 128 + tests/qemu-iotests/287.out

[PATCH v14 2/4] qcow2: rework the cluster compression routine

2020-03-31 Thread Denis Plotnikov
The patch enables processing the image compression type defined for the image and chooses an appropriate method for image clusters (de)compression. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Alberto Garcia --- block/qcow2-threads.c | 71

[PATCH v14 0/4] qcow2: Implement zstd cluster compression method

2020-03-31 Thread Denis Plotnikov
* fix compression type filtering in iotests [Eric] v1: the initial series Denis Plotnikov (4): qcow2: introduce compression type feature qcow2: rework the cluster compression routine qcow2: add zstd cluster compression iotests: 287: add qcow2 compression type test docs/interop/qcow2.txt

Re: [PATCH v13 3/4] qcow2: add zstd cluster compression

2020-03-31 Thread Denis Plotnikov
On 31.03.2020 17:49, Vladimir Sementsov-Ogievskiy wrote: 31.03.2020 16:17, Denis Plotnikov wrote: zstd significantly reduces cluster compression time. It provides better compression performance maintaining the same level of the compression ratio in comparison with zlib, which, at the moment

[PATCH v13 1/4] qcow2: introduce compression type feature

2020-03-31 Thread Denis Plotnikov
feature compression type backing_file_offset += 56 (8 + 48 -> header_change + feature_table_change) * add "compression type" for test output matching when it isn't filtered affected tests: 049, 060, 061, 065, 144, 182, 242, 255 Signed-off-by: Denis Plotnikov Reviewed

[PATCH v13 0/4] qcow2: Implement zstd cluster compression method

2020-03-31 Thread Denis Plotnikov
he compression type introduction patch [Vladimir, Eric] * fix zstd availability checking in zstd iotest [Vladimir] * remove unnecessry casting [Eric] * remove rudundant checks [Eric] * fix compressed cluster layout in qcow2 spec [Vladimir] * fix wording [Eric, Vladimir] * fix compressi

[PATCH v13 4/4] iotests: 287: add qcow2 compression type test

2020-03-31 Thread Denis Plotnikov
The test checks fulfilling qcow2 requiriements for the compression type feature and zstd compression type operability. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy --- tests/qemu-iotests/287 | 128 + tests/qemu-iotests/287.out

[PATCH v13 2/4] qcow2: rework the cluster compression routine

2020-03-31 Thread Denis Plotnikov
The patch enables processing the image compression type defined for the image and chooses an appropriate method for image clusters (de)compression. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Alberto Garcia --- block/qcow2-threads.c | 71

[PATCH v13 3/4] qcow2: add zstd cluster compression

2020-03-31 Thread Denis Plotnikov
size in both cases: 1.4G Signed-off-by: Denis Plotnikov QAPI part: Acked-by: Markus Armbruster --- docs/interop/qcow2.txt | 1 + configure | 2 +- qapi/block-core.json | 3 +- block/qcow2-threads.c | 150 + block/qcow2.c | 7

[PATCH v12 2/4] qcow2: rework the cluster compression routine

2020-03-31 Thread Denis Plotnikov
The patch enables processing the image compression type defined for the image and chooses an appropriate method for image clusters (de)compression. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Alberto Garcia --- block/qcow2-threads.c | 71

[PATCH v12 4/4] iotests: 287: add qcow2 compression type test

2020-03-31 Thread Denis Plotnikov
The test checks fulfilling qcow2 requiriements for the compression type feature and zstd compression type operability. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy --- tests/qemu-iotests/287 | 128 + tests/qemu-iotests/287.out

[PATCH v12 0/4] qcow2: Implement zstd cluster compression method

2020-03-31 Thread Denis Plotnikov
unnecessry casting [Eric] * remove rudundant checks [Eric] * fix compressed cluster layout in qcow2 spec [Vladimir] * fix wording [Eric, Vladimir] * fix compression type filtering in iotests [Eric] v1: the initial series Denis Plotnikov (4): qcow2: introduce compression type fea

[PATCH v12 1/4] qcow2: introduce compression type feature

2020-03-31 Thread Denis Plotnikov
feature compression type backing_file_offset += 56 (8 + 48 -> header_change + feature_table_change) * add "compression type" for test output matching when it isn't filtered affected tests: 049, 060, 061, 065, 144, 182, 242, 255 Signed-off-by: Denis Plotnikov Reviewed

[PATCH v12 3/4] qcow2: add zstd cluster compression

2020-03-31 Thread Denis Plotnikov
size in both cases: 1.4G Signed-off-by: Denis Plotnikov QAPI part: Acked-by: Markus Armbruster --- docs/interop/qcow2.txt | 1 + configure | 2 +- qapi/block-core.json | 3 +- block/qcow2-threads.c | 124 + block/qcow2.c | 7

Re: [PATCH v11 3/4] qcow2: add zstd cluster compression

2020-03-31 Thread Denis Plotnikov
On 31.03.2020 11:10, Vladimir Sementsov-Ogievskiy wrote: 31.03.2020 10:55, Denis Plotnikov wrote: On 31.03.2020 09:22, Vladimir Sementsov-Ogievskiy wrote: 30.03.2020 18:04, Denis Plotnikov wrote: On 30.03.2020 16:14, Vladimir Sementsov-Ogievskiy wrote: 30.03.2020 12:54, Denis

Re: [PATCH v11 3/4] qcow2: add zstd cluster compression

2020-03-31 Thread Denis Plotnikov
On 31.03.2020 09:22, Vladimir Sementsov-Ogievskiy wrote: 30.03.2020 18:04, Denis Plotnikov wrote: On 30.03.2020 16:14, Vladimir Sementsov-Ogievskiy wrote: 30.03.2020 12:54, Denis Plotnikov wrote: zstd significantly reduces cluster compression time. It provides better compression

Re: [PATCH v9 1/4] qcow2: introduce compression type feature

2020-03-30 Thread Denis Plotnikov
On 30.03.2020 18:06, Markus Armbruster wrote: Denis Plotnikov writes: The patch adds some preparation parts for incompatible compression type feature to qcow2 allowing the use different compression methods for image clusters (de)compressing. It is implied that the compression type is set

Re: [PATCH v11 3/4] qcow2: add zstd cluster compression

2020-03-30 Thread Denis Plotnikov
On 30.03.2020 16:14, Vladimir Sementsov-Ogievskiy wrote: 30.03.2020 12:54, Denis Plotnikov wrote: zstd significantly reduces cluster compression time. It provides better compression performance maintaining the same level of the compression ratio in comparison with zlib, which, at the moment

[PATCH v11 1/4] qcow2: introduce compression type feature

2020-03-30 Thread Denis Plotnikov
feature compression type backing_file_offset += 56 (8 + 48 -> header_change + feature_table_change) * add "compression type" for test output matching when it isn't filtered affected tests: 049, 060, 061, 065, 144, 182, 242, 255 Signed-off-by: Denis Plotnikov Reviewed

[PATCH v11 0/4] qcow2: Implement zstd cluster compression method

2020-03-30 Thread Denis Plotnikov
on patch [Vladimir, Eric] * fix zstd availability checking in zstd iotest [Vladimir] * remove unnecessry casting [Eric] * remove rudundant checks [Eric] * fix compressed cluster layout in qcow2 spec [Vladimir] * fix wording [Eric, Vladimir] * fix compression type filtering in i

[PATCH v11 4/4] iotests: 287: add qcow2 compression type test

2020-03-30 Thread Denis Plotnikov
The test checks fulfilling qcow2 requiriements for the compression type feature and zstd compression type operability. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy --- tests/qemu-iotests/287 | 128 + tests/qemu-iotests/287.out

[PATCH v11 2/4] qcow2: rework the cluster compression routine

2020-03-30 Thread Denis Plotnikov
The patch enables processing the image compression type defined for the image and chooses an appropriate method for image clusters (de)compression. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Alberto Garcia --- block/qcow2-threads.c | 71

[PATCH v11 3/4] qcow2: add zstd cluster compression

2020-03-30 Thread Denis Plotnikov
size in both cases: 1.4G Signed-off-by: Denis Plotnikov QAPI part: Acked-by: Markus Armbruster --- docs/interop/qcow2.txt | 1 + configure | 2 +- qapi/block-core.json | 3 +- block/qcow2-threads.c | 138 + block/qcow2.c | 7

[PATCH v10 1/4] qcow2: introduce compression type feature

2020-03-30 Thread Denis Plotnikov
feature compression type backing_file_offset += 56 (8 + 48 -> header_change + feature_table_change) * add "compression type" for test output matching when it isn't filtered affected tests: 049, 060, 061, 065, 144, 182, 242, 255 Signed-off-by: Denis Plotnikov Reviewed

[PATCH v10 3/4] qcow2: add zstd cluster compression

2020-03-30 Thread Denis Plotnikov
size in both cases: 1.4G Signed-off-by: Denis Plotnikov QAPI part: Acked-by: Markus Armbruster --- docs/interop/qcow2.txt | 1 + configure | 2 +- qapi/block-core.json | 3 +- block/qcow2-threads.c | 140 + block/qcow2.c | 7

[PATCH v10 0/4] qcow2: Implement zstd cluster compression method

2020-03-30 Thread Denis Plotnikov
adimir] * fix wording [Eric, Vladimir] * fix compression type filtering in iotests [Eric] v1: the initial series Denis Plotnikov (4): qcow2: introduce compression type feature qcow2: rework the cluster compression routine qcow2: add zstd cluster compression iotests: 287: add qc

[PATCH v10 4/4] iotests: 287: add qcow2 compression type test

2020-03-30 Thread Denis Plotnikov
The test checks fulfilling qcow2 requiriements for the compression type feature and zstd compression type operability. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy --- tests/qemu-iotests/287 | 128 + tests/qemu-iotests/287.out

[PATCH v10 2/4] qcow2: rework the cluster compression routine

2020-03-30 Thread Denis Plotnikov
The patch enables processing the image compression type defined for the image and chooses an appropriate method for image clusters (de)compression. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Alberto Garcia --- block/qcow2-threads.c | 71

Re: [PATCH v9 3/4] qcow2: add zstd cluster compression

2020-03-27 Thread Denis Plotnikov
compressed image size in both cases: 1.4G Signed-off-by: Denis Plotnikov QAPI part: Acked-by: Markus Armbruster --- [..] + +/* + * qcow2_zstd_compress() + * + * Compress @src_size bytes of data using zstd compression method + * + * @dest - destination buffer, @dest_size bytes + * @src

[PATCH] gdbstub: fix compiler complaining

2020-03-26 Thread Denis Plotnikov
’ was declared here g_autofree char *cpu_name; ^ cc1: all warnings being treated as errors Signed-off-by: Denis Plotnikov --- gdbstub.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/gdbstub.c b/gdbstub.c index 013fb1ac0f..171e150950

[PATCH v9 3/4] qcow2: add zstd cluster compression

2020-03-23 Thread Denis Plotnikov
size in both cases: 1.4G Signed-off-by: Denis Plotnikov QAPI part: Acked-by: Markus Armbruster --- docs/interop/qcow2.txt | 1 + configure | 2 +- qapi/block-core.json | 3 +- block/qcow2-threads.c | 134 + block/qcow2.c | 7

[PATCH v9 1/4] qcow2: introduce compression type feature

2020-03-23 Thread Denis Plotnikov
feature compression type backing_file_offset += 56 (8 + 48 -> header_change + feature_table_change) * add "compression type" for test output matching when it isn't filtered affected tests: 049, 060, 061, 065, 144, 182, 242, 255 Signed-off-by: Denis Plotnikov --

[PATCH v9 0/4] qcow2: Implement zstd cluster compression method

2020-03-23 Thread Denis Plotnikov
otests [Eric] v1: the initial series Denis Plotnikov (4): qcow2: introduce compression type feature qcow2: rework the cluster compression routine qcow2: add zstd cluster compression iotests: 287: add qcow2 compression type test docs/interop/qcow2.txt | 1 + configure

[PATCH v9 2/4] qcow2: rework the cluster compression routine

2020-03-23 Thread Denis Plotnikov
The patch enables processing the image compression type defined for the image and chooses an appropriate method for image clusters (de)compression. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Alberto Garcia --- block/qcow2-threads.c | 71

[PATCH v9 4/4] iotests: 287: add qcow2 compression type test

2020-03-23 Thread Denis Plotnikov
The test checks fulfilling qcow2 requiriements for the compression type feature and zstd compression type operability. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy --- tests/qemu-iotests/287 | 128 + tests/qemu-iotests/287.out

Re: [PATCH v8 3/4] qcow2: add zstd cluster compression

2020-03-23 Thread Denis Plotnikov
On 23.03.2020 15:47, Alberto Garcia wrote: On Mon 23 Mar 2020 11:20:42 AM CET, Denis Plotnikov wrote: But consider corrupted image: it may contain any data. And we should not crash because of it. So, we should return error here. If the image is corrupted we can't continue anyway. If we

Re: [PATCH v8 1/4] qcow2: introduce compression type feature

2020-03-23 Thread Denis Plotnikov
On 23.03.2020 15:26, Vladimir Sementsov-Ogievskiy wrote: 23.03.2020 15:22, Denis Plotnikov wrote: On 23.03.2020 11:00, Vladimir Sementsov-Ogievskiy wrote: 21.03.2020 17:34, Denis Plotnikov wrote: The patch adds some preparation parts for incompatible compression type feature to qcow2

Re: [PATCH v8 1/4] qcow2: introduce compression type feature

2020-03-23 Thread Denis Plotnikov
On 23.03.2020 11:00, Vladimir Sementsov-Ogievskiy wrote: 21.03.2020 17:34, Denis Plotnikov wrote: The patch adds some preparation parts for incompatible compression type feature to qcow2 allowing the use different compression methods for image clusters (de)compressing. It is implied

Re: [PATCH v8 3/4] qcow2: add zstd cluster compression

2020-03-23 Thread Denis Plotnikov
On 23.03.2020 11:44, Vladimir Sementsov-Ogievskiy wrote: 21.03.2020 17:34, Denis Plotnikov wrote: zstd significantly reduces cluster compression time. It provides better compression performance maintaining the same level of the compression ratio in comparison with zlib, which, at the moment

[PATCH v8 1/4] qcow2: introduce compression type feature

2020-03-21 Thread Denis Plotnikov
feature compression type backing_file_offset += 56 (8 + 48 -> header_change + feature_table_change) * add "compression type" for test output matching when it isn't filtered affected tests: 049, 060, 061, 065, 144, 182, 242, 255 Signed-off-by: Denis Plotnikov Reviewed

[PATCH v8 0/4] qcow2: Implement zstd cluster compression method

2020-03-21 Thread Denis Plotnikov
ng [Eric, Vladimir] * fix compression type filtering in iotests [Eric] v1: the initial series Denis Plotnikov (4): qcow2: introduce compression type feature qcow2: rework the cluster compression routine qcow2: add zstd cluster compression iotests: 287: add qcow2 compre

[PATCH v8 4/4] iotests: 287: add qcow2 compression type test

2020-03-21 Thread Denis Plotnikov
The test checks fulfilling qcow2 requiriements for the compression type feature and zstd compression type operability. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy --- tests/qemu-iotests/287 | 128 + tests/qemu-iotests/287.out

[PATCH v8 3/4] qcow2: add zstd cluster compression

2020-03-21 Thread Denis Plotnikov
size in both cases: 1.4G Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Alberto Garcia QAPI part: Acked-by: Markus Armbruster --- docs/interop/qcow2.txt | 1 + configure | 2 +- qapi/block-core.json | 3 +- block/qcow2-threads.c | 129

[PATCH v8 2/4] qcow2: rework the cluster compression routine

2020-03-21 Thread Denis Plotnikov
The patch enables processing the image compression type defined for the image and chooses an appropriate method for image clusters (de)compression. Signed-off-by: Denis Plotnikov Reviewed-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Alberto Garcia --- block/qcow2-threads.c | 71

Re: [PATCH v6 3/4] qcow2: add zstd cluster compression

2020-03-16 Thread Denis Plotnikov
On 16.03.2020 17:01, Eric Blake wrote: On 3/12/20 4:22 AM, Denis Plotnikov wrote: zstd significantly reduces cluster compression time. It provides better compression performance maintaining the same level of the compression ratio in comparison with zlib, which, at the moment, is the only

<    1   2   3   4   5   6   >