From: Andrii Nakryiko <andrii.nakryiko@gmail.com> To: Hao Luo <haoluo@google.com> Cc: linux-kernel@vger.kernel.org, bpf@vger.kernel.org, cgroups@vger.kernel.org, netdev@vger.kernel.org, Alexei Starovoitov <ast@kernel.org>, Andrii Nakryiko <andrii@kernel.org>, Daniel Borkmann <daniel@iogearbox.net>, Martin KaFai Lau <martin.lau@linux.dev>, Song Liu <song@kernel.org>, Yonghong Song <yhs@fb.com>, Tejun Heo <tj@kernel.org>, Zefan Li <lizefan.x@bytedance.com>, KP Singh <kpsingh@kernel.org>, Johannes Weiner <hannes@cmpxchg.org>, Michal Hocko <mhocko@kernel.org>, Benjamin Tissoires <benjamin.tissoires@redhat.com>, John Fastabend <john.fastabend@gmail.com>, Michal Koutny <mkoutny@suse.com>, Roman Gushchin <roman.gushchin@linux.dev>, David Rientjes <rientjes@google.com>, Stanislav Fomichev <sdf@google.com>, Shakeel Butt <shakeelb@google.com>, Yosry Ahmed <yosryahmed@google.com> Subject: Re: [PATCH bpf-next v7 5/8] selftests/bpf: Test cgroup_iter. Date: Mon, 8 Aug 2022 17:20:25 -0700 [thread overview] Message-ID: <CAEf4BzarAHuR7mOeHToNiNMc03QnR=74cxt_h5LRymf1U6HevQ@mail.gmail.com> (raw) In-Reply-To: <20220805214821.1058337-6-haoluo@google.com> On Fri, Aug 5, 2022 at 2:49 PM Hao Luo <haoluo@google.com> wrote: > > Add a selftest for cgroup_iter. The selftest creates a mini cgroup tree > of the following structure: > > ROOT (working cgroup) > | > PARENT > / \ > CHILD1 CHILD2 > > and tests the following scenarios: > > - invalid cgroup fd. > - pre-order walk over descendants from PARENT. > - post-order walk over descendants from PARENT. > - walk of ancestors from PARENT. > - walk from PARENT in the default order, which is pre-order. > - process only a single object (i.e. PARENT). > - early termination. > > Acked-by: Yonghong Song <yhs@fb.com> > Signed-off-by: Hao Luo <haoluo@google.com> > --- LGTM. Acked-by: Andrii Nakryiko <andrii@kernel.org> > .../selftests/bpf/prog_tests/cgroup_iter.c | 237 ++++++++++++++++++ > tools/testing/selftests/bpf/progs/bpf_iter.h | 7 + > .../testing/selftests/bpf/progs/cgroup_iter.c | 39 +++ > 3 files changed, 283 insertions(+) > create mode 100644 tools/testing/selftests/bpf/prog_tests/cgroup_iter.c > create mode 100644 tools/testing/selftests/bpf/progs/cgroup_iter.c > [...]
WARNING: multiple messages have this Message-ID (diff)
From: Andrii Nakryiko <andrii.nakryiko@gmail.com> To: Hao Luo <haoluo@google.com> Cc: linux-kernel@vger.kernel.org, bpf@vger.kernel.org, cgroups@vger.kernel.org, netdev@vger.kernel.org, Alexei Starovoitov <ast@kernel.org>, Andrii Nakryiko <andrii@kernel.org>, Daniel Borkmann <daniel@iogearbox.net>, Martin KaFai Lau <martin.lau@linux.dev>, Song Liu <song@kernel.org>, Yonghong Song <yhs@fb.com>, Tejun Heo <tj@kernel.org>, Zefan Li <lizefan.x@bytedance.com>, KP Singh <kpsingh@kernel.org>, Johannes Weiner <hannes@cmpxchg.org>, Michal Hocko <mhocko@kernel.org>, Benjamin Tissoires <benjamin.tissoires@redhat.com>, John Fastabend <john.fastabend@gmail.com>, Michal Koutny <mkoutny@suse.com>, Roman Gushchin <roman.gushchin@linux.dev>, David Rientjes <rientjes@google.com>, Stanislav Fomichev <sdf@google.com>, Shakeel Butt <shakeelb@google.com> Subject: Re: [PATCH bpf-next v7 5/8] selftests/bpf: Test cgroup_iter. Date: Mon, 8 Aug 2022 17:20:25 -0700 [thread overview] Message-ID: <CAEf4BzarAHuR7mOeHToNiNMc03QnR=74cxt_h5LRymf1U6HevQ@mail.gmail.com> (raw) In-Reply-To: <20220805214821.1058337-6-haoluo@google.com> On Fri, Aug 5, 2022 at 2:49 PM Hao Luo <haoluo@google.com> wrote: > > Add a selftest for cgroup_iter. The selftest creates a mini cgroup tree > of the following structure: > > ROOT (working cgroup) > | > PARENT > / \ > CHILD1 CHILD2 > > and tests the following scenarios: > > - invalid cgroup fd. > - pre-order walk over descendants from PARENT. > - post-order walk over descendants from PARENT. > - walk of ancestors from PARENT. > - walk from PARENT in the default order, which is pre-order. > - process only a single object (i.e. PARENT). > - early termination. > > Acked-by: Yonghong Song <yhs@fb.com> > Signed-off-by: Hao Luo <haoluo@google.com> > --- LGTM. Acked-by: Andrii Nakryiko <andrii@kernel.org> > .../selftests/bpf/prog_tests/cgroup_iter.c | 237 ++++++++++++++++++ > tools/testing/selftests/bpf/progs/bpf_iter.h | 7 + > .../testing/selftests/bpf/progs/cgroup_iter.c | 39 +++ > 3 files changed, 283 insertions(+) > create mode 100644 tools/testing/selftests/bpf/prog_tests/cgroup_iter.c > create mode 100644 tools/testing/selftests/bpf/progs/cgroup_iter.c > [...]
next prev parent reply other threads:[~2022-08-09 0:20 UTC|newest] Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-08-05 21:48 [PATCH bpf-next v7 0/8] bpf: rstat: cgroup hierarchical stats Hao Luo 2022-08-05 21:48 ` Hao Luo 2022-08-05 21:48 ` [PATCH bpf-next v7 1/8] btf: Add a new kfunc flag which allows to mark a function to be sleepable Hao Luo 2022-08-05 21:48 ` Hao Luo 2022-08-05 21:48 ` [PATCH bpf-next v7 2/8] cgroup: enable cgroup_get_from_file() on cgroup1 Hao Luo 2022-08-05 21:48 ` [PATCH bpf-next v7 3/8] bpf, iter: Fix the condition on p when calling stop Hao Luo 2022-08-05 21:48 ` [PATCH bpf-next v7 4/8] bpf: Introduce cgroup iter Hao Luo 2022-08-09 0:18 ` Andrii Nakryiko 2022-08-09 0:18 ` Andrii Nakryiko 2022-08-09 0:56 ` Hao Luo 2022-08-09 0:56 ` Hao Luo 2022-08-09 16:23 ` Alexei Starovoitov 2022-08-09 16:23 ` Alexei Starovoitov 2022-08-09 18:38 ` Hao Luo 2022-08-09 18:38 ` Hao Luo 2022-08-11 3:10 ` Hao Luo 2022-08-11 3:10 ` Hao Luo 2022-08-11 14:09 ` Yosry Ahmed 2022-08-11 14:09 ` Yosry Ahmed 2022-08-16 4:12 ` Andrii Nakryiko 2022-08-16 4:12 ` Andrii Nakryiko 2022-08-16 4:19 ` Andrii Nakryiko 2022-08-16 4:19 ` Andrii Nakryiko 2022-08-16 6:52 ` Hao Luo 2022-08-16 6:52 ` Hao Luo 2022-08-16 17:17 ` Andrii Nakryiko 2022-08-16 17:17 ` Andrii Nakryiko 2022-08-16 17:22 ` Hao Luo 2022-08-16 17:22 ` Hao Luo 2022-08-05 21:48 ` [PATCH bpf-next v7 5/8] selftests/bpf: Test cgroup_iter Hao Luo 2022-08-09 0:20 ` Andrii Nakryiko [this message] 2022-08-09 0:20 ` Andrii Nakryiko 2022-08-09 1:18 ` Hao Luo 2022-08-09 1:18 ` Hao Luo 2022-08-05 21:48 ` [PATCH bpf-next v7 6/8] cgroup: bpf: enable bpf programs to integrate with rstat Hao Luo 2022-08-05 21:48 ` Hao Luo 2022-08-05 21:48 ` [PATCH bpf-next v7 7/8] selftests/bpf: extend cgroup helpers Hao Luo 2022-08-05 21:48 ` [PATCH bpf-next v7 8/8] selftests/bpf: add a selftest for cgroup hierarchical stats collection Hao Luo 2022-08-09 16:20 ` [PATCH bpf-next v7 0/8] bpf: rstat: cgroup hierarchical stats patchwork-bot+netdevbpf 2022-08-09 16:20 ` patchwork-bot+netdevbpf-DgEjT+Ai2ygdnm+yROfE0A
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to='CAEf4BzarAHuR7mOeHToNiNMc03QnR=74cxt_h5LRymf1U6HevQ@mail.gmail.com' \ --to=andrii.nakryiko@gmail.com \ --cc=andrii@kernel.org \ --cc=ast@kernel.org \ --cc=benjamin.tissoires@redhat.com \ --cc=bpf@vger.kernel.org \ --cc=cgroups@vger.kernel.org \ --cc=daniel@iogearbox.net \ --cc=hannes@cmpxchg.org \ --cc=haoluo@google.com \ --cc=john.fastabend@gmail.com \ --cc=kpsingh@kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=lizefan.x@bytedance.com \ --cc=martin.lau@linux.dev \ --cc=mhocko@kernel.org \ --cc=mkoutny@suse.com \ --cc=netdev@vger.kernel.org \ --cc=rientjes@google.com \ --cc=roman.gushchin@linux.dev \ --cc=sdf@google.com \ --cc=shakeelb@google.com \ --cc=song@kernel.org \ --cc=tj@kernel.org \ --cc=yhs@fb.com \ --cc=yosryahmed@google.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.