qemu-block
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v5 0/5] qcow2: Implement zstd cluster compression method


From: Max Reitz
Subject: Re: [PATCH v5 0/5] qcow2: Implement zstd cluster compression method
Date: Wed, 11 Mar 2020 17:28:49 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.5.0

On 11.03.20 08:31, Denis Plotnikov wrote:
> ping!
> 
> Is there any other comments/concerns/objections/suggestions according to
> the series except the minor ones from Alberto and Vladimir?
> If not, please, let me know, so I can resend the series with the minor
> changes for applying to the corresponding branch.

Sounds good to me.

I’d like to note that most iotests that seem to do something with
compression (i.e., where grep finds a 'compress' somewhere; 013, 014,
023, 042, 046, 053, 055, ...) pass with -o compression_type=zstd, too.
060 hangs somewhere.  112 complains about v2 incompatibility; and 214
relies on intricacies of zlib, I think.  So that looks good, too.

Well, one thing I did have to fix for this to work is to quote
everything in common.pattern that tries to echo something with brackets
(e.g. “Clusters to be compressed [1]”).  I don’t quite know why the
brackets suddenly disappear when I run the tests with -o
compression_type, who knows.  But putting quotes around the echo
arguments fixes it, so...

Max

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]