diff options
author | Benoît Ganne <bganne@cisco.com> | 2021-04-29 18:24:24 +0200 |
---|---|---|
committer | Damjan Marion <dmarion@me.com> | 2021-05-21 19:50:14 +0000 |
commit | a42c41be4eed3e1ce2a42038b07ce1d3420891cd (patch) | |
tree | fc95c7c24cbef993cc2bef8742b3360123d70b66 /src/plugins/af_xdp/af_xdp.api | |
parent | 92a8d761c412590f5112239be4c511091b2b2d5a (diff) |
af_xdp: workaround kernel race between poll() and sendmsg()
Prior to Linux 5.6 there is a race condition between poll() and
sendmsg() in the kernel. This patch protects the syscalls with a lock
to prevent it, unless the NO_SYSCALL_LOCK flag is set at create time.
See
https://lore.kernel.org/bpf/BYAPR11MB365382C5DB1E5FCC53242609C1549@BYAPR11MB3653.namprd11.prod.outlook.com/
Type: fix
Change-Id: Ie7d4f5cb41f697b11a09b6046e54d190430d76df
Signed-off-by: Benoît Ganne <bganne@cisco.com>
Diffstat (limited to 'src/plugins/af_xdp/af_xdp.api')
-rw-r--r-- | src/plugins/af_xdp/af_xdp.api | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/src/plugins/af_xdp/af_xdp.api b/src/plugins/af_xdp/af_xdp.api index 14f51d87d6a..c6716123703 100644 --- a/src/plugins/af_xdp/af_xdp.api +++ b/src/plugins/af_xdp/af_xdp.api @@ -15,7 +15,7 @@ *------------------------------------------------------------------ */ -option version = "0.1.0"; +option version = "0.2.0"; import "vnet/interface_types.api"; enum af_xdp_mode @@ -25,6 +25,10 @@ enum af_xdp_mode AF_XDP_API_MODE_ZERO_COPY = 2, }; +enumflag af_xdp_flag : u8 +{ + AF_XDP_API_FLAGS_NO_SYSCALL_LOCK = 1, +}; /** \brief @param client_index - opaque cookie to identify the sender @@ -35,6 +39,7 @@ enum af_xdp_mode @param rxq_size - receive queue size (optional) @param txq_size - transmit queue size (optional) @param mode - operation mode (optional) + @param flags - flags (optional) @param prog - eBPF program path (optional) */ @@ -49,8 +54,9 @@ define af_xdp_create u16 rxq_size [default=0]; u16 txq_size [default=0]; vl_api_af_xdp_mode_t mode [default=0]; + vl_api_af_xdp_flag_t flags [default=0]; string prog[256]; - option vat_help = "<host-if linux-ifname> [name ifname] [rx-queue-size size] [tx-queue-size size] [num-rx-queues <num|all>] [prog pathname] [zero-copy|no-zero-copy]"; + option vat_help = "<host-if linux-ifname> [name ifname] [rx-queue-size size] [tx-queue-size size] [num-rx-queues <num|all>] [prog pathname] [zero-copy|no-zero-copy] [no-syscall-lock]"; option status="in_progress"; }; |