qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [v7 13/14] migration: Add qmp commands to set and query


From: Eric Blake
Subject: Re: [Qemu-devel] [v7 13/14] migration: Add qmp commands to set and query parameters
Date: Tue, 14 Apr 2015 06:16:04 -0600
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0

On 04/14/2015 06:09 AM, Eric Blake wrote:
> On 04/08/2015 12:20 AM, Liang Li wrote:
>> Add the qmp commands to tune and query the parameters used in live
>> migration.
>>
>> Signed-off-by: Liang Li <address@hidden>
>> Signed-off-by: Yang Zhang <address@hidden>
>> ---
>>  migration/migration.c | 56 
>> ++++++++++++++++++++++++++++++++++++++++++++++++++
>>  qapi-schema.json      | 45 ++++++++++++++++++++++++++++++++++++++++
>>  qmp-commands.hx       | 57 
>> +++++++++++++++++++++++++++++++++++++++++++++++++++
>>  3 files changed, 158 insertions(+)

>> +    if (has_compress_level && (compress_level < 0 || compress_level > 9)) {
>> +        error_set(errp, QERR_INVALID_PARAMETER_VALUE, "compress_level",
>> +                  "is invalid, it should be in the range of 0 to 9");

All three error messages are bogus; this expands to:
"Parameter 'compress_level' expects is invalid, it should be in the
range of 0 to 9"


> Minor enough that I'm okay if the maintainer fixes the release number
> and adds:
> Reviewed-by: Eric Blake <address@hidden>

I'm no longer okay with this patch; you'll need a v8 that improves the
error wording (QERR_ macros are seldom the right thing to use, you may
be better off open-coding error_setg() and doing a sane message in place).

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org



reply via email to

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