The cpufreq selftests are always executed through main.sh, which verifies
that the test is run as root before dispatching any test case.

Therefore, invoking sudo inside quick_shuffle() is redundant and may cause
failures in environments where sudo is unavailable.

Signed-off-by: Jinseok Kim <[email protected]>
---
 tools/testing/selftests/cpufreq/special-tests.sh | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/testing/selftests/cpufreq/special-tests.sh 
b/tools/testing/selftests/cpufreq/special-tests.sh
index 8d40505dc468..f45eb525f3b1 100755
--- a/tools/testing/selftests/cpufreq/special-tests.sh
+++ b/tools/testing/selftests/cpufreq/special-tests.sh
@@ -65,8 +65,8 @@ quick_shuffle()
        # this is called concurrently from governor_race
        for I in `seq 1000`
        do
-               echo ondemand | sudo tee $CPUFREQROOT/policy*/scaling_governor &
-               echo userspace | sudo tee $CPUFREQROOT/policy*/scaling_governor 
&
+               echo ondemand | tee $CPUFREQROOT/policy*/scaling_governor &
+               echo userspace | tee $CPUFREQROOT/policy*/scaling_governor &
        done
 }

--
2.43.0

Reply via email to