Re: [PATCH v2 0/6] perf test: Small improvements

[Falcon, Thomas]

On Mon, 2025-03-03 at 18:28 -0800, Namhyung Kim wrote:
> Hello,
> 
> This is an assorted list of changes in the perf test.
> 
> I've added perf stat --metric-only test and perf trace record and
> replay.
> And other changes to skip privileged tests for non-root and simplify
> data symbol test.
> 
> v2 changes)
> * fix shellcheck errors
> * simplify datasym test with timeout
> 
> Thanks,
> Namhyung
> 

Looks good on Alder Lake and Arrow Lake.

Tested-by: Thomas Falcon <thomas.falcon@xxxxxxxxx>

> 
> Namhyung Kim (6):
>   perf test: Add --metric-only to perf stat output tests
>   perf test: Skip perf probe tests when running as non-root
>   perf test: Skip perf trace tests when running as non-root
>   perf test: Add trace record and replay test
>   perf test: Add timeout to datasym workload
>   perf test: Simplify data symbol test
> 
>  .../tests/shell/lib/perf_json_output_lint.py  |  7 ++++++
>  tools/perf/tests/shell/lib/stat_output.sh     |  8 +++++++
>  .../tests/shell/perftool-testsuite_probe.sh   |  1 +
>  tools/perf/tests/shell/probe_vfs_getname.sh   |  1 +
>  .../shell/record+probe_libc_inet_pton.sh      |  1 +
>  .../shell/record+script_probe_vfs_getname.sh  |  1 +
>  tools/perf/tests/shell/stat+csv_output.sh     |  2 ++
>  tools/perf/tests/shell/stat+json_output.sh    |  9 ++++++++
>  tools/perf/tests/shell/stat+std_output.sh     |  8 +++++++
>  tools/perf/tests/shell/test_data_symbol.sh    | 15 ++----------
>  .../shell/test_uprobe_from_different_cu.sh    | 11 ++++-----
>  .../tests/shell/trace+probe_vfs_getname.sh    |  1 +
>  tools/perf/tests/shell/trace_btf_enum.sh      |  1 +
>  tools/perf/tests/shell/trace_btf_general.sh   |  1 +
>  tools/perf/tests/shell/trace_exit_race.sh     |  1 +
>  tools/perf/tests/shell/trace_record_replay.sh | 21 +++++++++++++++++
>  tools/perf/tests/workloads/datasym.c          | 23
> +++++++++++++++++--
>  17 files changed, 91 insertions(+), 21 deletions(-)
>  create mode 100755 tools/perf/tests/shell/trace_record_replay.sh
>