diff options
author | John Fastabend | 2017-10-18 16:10:36 +0200 |
---|---|---|
committer | David S. Miller | 2017-10-20 14:01:29 +0200 |
commit | 34f79502bbcfab659b8729da68b5e387f96eb4c1 (patch) | |
tree | 27084757c315fdf13c682c3d6c57922ed5f8447c /include/linux/filter.h | |
parent | bpf: enforce TCP only support for sockmap (diff) | |
download | kernel-qcow2-linux-34f79502bbcfab659b8729da68b5e387f96eb4c1.tar.gz kernel-qcow2-linux-34f79502bbcfab659b8729da68b5e387f96eb4c1.tar.xz kernel-qcow2-linux-34f79502bbcfab659b8729da68b5e387f96eb4c1.zip |
bpf: avoid preempt enable/disable in sockmap using tcp_skb_cb region
SK_SKB BPF programs are run from the socket/tcp context but early in
the stack before much of the TCP metadata is needed in tcp_skb_cb. So
we can use some unused fields to place BPF metadata needed for SK_SKB
programs when implementing the redirect function.
This allows us to drop the preempt disable logic. It does however
require an API change so sk_redirect_map() has been updated to
additionally provide ctx_ptr to skb. Note, we do however continue to
disable/enable preemption around actual BPF program running to account
for map updates.
Signed-off-by: John Fastabend <john.fastabend@gmail.com>
Acked-by: Daniel Borkmann <daniel@iogearbox.net>
Acked-by: Alexei Starovoitov <ast@kernel.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/linux/filter.h')
-rw-r--r-- | include/linux/filter.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/filter.h b/include/linux/filter.h index d29e58fde364..818a0b26249e 100644 --- a/include/linux/filter.h +++ b/include/linux/filter.h @@ -728,7 +728,7 @@ void xdp_do_flush_map(void); void bpf_warn_invalid_xdp_action(u32 act); void bpf_warn_invalid_xdp_redirect(u32 ifindex); -struct sock *do_sk_redirect_map(void); +struct sock *do_sk_redirect_map(struct sk_buff *skb); #ifdef CONFIG_BPF_JIT extern int bpf_jit_enable; |