在 2026/7/17 19:06, guojian 写道:
> From: Jian Guo <[email protected]>
>
> The test_cgcore_no_internal_process_constraint_on_threads test has two
> back-to-back cg_enter_current(root) calls in its cleanup path.
>
> A single cg_enter_current() call atomically migrates the entire thread
> group to the target cgroup even for multi-threaded processes, and this
> test creates no extra threads or child processes that would require a
> second migration attempt. The second call is a harmless no-op once the
> process is already in the root cgroup, but it is redundant and
> inconsistent with the cleanup logic used in all other cgroup core
> selftest cases.
>
> Remove the duplicate call to clean up the code. No functional change is
> intended.
>
> Signed-off-by: Jian Guo <[email protected]>
> ---
> tools/testing/selftests/cgroup/test_core.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/tools/testing/selftests/cgroup/test_core.c
> b/tools/testing/selftests/cgroup/test_core.c
> index 88ca832d4fc1..8e3f9b391e44 100644
> --- a/tools/testing/selftests/cgroup/test_core.c
> +++ b/tools/testing/selftests/cgroup/test_core.c
> @@ -426,7 +426,6 @@ static int
> test_cgcore_no_internal_process_constraint_on_threads(const char *roo
> ret = KSFT_PASS;
>
> cleanup:
> - cg_enter_current(root);
> cg_enter_current(root);
> if (child)
> cg_destroy(child);
Ran test_core on x86_64 (cgroup v2, all controllers available),
12/12 pass, 0 fail, 0 skip — output byte-identical before and
after the patch. Confirms the removed call is a no-op.
Acked-by: Tao Cui <[email protected]>
Thanks,
Tao
