RDMA/rxe: Get rid of pkt resend on err
authorBob Pearson <rpearsonhpe@gmail.com>
Fri, 29 Mar 2024 14:55:14 +0000 (09:55 -0500)
committerJason Gunthorpe <jgg@nvidia.com>
Mon, 22 Apr 2024 19:54:34 +0000 (16:54 -0300)
commit9cc6290991e6cfc9a6447823275fa4ba4d902103
treeb8a540f19430af7b35b8db7331ae65b6caa1c3e9
parent55bec1c440e6852e907c47cd33fbbf63fcc5f1ba
RDMA/rxe: Get rid of pkt resend on err

Currently the rxe_driver detects packet drops by ip_local_out() which
occur before the packet is sent on the wire and attempts to resend
them. This is redundant with the usual retry mechanism which covers
packets that get dropped in transit to or from the remote node.

The way this is implemented is not robust since it sets need_req_skb and
waits for the number of local skbs outstanding for this qp to drop below a
low water mark. This is racy since the skb may be sent to the destructor
before the requester can set the need_req_skb flag. This will cause a
deadlock in the send path for that qp.

This patch removes this mechanism since the normal retry path will correct
the error and resend the packet and it makes no difference if the packet
is dropped locally or later.

Link: https://lore.kernel.org/r/20240329145513.35381-14-rpearsonhpe@gmail.com
Signed-off-by: Bob Pearson <rpearsonhpe@gmail.com>
Signed-off-by: Jason Gunthorpe <jgg@nvidia.com>
drivers/infiniband/sw/rxe/rxe_net.c
drivers/infiniband/sw/rxe/rxe_req.c