On Wed, Oct 14, 2015 at 11:29:42AM +0300, Haggai Eran wrote: > respect the pkey_index in ib_send_wr.ud for GSI packets. Apparently having > the pkey_index in a work request isn't required by the IBA > specifications, I disagree. The spec is very clear, 13.5.3.2.1 perscribes how GMP replies must be generated and there are only two ways to follow that perscription: - Have some mechanism to set the pkey on outgoing QP1 GMPs - Support only one pkey What this series is doing (src QPN != 1) is absolutely not complainant with the spec. If hardware doesn't have the ability to set the pkey on outbound, then it can only support 1 pkey. This may be why reading other parts of the spec is confusing. pkey_index in the verbs section is optional, but without it an implementation cannot support multiple pkeys. Thus when multiple pkeys are supported it is not optional at all. This proposed hack violates parts of 13.5.3.2.1 so I don't think it is acceptable for core code to endorse ignoring the spec so badly. Especially when the violations are visible on the wire. Put the hack in the driver, and obsolete it when the hardware is fixed to follow the spec. Even better would be to fix this in firmware and leave the kernel alone. FWIW, IMHO, no device that works like this should be a candidate for the IBTA Interop Logo. 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
