Re: [PATCH] [RESEND] IB/uverbs: fix gcc-7 type warning

From: Arnd Bergmann
Date: Wed Sep 06 2017 - 16:43:55 EST


On Wed, Sep 6, 2017 at 3:58 PM, Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote:
> On Wed, Sep 06, 2017 at 03:45:31PM +0200, Arnd Bergmann wrote:
>> We get a harmless warning about the fact that we use the result of a
>> multiplication as a condition:
>
> Please just turn the thing into a function while you're at it.
> I'm not even sure it needs to be inline..

Ok, the patch gets quite big once I also modify INIT_UDATA the same way,
but it seems like a good cleanup anyway. INIT_UDATA (renamed to
ib_uverbs_init_udata()) should definitely remain inline, so I'm leaving
both that way.

Arnd