In test_cpucg_nice, after the child process exits, user_usec is read
from cpu.stat but never used. Only nice_usec is validated against the
expected value. Remove this redundant read.

Signed-off-by: Shaojie Sun <[email protected]>
---
 tools/testing/selftests/cgroup/test_cpu.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/tools/testing/selftests/cgroup/test_cpu.c 
b/tools/testing/selftests/cgroup/test_cpu.c
index 7a40d76b9548..fe637e1d2fcd 100644
--- a/tools/testing/selftests/cgroup/test_cpu.c
+++ b/tools/testing/selftests/cgroup/test_cpu.c
@@ -289,7 +289,6 @@ static int test_cpucg_nice(const char *root)
                if (!WIFEXITED(status))
                        goto cleanup;
 
-               user_usec = cg_read_key_long(cpucg, "cpu.stat", "user_usec");
                nice_usec = cg_read_key_long(cpucg, "cpu.stat", "nice_usec");
                if (!values_close_report(nice_usec, expected_nice_usec, 1))
                        goto cleanup;
-- 
2.25.1


Reply via email to