Re: [PATCH v5 22/27] IB/Verbs: Use management helper cap_ipoib()

[Jason Gunthorpe]

On Wed, Apr 22, 2015 at 10:49:44AM +0200, Michael Wang wrote:
> 
> On 04/22/2015 07:40 AM, Jason Gunthorpe wrote:
> > On Mon, Apr 20, 2015 at 10:41:38AM +0200, Michael Wang wrote:
> >  
> >> Introduce helper cap_ipoib() to help us check if the port of an
> >> IB device support IP over Infiniband.
> > 
> > I thought we were dropping this in favor of listing the actual
> > features the ULP required unconditionally? One of my messages had the
> > start of a list..
> 
> Shall we drop it now or wait until the mechanism introduced?
> 
> Just wondering the requirement of ULP could be similar to the
> requirement of management, isn't it? if the device can tell
> which ULP it support, then may be a cap_XX() make sense in here?

You have to audit the ipoib dirver and see what core functions it
calls that have cap requirements themselves.

At least SA, multicast and CM. It also requires cap_ib_ah() or
whatever we called that.

JAson
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/