xsk: Avoid starving the xsk further down the list
authorAlbert Huang <huangjie.albert@bytedance.com>
Mon, 23 Oct 2023 12:57:31 +0000 (20:57 +0800)
committerDaniel Borkmann <daniel@iogearbox.net>
Tue, 24 Oct 2023 09:55:36 +0000 (11:55 +0200)
commit99b29a499b5fdfb7ab274835b8e4d4c11df2f6d7
tree1d4aa58f8b314eabe16eb93feaa6a1fe4cbbdb8e
parentdedd6c894110371d3c218cf24ecca2f0730408ac
xsk: Avoid starving the xsk further down the list

In the previous implementation, when multiple xsk sockets were
associated with a single xsk_buff_pool, a situation could arise
where the xsk_tx_list maintained data at the front for one xsk
socket while starving the xsk sockets at the back of the list.
This could result in issues such as the inability to transmit packets,
increased latency, and jitter. To address this problem, we introduce
a new variable called tx_budget_spent, which limits each xsk to transmit
a maximum of MAX_PER_SOCKET_BUDGET tx descriptors. This allocation ensures
equitable opportunities for subsequent xsk sockets to send tx descriptors.
The value of MAX_PER_SOCKET_BUDGET is set to 32.

Signed-off-by: Albert Huang <huangjie.albert@bytedance.com>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Acked-by: Magnus Karlsson <magnus.karlsson@intel.com>
Link: https://lore.kernel.org/bpf/20231023125732.82261-1-huangjie.albert@bytedance.com
include/net/xdp_sock.h
net/xdp/xsk.c