summaryrefslogtreecommitdiff
path: root/tools/testing/selftests/ftrace/test.d/instances/instance.tc
diff options
context:
space:
mode:
authorSteven Rostedt <rostedt@goodmis.org>2015-12-11 14:36:17 -0500
committerSteven Rostedt <rostedt@goodmis.org>2016-02-19 12:11:21 -0500
commit9a154c8911e39e32bdbc2cd6c9b08a64e17612aa (patch)
treec48c7dffe2277b2225f27f5fe6c2ddd71ea78b02 /tools/testing/selftests/ftrace/test.d/instances/instance.tc
parentb33c8ff4431a343561e2319f17c14286f2aa52e2 (diff)
downloadlinux-sh-9a154c8911e39e32bdbc2cd6c9b08a64e17612aa.tar.gz
ftracetest: Fix instance test to use proper shell command for pids
The ftracetest instance test used parsing of the "jobs" output to find the pid of the subshell that is executed previously. But this is not portable to all major shells that may run these tests. The proper way to get the pid of the subshell is the shell command "$!". This will return the pid of the previously executed command. Use that instead, otherwise the test does not work in all environments. Link: http://lkml.kernel.org/r/20151211143617.65f4d7a1@gandalf.local.home Reported-by: Michael Ellerman <mpe@ellerman.id.au> Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'tools/testing/selftests/ftrace/test.d/instances/instance.tc')
-rw-r--r--tools/testing/selftests/ftrace/test.d/instances/instance.tc15
1 files changed, 5 insertions, 10 deletions
diff --git a/tools/testing/selftests/ftrace/test.d/instances/instance.tc b/tools/testing/selftests/ftrace/test.d/instances/instance.tc
index 773e276ff90b..1e1abe0ad354 100644
--- a/tools/testing/selftests/ftrace/test.d/instances/instance.tc
+++ b/tools/testing/selftests/ftrace/test.d/instances/instance.tc
@@ -39,28 +39,23 @@ instance_slam() {
}
instance_slam &
-x=`jobs -l`
-p1=`echo $x | cut -d' ' -f2`
+p1=$!
echo $p1
instance_slam &
-x=`jobs -l | tail -1`
-p2=`echo $x | cut -d' ' -f2`
+p2=$!
echo $p2
instance_slam &
-x=`jobs -l | tail -1`
-p3=`echo $x | cut -d' ' -f2`
+p3=$!
echo $p3
instance_slam &
-x=`jobs -l | tail -1`
-p4=`echo $x | cut -d' ' -f2`
+p4=$!
echo $p4
instance_slam &
-x=`jobs -l | tail -1`
-p5=`echo $x | cut -d' ' -f2`
+p5=$!
echo $p5
ls -lR >/dev/null