On Tue, Feb 21, 2023 at 2:09 PM Tejun Heo <tj@xxxxxxxxxx> wrote: > > cgroup ID is an userspace-visible 64bit value uniquely identifying a given > cgroup. As the IDs are used widely, it's useful to be able to look up the > matching cgroups. Add bpf_cgroup_from_id(). > > Signed-off-by: Tejun Heo <tj@xxxxxxxxxx> > --- > Documentation/bpf/kfuncs.rst | 10 +++-- > kernel/bpf/helpers.c | 18 ++++++++ > .../selftests/bpf/prog_tests/cgrp_kfunc.c | 1 + > .../selftests/bpf/progs/cgrp_kfunc_common.h | 1 + > .../selftests/bpf/progs/cgrp_kfunc_success.c | 42 +++++++++++++++++++ > 5 files changed, 69 insertions(+), 3 deletions(-) Looks great, but could you please split it into two patches: One for helpers.c and kfuncs.rst and another for selftests with [PATCH bpf-next] selftests/bpf: subject ? Doesn't matter much in this case, but it might ease backporting and easier to track patches grouped by subj.
