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

From: Martin KaFai Lau
Date: Tue Oct 26 2021 - 15:25:48 EST


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:
> > 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 <kafai@xxxxxx>

btw, this should be the fix tag that has both task and inode storage.
Fixes: a10787e6d58c ("bpf: Enable task local storage for tracing programs")