[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Xen-devel] [PATCH 1/2] xen: fix usage of ENODATA
From: |
Roger Pau Monné |
Subject: |
Re: [Qemu-devel] [Xen-devel] [PATCH 1/2] xen: fix usage of ENODATA |
Date: |
Wed, 16 Apr 2014 16:31:03 +0200 |
User-agent: |
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:24.0) Gecko/20100101 Thunderbird/24.4.0 |
On 15/04/14 14:33, Egger, Christoph wrote:
> On 15.04.14 10:33, Roger Pau Monne wrote:
>> ENODATA doesn't exist on FreeBSD, so ENODATA errors returned by the
>> hypervisor are translated to ENOENT.
>>
>> Also, the error code is returned in errno if the call returns -1, so
>> compare the error code with the value in errno instead of the value
>> returned by the function.
>>
>> Signed-off-by: Roger Pau Monné <address@hidden>
>> Cc: address@hidden
>> Cc: Stefano Stabellini <address@hidden>
>> Cc: Anthony Perard <address@hidden>
>> ---
>> xen-all.c | 8 ++++++--
>> 1 files changed, 6 insertions(+), 2 deletions(-)
>>
>> diff --git a/xen-all.c b/xen-all.c
>> index 0b4d934..4026b7b 100644
>> --- a/xen-all.c
>> +++ b/xen-all.c
>> @@ -499,11 +499,15 @@ static void xen_sync_dirty_bitmap(XenIOState *state,
>> start_addr >> TARGET_PAGE_BITS, npages,
>> bitmap);
>> if (rc < 0) {
>> - if (rc != -ENODATA) {
>> +#ifdef ENODATA
>> + if (errno == ENODATA) {
>> +#else
>> + if (errno == ENOENT) {
>> +#endif
>
> This does not look good to me.
> I suggest this:
>
> #ifndef ENODATA
> #define ENODATA ENOENT
> #endif
Hello Christoph,
Thanks for the review, I slightly prefer my original proposal over what
you suggest, I think it's more clear that there are different error
codes, while your suggestion simply masks the fact than FreeBSD doesn't
have ENODATA. Anyway, I wouldn't mind changing it if the maintainer
prefers your approach.
Roger.