diff options
| author | Guopeng Zhang <zhangguopeng@kylinos.cn> | 2026-05-20 17:31:30 +0800 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2026-05-22 06:20:43 -1000 |
| commit | 98f0adb2284a0a4599a40337a30bef429167bb7b (patch) | |
| tree | f60d3fe09a993edb2c7fce6bf721c1e60b53cb35 | |
| parent | 0d25e3865841ea5edfedb5af42bf15cef075192e (diff) | |
| download | linux-next-98f0adb2284a0a4599a40337a30bef429167bb7b.tar.gz linux-next-98f0adb2284a0a4599a40337a30bef429167bb7b.zip | |
selftests/cgroup: enable memory controller in hugetlb memcg test
test_hugetlb_memcg creates a child cgroup and then writes memory.max and
memory.swap.max. When the test is run standalone, the memory controller
may not be enabled in the test root cgroup's subtree_control.
In that case, the child cgroup is created without the memory control
files, and the test fails during setup before reaching the hugetlb memcg
accounting checks.
Skip the test when the memory controller is unavailable. Otherwise, enable
it in subtree_control before creating the test cgroup.
Signed-off-by: Guopeng Zhang <zhangguopeng@kylinos.cn>
Acked-by: Michal Koutný <mkoutny@suse.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
| -rw-r--r-- | tools/testing/selftests/cgroup/test_hugetlb_memcg.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/tools/testing/selftests/cgroup/test_hugetlb_memcg.c b/tools/testing/selftests/cgroup/test_hugetlb_memcg.c index f451aa449be6..b627d84358b1 100644 --- a/tools/testing/selftests/cgroup/test_hugetlb_memcg.c +++ b/tools/testing/selftests/cgroup/test_hugetlb_memcg.c @@ -217,6 +217,14 @@ int main(int argc, char **argv) if (cg_find_unified_root(root, sizeof(root), NULL)) ksft_exit_skip("cgroup v2 isn't mounted\n"); + if (cg_read_strstr(root, "cgroup.controllers", "memory")) + ksft_exit_skip("memory controller isn't available\n"); + + if (cg_read_strstr(root, "cgroup.subtree_control", "memory")) { + if (cg_write(root, "cgroup.subtree_control", "+memory")) + ksft_exit_skip("Failed to set memory controller\n"); + } + switch (test_hugetlb_memcg(root)) { case KSFT_PASS: ksft_test_result_pass("test_hugetlb_memcg\n"); |