[Top][All Lists]

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

Re: [Qemu-block] [Qemu-devel] [PATCH v6 00/15] nbd: efficient write zero

From: Eric Blake
Subject: Re: [Qemu-block] [Qemu-devel] [PATCH v6 00/15] nbd: efficient write zeroes
Date: Fri, 14 Oct 2016 09:19:46 -0500
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.3.0

On 10/13/2016 11:59 PM,
address@hidden wrote:

> Checking PATCH 13/15: nbd: Improve server handling of shutdown requests...
> ERROR: return of an errno should typically be -ve (return -ESHUTDOWN)
> #63: FILE: nbd/client.c:38:
> +        return ESHUTDOWN;
> total: 1 errors, 0 warnings, 95 lines checked
> Your patch has style problems, please review.  If any of these errors
> are false positives report them to the maintainer, see

False positive; although we COULD rewrite the function to do:

return ret;

(and so on for all the cases in the switch), to hide the fact that we
intentionally want to return a positive errno value, if it bothers
anyone enough to ask me to do so.

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]