Commit 53af1a4b authored by Steven Rostedt (Google)'s avatar Steven Rostedt (Google) Committed by Shuah Khan

tracing/selftests: Run the ownership test twice

A regression happened where running the ownership test passes on the first
iteration but fails running it a second time. This was caught and fixed,
but a later change brought it back. The regression was missed because the
automated tests only run the tests once per boot.

Change the ownership test to iterate through the tests twice, as this will
catch the regression with a single run.
Signed-off-by: default avatarSteven Rostedt (Google) <rostedt@goodmis.org>
Acked-by: default avatarMasami Hiramatsu (Google) <mhiramat@kernel.org>
Signed-off-by: default avatarShuah Khan <skhan@linuxfoundation.org>
parent 98f8faea
......@@ -83,32 +83,38 @@ run_tests() {
done
}
mount -o remount,"$new_options" .
# Run the tests twice as leftovers can cause issues
for loop in 1 2 ; do
run_tests
echo "Running iteration $loop"
mount -o remount,"$mount_options" .
mount -o remount,"$new_options" .
for d in "." "events" "events/sched" "events/sched/sched_switch" "events/sched/sched_switch/enable" $canary; do
test "$d" $original_group
done
run_tests
mount -o remount,"$mount_options" .
for d in "." "events" "events/sched" "events/sched/sched_switch" "events/sched/sched_switch/enable" $canary; do
test "$d" $original_group
done
# check instances as well
chgrp $other_group instances
chgrp $other_group instances
instance="$(mktemp -u test-XXXXXX)"
instance="$(mktemp -u test-XXXXXX)"
mkdir instances/$instance
mkdir instances/$instance
cd instances/$instance
cd instances/$instance
run_tests
run_tests
cd ../..
cd ../..
rmdir instances/$instance
rmdir instances/$instance
chgrp $original_group instances
chgrp $original_group instances
done
exit 0
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment