summaryrefslogtreecommitdiff
path: root/tools/perf/tests
diff options
context:
space:
mode:
authorIan Rogers <irogers@google.com>2026-09-25 23:19:57 -0700
committerArnaldo Carvalho de Melo <acme@redhat.com>2026-09-26 20:49:29 +0200
commitb103b3338b99c3edfe982ef1e6198db8aaa039b6 (patch)
treef2a756e4f9402230fb722e45eb65cc586f2e61a3 /tools/perf/tests
parent605fd1c8b7e6677428cd8395b8191d0846e18adf (diff)
downloadlinux-next-b103b3338b99c3edfe982ef1e6198db8aaa039b6.tar.gz
linux-next-b103b3338b99c3edfe982ef1e6198db8aaa039b6.zip
perf python: Port futex-contention to perf module
Port tools/perf/scripts/python/futex-contention.py to a standalone script in tools/perf/python/ using the perf module. Avoiding the embedded interpreter overhead improves execution speed by ~3.2x: ``` $ perf record -e syscalls:sys_*_futex -a sleep 1 ... $ time perf script tools/perf/scripts/python/futex-contention.py ... real 0m1.007s user 0m0.935s sys 0m0.072s $ time python3 tools/perf/python/futex-contention.py ... real 0m0.314s user 0m0.259s sys 0m0.056s ``` Additional improvements compared to the legacy script: - Consolidate per-(tid, uaddr) contention count, total_time, min_time, and max_time into a single LockStats class instead of maintaining three separate dictionaries. - Validate that uaddr and op tracepoint attributes are present on syscalls:sys_enter_futex samples rather than silently attributing missing fields to uaddr=0, op=0 (FUTEX_WAIT). - Add -i/--input CLI option support via argparse and full type annotations. Add a shell test (test_futex_contention_python.sh) to verify the standalone script. Assisted-by: Antigravity:gemini-3.1-pro Signed-off-by: Ian Rogers <irogers@google.com> Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/tests')
-rwxr-xr-xtools/perf/tests/shell/test_futex_contention_python.sh115
1 files changed, 115 insertions, 0 deletions
diff --git a/tools/perf/tests/shell/test_futex_contention_python.sh b/tools/perf/tests/shell/test_futex_contention_python.sh
new file mode 100755
index 000000000000..26b13d263d20
--- /dev/null
+++ b/tools/perf/tests/shell/test_futex_contention_python.sh
@@ -0,0 +1,115 @@
+#!/bin/bash
+# SPDX-License-Identifier: GPL-2.0
+# futex-contention python test
+
+set -e
+
+shelldir=$(dirname "$0")
+# shellcheck source=lib/setup_python.sh
+. "${shelldir}"/lib/setup_python.sh
+
+# If we don't have the perf python module, we can't test
+if ! "$PYTHON" -c 'import perf' > /dev/null 2>&1; then
+ echo "Skipping test, perf python module not found"
+ exit 2
+fi
+
+script_dir="$(dirname "$0")/../../python"
+script_path="${script_dir}/futex-contention.py"
+
+if ! perf check feature -q libtraceevent > /dev/null 2>&1; then
+ echo "Skipping test, libtraceevent is disabled"
+ exit 2
+fi
+
+if [ ! -f "$script_path" ]; then
+ echo "Skipping test, futex-contention.py not found at $script_path"
+ exit 2
+fi
+
+err=0
+temp_data=""
+temp_out=""
+
+cleanup() {
+ rm -f "${temp_data}" "${temp_out}"
+}
+
+trap 'cleanup' EXIT TERM INT
+
+temp_data=$(mktemp /tmp/perf.data.XXXXXX)
+temp_out=$(mktemp /tmp/perf.out.XXXXXX)
+
+test_synthetic() {
+ echo "Testing futex-contention.py synthetic events..."
+ if ! "$PYTHON" - "$script_path" << 'EOF'
+import importlib.util
+import sys
+from types import SimpleNamespace
+
+spec = importlib.util.spec_from_file_location("futex_contention", sys.argv[1])
+mod = importlib.util.module_from_spec(spec)
+spec.loader.exec_module(mod)
+
+enter_sample = SimpleNamespace(
+ evsel="evsel(syscalls:sys_enter_futex:k)",
+ sample_pid=1234,
+ sample_tid=1234,
+ uaddr=0xdeadbeef,
+ op=mod.FUTEX_WAIT | mod.FUTEX_PRIVATE_FLAG,
+ sample_time=1000,
+)
+exit_sample = SimpleNamespace(
+ evsel="evsel(syscalls:sys_exit_futex:k)",
+ sample_pid=1234,
+ sample_tid=1234,
+ sample_time=2500,
+)
+
+mod.process_event(enter_sample)
+mod.process_event(exit_sample)
+
+stats = mod.durations.get((1234, 0xdeadbeef))
+assert stats is not None and stats.count == 1, f"Unexpected stats: {stats}"
+assert stats.min_time == 1500 and stats.max_time == 1500 and stats.avg() == 1500.0
+line = (f"{mod.process_names.get(1234, 'unknown')}[1234] lock {0xdeadbeef:x} "
+ f"contended {stats.count} times, {stats.avg():.0f} avg ns "
+ f"[max: {stats.max_time} ns, min {stats.min_time} ns]")
+expected = "unknown[1234] lock deadbeef contended 1 times, 1500 avg ns [max: 1500 ns, min 1500 ns]"
+assert line == expected, line
+EOF
+ then
+ echo "Synthetic test failed."
+ err=1
+ else
+ echo "Synthetic test passed."
+ fi
+}
+
+test_file_mode() {
+ echo "Testing futex-contention.py..."
+ # Some systems might not have syscalls:sys_enter_futex
+ if ! perf list | grep -q syscalls:sys_enter_futex; then
+ echo "Skipping file mode test, syscalls:sys_enter_futex not found"
+ return
+ fi
+
+ # Generate some futex events
+ if ! perf record -e syscalls:sys_enter_futex,syscalls:sys_exit_futex -a -o "${temp_data}" \
+ -- sleep 0.5 2>/dev/null; then
+ echo "Skipping file mode test (record failed)"
+ return
+ fi
+
+ if ! "$PYTHON" "$script_path" -i "${temp_data}" > "${temp_out}"; then
+ echo "File mode test failed."
+ err=1
+ else
+ echo "File mode test passed."
+ fi
+}
+
+test_synthetic
+test_file_mode
+
+exit $err