diff options
| author |   Martin KaFai Lau <martin.lau@linux.dev> | 2022-08-30 16:19:46 -0700 |
|---|---|---|
| committer |   Daniel Borkmann <daniel@iogearbox.net> | 2022-08-31 22:21:45 +0200 |
| commit | 84e5a0f208ca341ec1ea88a97c40849a2d541faa (patch) | |
| tree | 3eb366755911c11a8e262a2aae1d68c569d25f36 /kernel/bpf/bpf_task_storage.c | |
| parent | libbpf: Add GCC support for bpf_tail_call_static (diff) | |
| download | linux-rng-84e5a0f208ca341ec1ea88a97c40849a2d541faa.tar.xz linux-rng-84e5a0f208ca341ec1ea88a97c40849a2d541faa.zip | |
bpf, net: Avoid loading module when calling bpf_setsockopt(TCP_CONGESTION)
When bpf prog changes tcp-cc by calling bpf_setsockopt(TCP_CONGESTION),
it should not try to load module which may be a blocking operation.
This details was correct in the v1 [0] but missed by mistake in the
later revision in commit cb388e7ee3a8 ("bpf: net: Change do_tcp_setsockopt()
to use the sockopt's lock_sock() and capable()"). This patch fixes it by
checking the has_current_bpf_ctx().
[0] https://lore.kernel.org/bpf/20220727060921.2373314-1-kafai@fb.com/
Fixes: cb388e7ee3a8 ("bpf: net: Change do_tcp_setsockopt() to use the sockopt's lock_sock() and capable()")
Signed-off-by: Martin KaFai Lau <martin.lau@linux.dev>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Link: https://lore.kernel.org/bpf/20220830231946.791504-1-martin.lau@linux.dev
Diffstat (limited to 'kernel/bpf/bpf_task_storage.c')
0 files changed, 0 insertions, 0 deletions