path: root/tools/testing/selftests
diff options
authorSteven Rostedt (VMware) <rostedt@goodmis.org>2020-01-30 21:45:27 -0500
committerShuah Khan <skhan@linuxfoundation.org>2020-02-10 17:59:44 -0700
commita098d9c82a0bb2f91e5cf6c780859bc00c15e1e4 (patch)
tree8cf256e6f22652da739c9e2be2ff94115c603b21 /tools/testing/selftests
parentselftests: fix spelling mistaked "chaigned" -> "chained" (diff)
selftests/ftrace: Have pid filter test use instance flag
While running the ftracetests, the pid filter test failed because the instance "foo" existed, and it was using it to rerun the test under a instance named foo. The collision caused the test to fail as the mkdir failed as the name already existed. As of commit b5b77be812de7 ("selftests: ftrace: Allow some tests to be run in a tracing instance") all a selftest needs to do to be tested in an instance is to set the "instance" flag. There's no reason a selftest needs to create an instance to run its test in an instance directly. Remove the open coded testing in an instance for the pid filter test and have it set the "instance" flag instead. Signed-off-by: Steven Rostedt (VMware) <rostedt@goodmis.org> Acked-by: Masami Hiramatsu <mhiramat@kernel.org> Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
Diffstat (limited to 'tools/testing/selftests')
1 files changed, 1 insertions, 7 deletions
diff --git a/tools/testing/selftests/ftrace/test.d/ftrace/func-filter-pid.tc b/tools/testing/selftests/ftrace/test.d/ftrace/func-filter-pid.tc
index 64cfcc75e3c1..f2ee1e889e13 100644
--- a/tools/testing/selftests/ftrace/test.d/ftrace/func-filter-pid.tc
+++ b/tools/testing/selftests/ftrace/test.d/ftrace/func-filter-pid.tc
@@ -1,6 +1,7 @@
# SPDX-License-Identifier: GPL-2.0
# description: ftrace - function pid filters
+# flags: instance
# Make sure that function pid matching filter works.
# Also test it on an instance directory
@@ -96,13 +97,6 @@ do_test() {
-mkdir instances/foo
-cd instances/foo
-cd ../../
-rmdir instances/foo
exit 0