> However this patch is not correct where it uses random udp source port number. > ib_core needs to bind the source udp port number and store it into ib_qp. > I have been holding that up, due to gid ref count series, so that I can use that logic for destination and source udp port and store the socket pointer in ib_gid_attr. > Once that is available that that can be used by rxe and other drivers. > However multiple src udp port number has benefits which can be a list per gid entry as mentioned in this patch. > Per device would be simpler, but given that we are sharing the device among multiple namespaces in some cases, having per gid entry is simpler to start with. I'm not sure that I understand where your'e aiming with changes in ib_core. This change looks fine to me and I would like to accept it. When you present your idea we can adapt it if it makes sense to resolving this problem but until then I would like to have this fix for a real problem that RXE has. -- 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
