diff options
| author | Shaojie Sun <sunshaojie@kylinos.cn> | 2026-08-14 18:00:02 +0800 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2026-08-14 08:32:58 -1000 |
| commit | 2d19207f3fc8e08bab3270af2e3835358ab1473a (patch) | |
| tree | e4aa52f9415b4e00df922f1675f911c488c25287 /tools/perf/scripts/python/stackcollapse.py | |
| parent | ddabc5dbd262f3ca981e679ddae99ed7a9adb279 (diff) | |
selftests/cgroup: Remove redundant chown in test_cgcore_lesser_ns_open
test_cgcore_lesser_ns_open runs as root throughout and never changes its
euid, so chowning the two cgroup.procs files to a non-root uid has no
effect on the test.
The ENOENT the test expects comes from the cgroup namespace delegation
check in cgroup_procs_write_permission(): the source and destination
cgroups must both be descendants of the namespace root captured at open
time. That check does not depend on file ownership. In addition, the
permission check only examines the common ancestor's cgroup.procs file
(the test root here), which the chown calls do not touch.
Remove the redundant chown calls and the now unused test_euid and
cg_test_a_procs variables.
Signed-off-by: Shaojie Sun <sunshaojie@kylinos.cn>
Reviewed-by: Tao Cui <cuitao@kylinos.cn>
Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'tools/perf/scripts/python/stackcollapse.py')
0 files changed, 0 insertions, 0 deletions
