Re: [PATCH for-next v2 15/16] IB/core: Map iWarp AH type to undefined in rdma_ah_find_type

[Doug Ledford <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>]

[-- Attachment #1: Type: text/plain, Size: 2011 bytes --]

On Wed, 2018-01-31 at 08:55 -0500, Dennis Dalessandro wrote:
> On 1/29/2018 8:24 AM, Dennis Dalessandro wrote:
> > On 1/26/2018 5:32 PM, Jason Gunthorpe wrote:
> > > >       if (rdma_protocol_roce(dev, port_num))
> > > >           return RDMA_AH_ATTR_TYPE_ROCE;
> > > > +    else if (rdma_protocol_iwarp(dev, port_num))
> > > > +        return RDMA_AH_ATTR_TYPE_UNDEFINED;
> > > >       else if ((rdma_protocol_ib(dev, port_num)) &&
> > > >            (rdma_cap_opa_ah(dev, port_num)))
> > > >         return RDMA_AH_ATTR_TYPE_OPA;
> > > 
> > > Let's not add an UNDEFINED type in the middle of an if casecade with 
> > > an all
> > > other cases fall through please:
> > 
> > Sounds reasonable to me, we will fix that up.
> > 
> > > I'm not sure sending all these unrelated patches in one series is such
> > > a helpful idea. Lot of trouble for everyone just to respond to one
> > > minor comment.
> > > 
> > > This series looks like 4-5 topics to me:
> > >   - Optimize HFI
> > >   - 2 patches realted to the interrupt race
> > >   - Random driver fixes
> > >   - IPoIB fix
> > >   - This patch
> > 
> > Totally agree. I have been just batching up all our patches at once and 
> > sending as that is what Doug preferred at one time. Driver is a 
> > different animal though at this point I guess. So I can start breaking 
> > them into logical series again.
> > 
> > How about just drop this series from patchworks and I'll resubmit with 
> > it broken out into multiple patch sets?
> 
> Looks like you did drop this from patchworks so I'll assume you want me 
> to break up and re-send. Will try to get that out soon.

Sorry Denny, I saw your comment and I dropped it from patchworks, but
didn't realize you were waiting on confirmation before reposting, so I
didn't say anything.  My bad.

-- 
Doug Ledford <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
    GPG KeyID: B826A3330E572FDD
    Key fingerprint = AE6B 1BDA 122B 23B4 265B  1274 B826 A333 0E57 2FDD

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 833 bytes --]