]> www.infradead.org Git - users/jedix/linux-maple.git/commit
vmxnet3: Fix tx queue race condition with XDP
authorSankararaman Jayaraman <sankararaman.jayaraman@broadcom.com>
Fri, 31 Jan 2025 04:23:41 +0000 (09:53 +0530)
committerJakub Kicinski <kuba@kernel.org>
Sun, 2 Feb 2025 01:17:32 +0000 (17:17 -0800)
commit3f1baa91a1fdf3de9dbad4bd615b35fab347874b
tree57ed59ffeb73117062ad8eab119ae26ef7081abe
parenta8aa6a6ddce9b5585f2b74f27f3feea1427fb4e7
vmxnet3: Fix tx queue race condition with XDP

If XDP traffic runs on a CPU which is greater than or equal to
the number of the Tx queues of the NIC, then vmxnet3_xdp_get_tq()
always picks up queue 0 for transmission as it uses reciprocal scale
instead of simple modulo operation.

vmxnet3_xdp_xmit() and vmxnet3_xdp_xmit_frame() use the above
returned queue without any locking which can lead to race conditions
when multiple XDP xmits run in parallel on different CPU's.

This patch uses a simple module scheme when the current CPU equals or
exceeds the number of Tx queues on the NIC. It also adds locking in
vmxnet3_xdp_xmit() and vmxnet3_xdp_xmit_frame() functions.

Fixes: 54f00cce1178 ("vmxnet3: Add XDP support.")
Signed-off-by: Sankararaman Jayaraman <sankararaman.jayaraman@broadcom.com>
Signed-off-by: Ronak Doshi <ronak.doshi@broadcom.com>
Reviewed-by: Simon Horman <horms@kernel.org>
Link: https://patch.msgid.link/20250131042340.156547-1-sankararaman.jayaraman@broadcom.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/vmxnet3/vmxnet3_xdp.c