Re: [patch] IB/core: missing curly braces in ib_find_gid()

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

 



On 08/30/2015 12:59 AM, Or Gerlitz wrote:
> On Fri, Aug 28, 2015 at 9:02 PM, Doug Ledford <dledford@xxxxxxxxxx> wrote:
>> On 08/28/2015 09:10 PM, ira.weiny wrote:
>>> On Tue, Aug 18, 2015 at 12:22:10PM +0300, Dan Carpenter wrote:
>>>> Smatch says that, based on the indenting, we should probably add curly
>>>> braces here.
>>>>
>>>> Fixes: 230145ff8124 ('IB/core: Add RoCE GID table management')
>>>> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
>>>>
>>>
>>> Reviewed-by: Ira Weiny <ira.weiny@xxxxxxxxx>
>>
>> I've picked this patch up (earlier today), so I missed your reviewed-by:
>> tag.  Added.
> 
> wait, Doug, when the offending patch goes upstream, their commit ID
> will not be the one
> denoted here, right?

Fair point...

> I would suggest that you either hold off with
> this fix till the other patch
> goes upstream in 4,3-rc1 and you have a proper ID

I have a proper ID now.  Since I've put together my final k.o/for-4.3
branch, the ID is fixed.

> or squash it there
> now.

This patch is from someone different than the patch it fixes, so it
wouldn't be right to squash it into their patch.  I fixed up the Fixes:
line instead.

> BTW - the way
> net-next is organized/managed, pre-linus-tree commit ID remain the
> same also after they land there,
> so in that respect and other linux-rdma maintenance related aspects we
> discussed during LPC  - it
> would be good if you can talk to Dave and see how he gets things going
> for net-next

I'm going to be doing things differently for 4.4.  We'll see if that
resolves the issues you are unhappy about.

-- 
Doug Ledford <dledford@xxxxxxxxxx>
              GPG KeyID: 0E572FDD


Attachment: signature.asc
Description: OpenPGP digital signature


[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux