On Wed, Mar 22, 2017 at 01:18:30PM +0200, Erez Shitrit wrote: > 1. The qkey is not assigned at the initialization stage and the value > in ah struct is not correct. It looks like, there is no way to avoid > forwarding qkey (the item #6 - “no need to pass qkey, it is in the ah > struct”). It does make some sense that the qkey in the AH is not right, but it still does not really make sense to forward it.. Just have the driver capture the qkey during 'attach_mcast'.. Looking at it more, there is something wrong with the implementation in mlx5 since attach_mcast does not even use the qkey. I have no idea how this works since the rx side must use a filter that only allows packets that match the qkey for security. Likely you will want to add a qkey argument to 'attach_mcast' to sort this out.. > 2. In order to use the send from the ndo, there is need to pass the > pointer to ib_av struct. The immediate solution is to embed it into > the skb->data (we have 20 bytes free from the pseudo header). Well, there are two solutions, one is to encode the ah in the pseudo header during ipoib_hard_header. This is actually an optimization because ipoib_hard_header is called bfore GSO splitting, so the ah can be computed once instead of computed for every split packet. The downside here is that it may be very complicated to store a kref in a skb pseudo header and properly unref it at all the right times.. The other is to factor out the AH extraction from ipoib_start_xmit into a helper function and have all the rn drivers call it to get the AH. 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
