Skip to content

Commit 118999d

Browse files
committed
perf test: Simplify data symbol test
JIRA: https://issues.redhat.com/browse/RHEL-78197 upstream ======== commit b627b44 Author: Namhyung Kim <namhyung@kernel.org> Date: Mon Mar 3 18:28:37 2025 -0800 description =========== Now the workload will end after 1 second. Just run it with perf instead of waiting for the background process. Reviewed-by: Leo Yan <leo.yan@arm.com> Tested-by: Thomas Richter <tmricht@linux.ibm.com> Tested-by: Thomas Falcon <thomas.falcon@intel.com> Cc: Thomas Richter <tmricht@linux.ibm.com> Cc: Leo Yan <leo.yan@arm.com> Link: https://lore.kernel.org/r/20250304022837.1877845-7-namhyung@kernel.org Signed-off-by: Namhyung Kim <namhyung@kernel.org> Signed-off-by: Michael Petlan <mpetlan@redhat.com>
1 parent dc81df0 commit 118999d

File tree

1 file changed

+2
-13
lines changed

1 file changed

+2
-13
lines changed

tools/perf/tests/shell/test_data_symbol.sh

Lines changed: 2 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,6 @@
55
# Leo Yan <leo.yan@linaro.org>, 2022
66

77
shelldir=$(dirname "$0")
8-
# shellcheck source=lib/waiting.sh
9-
. "${shelldir}"/lib/waiting.sh
108

119
# shellcheck source=lib/perf_has_symbol.sh
1210
. "${shelldir}"/lib/perf_has_symbol.sh
@@ -60,19 +58,10 @@ echo "Recording workload..."
6058
# specific CPU and test in per-CPU mode.
6159
is_amd=$(grep -E -c 'vendor_id.*AuthenticAMD' /proc/cpuinfo)
6260
if (($is_amd >= 1)); then
63-
perf mem record -vvv -o ${PERF_DATA} -C 0 -- taskset -c 0 $TEST_PROGRAM 2>"${ERR_FILE}" &
61+
perf mem record -vvv -o ${PERF_DATA} -C 0 -- taskset -c 0 $TEST_PROGRAM 2>"${ERR_FILE}"
6462
else
65-
perf mem record -vvv --all-user -o ${PERF_DATA} -- $TEST_PROGRAM 2>"${ERR_FILE}" &
63+
perf mem record -vvv --all-user -o ${PERF_DATA} -- $TEST_PROGRAM 2>"${ERR_FILE}"
6664
fi
6765

68-
PERFPID=$!
69-
70-
wait_for_perf_to_start ${PERFPID} "${ERR_FILE}"
71-
72-
sleep 1
73-
74-
kill $PERFPID
75-
wait $PERFPID
76-
7766
check_result
7867
exit $?

0 commit comments

Comments
 (0)