[PATCH linux-next] perf tests: Handle s390 metrics in perf all metrics test

[PATCH linux-next] perf tests: Handle s390 metrics in perf all metrics test

[Thomas Richter]

This test case fails on s390 because some counters metrics
are not available and the metric defined on them fail.
Add s390x specific list of metrics to be skipped.

Add an extra line to display which metric has been skipped.
It is shown with the verbose option turned on.

Also return "Ok" instead of "Skipped" when no errors
have been detected, but some metric have been skipped.
The seems more logical otherwise "Skipped" means all
metric tests have been skipped instead of just a few.

Signed-off-by: Thomas Richter <tmricht@linux.ibm.com>
---
 tools/perf/tests/shell/stat_all_metrics.sh | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/tools/perf/tests/shell/stat_all_metrics.sh b/tools/perf/tests/shell/stat_all_metrics.sh
index a7edf01b3943..b86b36a49228 100755
--- a/tools/perf/tests/shell/stat_all_metrics.sh
+++ b/tools/perf/tests/shell/stat_all_metrics.sh
@@ -7,6 +7,10 @@ ParanoidAndNotRoot()
   [ "$(id -u)" != 0 ] && [ "$(cat /proc/sys/kernel/perf_event_paranoid)" -gt $1 ]
 }
 
+# Ignore metric which are not supported on s390x
+[ "$(uname -m)" = "s390x" ] && ignore="|branch_miss_rate|l1d_miss_rate|llc_miss_rate|\
+		dtlb_miss_rate|itlb_miss_rate|l1i_miss_rate|l1_prefetch_miss_rate"
+
 test_prog="sleep 0.01"
 system_wide_flag="-a"
 if ParanoidAndNotRoot 0
@@ -27,9 +31,10 @@ for m in $(perf list --raw-dump metrics); do
   fi
   if [[ "$result" =~ "Cannot resolve IDs for" || "$result" =~ "No supported events found" ]]
   then
-    if [[ "$m" == @(l1_prefetch_miss_rate|stalled_cycles_per_instruction) ]]
+    if [[ "$m" == @(l1_prefetch_miss_rate|stalled_cycles_per_instruction$ignore) ]]
     then
       # Default metrics that may use unsupported events.
+      echo "Skipped metric $m"
       continue
     fi
     echo "Metric contains missing events"
@@ -106,4 +111,5 @@ for m in $(perf list --raw-dump metrics); do
   err=1
 done
 
+[ "$err" -eq 2 ] && err=0
 exit "$err"
-- 
2.51.1

Re: [PATCH linux-next] perf tests: Handle s390 metrics in perf all metrics test

[Ian Rogers]

On Wed, Nov 19, 2025 at 2:48 AM Thomas Richter <tmricht@linux.ibm.com> wrote:
>
> This test case fails on s390 because some counters metrics
> are not available and the metric defined on them fail.
> Add s390x specific list of metrics to be skipped.
>
> Add an extra line to display which metric has been skipped.
> It is shown with the verbose option turned on.
>
> Also return "Ok" instead of "Skipped" when no errors
> have been detected, but some metric have been skipped.
> The seems more logical otherwise "Skipped" means all
> metric tests have been skipped instead of just a few.
>
> Signed-off-by: Thomas Richter <tmricht@linux.ibm.com>

Thanks Thomas, I believe the intent with these metrics is they don't
fail but report unsupported for their counters. There's been some
discussion here:
https://lore.kernel.org/lkml/aRi9xnwdLh3Dir9f@google.com/
with Namhyung and James pointing out the issue on AMD and ARM, and me
being confused we Intel isn't failing similarly for events like itlb
accesses that Intel doesn't support. I'll try to find time to dig into
the issue.

Thanks,
Ian

> ---
>  tools/perf/tests/shell/stat_all_metrics.sh | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/tools/perf/tests/shell/stat_all_metrics.sh b/tools/perf/tests/shell/stat_all_metrics.sh
> index a7edf01b3943..b86b36a49228 100755
> --- a/tools/perf/tests/shell/stat_all_metrics.sh
> +++ b/tools/perf/tests/shell/stat_all_metrics.sh
> @@ -7,6 +7,10 @@ ParanoidAndNotRoot()
>    [ "$(id -u)" != 0 ] && [ "$(cat /proc/sys/kernel/perf_event_paranoid)" -gt $1 ]
>  }
>
> +# Ignore metric which are not supported on s390x
> +[ "$(uname -m)" = "s390x" ] && ignore="|branch_miss_rate|l1d_miss_rate|llc_miss_rate|\
> +               dtlb_miss_rate|itlb_miss_rate|l1i_miss_rate|l1_prefetch_miss_rate"
> +
>  test_prog="sleep 0.01"
>  system_wide_flag="-a"
>  if ParanoidAndNotRoot 0
> @@ -27,9 +31,10 @@ for m in $(perf list --raw-dump metrics); do
>    fi
>    if [[ "$result" =~ "Cannot resolve IDs for" || "$result" =~ "No supported events found" ]]
>    then
> -    if [[ "$m" == @(l1_prefetch_miss_rate|stalled_cycles_per_instruction) ]]
> +    if [[ "$m" == @(l1_prefetch_miss_rate|stalled_cycles_per_instruction$ignore) ]]
>      then
>        # Default metrics that may use unsupported events.
> +      echo "Skipped metric $m"
>        continue
>      fi
>      echo "Metric contains missing events"
> @@ -106,4 +111,5 @@ for m in $(perf list --raw-dump metrics); do
>    err=1
>  done
>
> +[ "$err" -eq 2 ] && err=0
>  exit "$err"
> --
> 2.51.1
>