On Fri, Jul 17, 2026 at 07:06:06PM +0800, guojian <[email protected]> wrote: > 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(-)
Fixes: d863cb03fc5f9 ("Add cgroup core selftests")
Good catch.
Acked-by: Michal Koutný <[email protected]>
signature.asc
Description: PGP signature
