On Wed, Dec 6, 2023 at 10:47 AM Alexei Starovoitov <alexei.starovoitov@xxxxxxxxx> wrote: > > On Tue, Dec 5, 2023 at 9:15 AM Tejun Heo <tj@xxxxxxxxxx> wrote: > > > > On Tue, Dec 05, 2023 at 02:37:22PM +0000, Yafang Shao wrote: > > > In the current cgroup1 environment, associating operations between a cgroup > > > and applications in a BPF program requires storing a mapping of cgroup_id > > > to application either in a hash map or maintaining it in userspace. > > > However, by enabling bpf_cgrp_storage for cgroup1, it becomes possible to > > > conveniently store application-specific information in cgroup-local storage > > > and utilize it within BPF programs. Furthermore, enabling this feature for > > > cgroup1 involves minor modifications for the non-attach case, streamlining > > > the process. > > > > > > However, when it comes to enabling this functionality for the cgroup1 > > > attach case, it presents challenges. Therefore, the decision is to focus on > > > enabling it solely for the cgroup1 non-attach case at present. If > > > attempting to attach to a cgroup1 fd, the operation will simply fail with > > > the error code -EBADF. > > > > > > Yafang Shao (3): > > > bpf: Enable bpf_cgrp_storage for cgroup1 non-attach case > > > selftests/bpf: Add a new cgroup helper open_classid() > > > selftests/bpf: Add selftests for cgroup1 local storage > > > > Acked-by: Tejun Heo <tj@xxxxxxxxxx> > > > Yafang, > please resubmit without RFC tag, so it can get tested by BPF CI. will do it. -- Regards Yafang
