Hello, On Tue, May 10, 2022 at 12:17:59AM +0000, Yosry Ahmed wrote: > @@ -706,6 +707,9 @@ struct cgroup_subsys { > * specifies the mask of subsystems that this one depends on. > */ > unsigned int depends_on; > + > + /* used to store bpf programs.*/ > + struct cgroup_subsys_bpf bpf; > }; Care to elaborate on rationales around associating this with a specific cgroup_subsys rather than letting it walk cgroups and access whatever csses as needed? I don't think it's a wrong approach or anything but I can think of plenty of things that would be interesting without being associated with a specific subsystem - even all the cpu usage statistics are built to in the cgroup core and given how e.g. systemd uses cgroup to organize the applications in the system whether resource control is active or not, there are a lot of info one can gather about those without being associated with a specific subsystem. Thanks. -- tejun