On Tue, Apr 07, 2015 at 05:41:34PM +0000, Hefty, Sean wrote: > > > > The IBA is pretty clear what should happen to process an unsupported > > class > > > > version and and adding OPA shouldn't suddenly make the IB side > > > > non-conformant, however aesthetically unpleasing the code may be. > > > > > > Are there checks to ensure that MADs not supported by RoCE aren't > > > processed on RoCE ports? > > > > RoCE isn't different, it uses the same numbering space and rules as > > IB. > > I was agreeing with the point made about non-compliance. RoCE only > supports a subset of IB management, and AFAIK, there are no checks > to validate MADs received over a RoCE port. Those should be added. RoCE follows the base IB spec pretty well, except that there is no QP0 (or SMPs, or SMI, etc). There is already code that avoids creating QP0 MAD stuff and the SMI for RoCE. A RoCE driver should never deliver a QP0 packet to the mad layer. There are already various checks that tie SMP's to QP0, on tx and also in validate_mad. Eg validate_mad already refuses every SMP on RoCE since qp_num can never be 0. So I think we are good already. Every difference not caught by the missing QP0 is handled by not registering certain agents for RoCE, which looks OK already since I think most GSI's are supported. Jason -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html