Thread (3 messages) 3 messages, 2 authors, 2021-10-26

Re: [PATCH] bpf: Move BPF_MAP_TYPE for INODE_STORAGE and TASK_STORAGE outside of CONFIG_NET

From: Martin KaFai Lau <hidden>
Date: 2021-10-26 19:25:47
Also in: bpf, lkml

On Tue, Oct 26, 2021 at 12:19:36PM -0700, Martin KaFai Lau wrote:
On Thu, Oct 21, 2021 at 08:46:10AM -1000, Tejun Heo wrote:
quoted
bpf_types.h has BPF_MAP_TYPE_INODE_STORAGE and BPF_MAP_TYPE_TASK_STORAGE
declared inside #ifdef CONFIG_NET although they are built regardless of
CONFIG_NET. So, when CONFIG_BPF_SYSCALL && !CONFIG_NET, they are built
without the declarations leading to spurious build failures and not
registered to bpf_map_types making them unavailable.

Fix it by moving the BPF_MAP_TYPE for the two map types outside of
CONFIG_NET.
Acked-by: Martin KaFai Lau <redacted>
btw, this should be the fix tag that has both task and inode storage.
Fixes: a10787e6d58c ("bpf: Enable task local storage for tracing programs")
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help