perf tests: Add daemon 'stop' command test
authorJiri Olsa <jolsa@kernel.org>
Mon, 8 Feb 2021 20:09:05 +0000 (21:09 +0100)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Thu, 11 Feb 2021 13:19:53 +0000 (10:19 -0300)
Add a test for the perf daemon 'stop' command. The test stops the daemon
and verifies all the configured sessions are properly terminated.

Committer testing:

  [root@five ~]# time perf test daemon
  76: daemon operations                                               : Ok
  [root@five ~]# time perf test -v daemon
  76: daemon operations                                               :
  --- start ---
  test child forked, pid 788560
  test daemon list
  test daemon reconfig
  test daemon stop
  test child finished with 0
  ---- end ----
  daemon operations: Ok
  #

Signed-off-by: Jiri Olsa <jolsa@kernel.org>
Tested-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Alexei Budankov <abudankov@huawei.com>
Cc: Ian Rogers <irogers@google.com>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Michael Petlan <mpetlan@redhat.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: https://lore.kernel.org/r/20210208200908.1019149-22-jolsa@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/tests/shell/daemon.sh

index 90a6e8c..3b6b5aa 100755 (executable)
@@ -295,9 +295,63 @@ EOF
        rm -f ${config_new}
        rm -f ${config_empty}
 }
+
+test_stop()
+{
+       echo "test daemon stop"
+
+       local config=$(mktemp /tmp/perf.daemon.config.XXX)
+       local base=$(mktemp -d /tmp/perf.daemon.base.XXX)
+
+       # prepare config
+       cat <<EOF > ${config}
+[daemon]
+base=BASE
+
+[session-size]
+run = -e cpu-clock
+
+[session-time]
+run = -e task-clock
+EOF
+
+       sed -i -e "s|BASE|${base}|" ${config}
+
+       # start daemon
+       daemon_start ${config} size
+
+       local pid_size=`perf daemon --config ${config} -x: | head -2 | tail -1 | awk 'BEGIN { FS = ":" } ; { print $1 }'`
+       local pid_time=`perf daemon --config ${config} -x: | head -3 | tail -1 | awk 'BEGIN { FS = ":" } ; { print $1 }'`
+
+       # check that sessions are running
+       if [ ! -d "/proc/${pid_size}" ]; then
+               echo "FAILED: session size not up"
+       fi
+
+       if [ ! -d "/proc/${pid_time}" ]; then
+               echo "FAILED: session time not up"
+       fi
+
+       # stop daemon
+       daemon_exit ${base} ${config}
+
+       # check that sessions are gone
+       if [ -d "/proc/${pid_size}" ]; then
+               echo "FAILED: session size still up"
+       fi
+
+       if [ -d "/proc/${pid_time}" ]; then
+               echo "FAILED: session time still up"
+       fi
+
+       rm -rf ${base}
+       rm -f ${config}
+}
+
 error=0
 
 test_list
 test_reconfig
+test_stop
 
 exit ${error}