On 04/22/2015 07:24 PM, Jason Gunthorpe wrote: > 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.
I get your point :-) I'd like to suggest we put these in different threads: 1. bitmask reform 2. ulp check mechanism 3. naming (i think it'll be a really big discussion :-P) Separate them can help us focus on a particular topic at once, and the purpose of patches will be more clear ;-) Regards, Michael Wang > > JAson > -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html