On Mon, May 15, 2017 at 05:44:54PM +0000, Hefty, Sean wrote: > > > + if (ib_is_opa_gid(&req_msg->primary_local_gid) && > > > + rdma_cap_opa_ah(work->port->cm_dev->ib_device, > > > + work->port->port_num)) > > > > Why isn't ib_is_opa_gid enough? > > It may be in reality, but that implies that IB and OPA are sharing GID space definitions. I understood from Ira's presentation [1] that it is the case. And doesn't it need to be "||" and not "&&"? > > It looks like rdma_cap_opa_ah() is basically being used as a check to see if the code is handling the OPA CM protocol or IB CM protocol. (Even though the two protocols are nearly identical). Where can I read the difference between these protocols? [1] https://www.openfabrics.org/images/eventpresos/2016presentations/104rdmaaltfabs.pdf Thanks
Attachment:
signature.asc
Description: PGP signature
