On Tue, Oct 20, 2015 at 09:45:27AM +0300, Haggai Eran wrote: > On 19/10/2015 21:19, Jason Gunthorpe wrote: > > On Mon, Oct 19, 2015 at 09:09:25PM +0300, Haggai Eran wrote: > >> When discussing the patches to demux ids in rdma_cm instead of ib_cm, it > >> was decided that it is best to use the P_Key value in the packet headers > >> [1]. However, some drivers are currently unable to send correct P_Key in > >> GMP headers. > > > > You should explicitly describe the broken drivers in the commit text. > These are mlx5 and ipath. I'll add them to the commit message. And ipath is actually ipath, the obsolete driver being removed, not qib? (ie we don't care about it?) > The remaining issue is that it doesn't respect the > ib_send_wr.ud.pkey_index field when sending. But this is a very serious bug, to the point the mis-labeled packets may not even be delivered in some cases - you really care about the sub case where mis-labeled packets happen to be deliverable but don't parse right? Well, don't forget to undo this patch when the mlx5 driver is fixed.. 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