On 11/12/2015 3:30 PM, Hefty, Sean wrote:
>>> + /* CM attributes other than ClassPortInfo only use Send method
>> */
>>> + if (mad_hdr->mgmt_class == IB_MGMT_CLASS_CM) {
>>> + if (mad_hdr->attr_id != IB_MGMT_CLASSPORTINFO_ATTR_ID) {
>>> + if (mad_hdr->method != IB_MGMT_METHOD_SEND)
>>> + goto out;
>>> + } else if (mad_hdr->method != IB_MGMT_METHOD_GET_RESP)
>>> + goto out;
>>> + }
>>
>> Doesn't this invalidate a CM Get(ClassPortInfo) mad?
>
> I believe this does. I think you could remove the else if clause and let the
> received MAD get passed to the CM. It would be dropped there as unsupported.
> The net result is likely the same.
Right now this is indeed the case as CPI is not supported in CM.
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
