]> git.hungrycats.org Git - linux/commit
RDMA/rxe: Fix seg fault in rxe_comp_queue_pkt
authorBob Pearson <rpearsonhpe@gmail.com>
Fri, 29 Mar 2024 14:55:04 +0000 (09:55 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 30 May 2024 07:49:44 +0000 (09:49 +0200)
commitbbad88f111a1829f366c189aa48e7e58e57553fc
tree9c5d5efdd25fe8d4db99ebd03b1a3246770f2cfb
parent8ab9eeef728ab62ff7f43efc4b9d835ef0a465b1
RDMA/rxe: Fix seg fault in rxe_comp_queue_pkt

[ Upstream commit 2b23b6097303ed0ba5f4bc036a1c07b6027af5c6 ]

In rxe_comp_queue_pkt() an incoming response packet skb is enqueued to the
resp_pkts queue and then a decision is made whether to run the completer
task inline or schedule it. Finally the skb is dereferenced to bump a 'hw'
performance counter. This is wrong because if the completer task is
already running in a separate thread it may have already processed the skb
and freed it which can cause a seg fault.  This has been observed
infrequently in testing at high scale.

This patch fixes this by changing the order of enqueuing the packet until
after the counter is accessed.

Link: https://lore.kernel.org/r/20240329145513.35381-4-rpearsonhpe@gmail.com
Signed-off-by: Bob Pearson <rpearsonhpe@gmail.com>
Fixes: 0b1e5b99a48b ("IB/rxe: Add port protocol stats")
Signed-off-by: Jason Gunthorpe <jgg@nvidia.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/infiniband/sw/rxe/rxe_comp.c