[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC PATCH 1/6] migration: Add multi-thread compress method
From: |
Zeyu Jin |
Subject: |
Re: [RFC PATCH 1/6] migration: Add multi-thread compress method |
Date: |
Tue, 10 Nov 2020 10:07:38 +0800 |
User-agent: |
Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101 Thunderbird/68.7.0 |
On 2020/11/9 23:57, Eric Blake wrote:
> On 11/9/20 3:08 AM, Zeyu Jin wrote:
>> A multi-thread compress method parameter is added to hold the method we
>> are going to use. By default the 'zlib' method is used to maintain the
>> compatibility as before.
>>
>> Signed-off-by: Zeyu Jin <jinzeyu@huawei.com>
>> Signed-off-by: Ying Fang <fangying1@huawei.com>
>> ---
>
> Focusing on just the UI:
>
>> +++ b/qapi/migration.json
>> @@ -525,6 +525,19 @@
>> 'data': [ 'none', 'zlib',
>> { 'name': 'zstd', 'if': 'defined(CONFIG_ZSTD)' } ] }
>>
>> +##
>> +# @CompressMethod:
>> +#
>> +# An enumeration of multi-thread compression methods.
>> +#
>> +# @zlib: use zlib compression method.
>> +#
>> +# Since: 5.0
>
> This will have to be Since: 6.0, as it has missed feature freeze for 5.2.
>
Thank you for noticing. I will fix that.
>> +#
>> +##
>> +{ 'enum': 'CompressMethod',
>> + 'data': [ 'zlib' ] }
>> +
>> ##
>> # @BitmapMigrationBitmapAlias:
>> #
>> @@ -599,6 +612,9 @@
>> # compression, so set the decompress-threads to the
>> number about 1/4
>> # of compress-threads is adequate.
>> #
>> +# @compress-method: Set compression method to use in multi-thread
>> compression.
>> +# Defaults to none. (Since 5.0)
>
> Looks like it will be a recurring issue in the series, so I'll quit
> pointing it out.
>
> CompressMethod above does not have a 'none' option, so how can it
> default to none? Did you forget an enum option?
>
Actually this is a comment mistake. The default option is expected to
be 'zlib'. I will fix the comment.
- [RFC PATCH 0/6] migration: Multi-thread compression with zstd method, Zeyu Jin, 2020/11/09
- [RFC PATCH 4/6] migration: Add zstd support in multi-thread compression, Zeyu Jin, 2020/11/09
- [RFC PATCH 1/6] migration: Add multi-thread compress method, Zeyu Jin, 2020/11/09
- [RFC PATCH 5/6] migration: Add compress_level sanity check, Zeyu Jin, 2020/11/09
- [RFC PATCH 6/6] doc: Update multi-thread compression doc, Zeyu Jin, 2020/11/09
- [RFC PATCH 2/6] migration: Refactoring multi-thread compress migration, Zeyu Jin, 2020/11/09
- [RFC PATCH 3/6] migration: Add multi-thread compress ops, Zeyu Jin, 2020/11/09
- Re: [RFC PATCH 0/6] migration: Multi-thread compression with zstd method, Zeyu Jin, 2020/11/15