qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH V19 06/12] quorum: Add quorum mechanism.


From: Eric Blake
Subject: Re: [Qemu-devel] [PATCH V19 06/12] quorum: Add quorum mechanism.
Date: Fri, 21 Feb 2014 16:32:31 -0700
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.3.0

On 02/21/2014 03:50 PM, Benoît Canet wrote:
> The Friday 21 Feb 2014 à 15:44:06 (-0700), Eric Blake wrote :
>> On 02/21/2014 03:38 PM, Kevin Wolf wrote:
>>>>>> +- "ret":          The IO return code.
>>>>>
>>>>> What values is this likely to contain?  Is it a finite set, in which
>>>>> case it would be nice to have a QAPI enum that describes the set of
>>>>> return codes, rather than a raw number?
>>>>
>>>> It's anything that the block stack could return as an error.
>>>
>>> In other words, it's meaning depends on the host and the value is only
>>> suitable for human readers. Perhaps we could change this to strerror(),
>>> which is, I believe, the same as error_setg_errno() does.
>>
>> Ah, so you're saying that 'ret' would be a '-errno' value - in that
>> case, yes, converting it to string, and documenting this field as a
>> human-only strerror() representation of the error would also work (using
>> 'str', rather than an enum type).
> 
> How do I proceed ?
> 
> Should I respin the serie ? Or do a follow up patch ? 

Followup is fine, as long as we get it done before 2.0.

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

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

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