On Wed, Feb 22, 2023 at 01:20:19PM -0800, Alexei Starovoitov wrote: > 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. Will do. Thanks. -- tejun
