Re: [PATCH] perf test: Skip the extra metric line for stalled_cycles_per_instruction

[sashiko-bot@kernel.org]

> The STD output linter maps one metric to each default event and stops at
> the first line that shows a different one.  stalled_cycles_per_instruction
> is computed as max(stalled-cycles-frontend, stalled-cycles-backend) /
> instructions, so its event set differs from the one behind
> backend_cycles_idle and perf stat prints it on a line of its own: a
> second stalled-cycles-backend line carrying that metric.  The linter
> rejects it with "wrong event metric. expected backend_cycles_idle in
> ...".
> 
> Metrics printed on their own line are already skipped
> (skip_metric=("tma_" "TopdownL1")); add this one to that list.
> 
> Verified on an Ampere Altra (arm64, Neoverse-N1): `perf test "perf stat
> STD output linter"` fails before the change and passes after it.  Only
> the linter is changed; the perf stat output is untouched.
> 
> Fixes: c7adeb0974f1 ("perf jevents: Add set of common metrics based on default ones")
> Assisted-by: LLM
> Signed-off-by: TAKANO Mitsuhiro <takano32@gmail.com>

Sashiko has reviewed this patch and found no issues.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261022024950.264235-1-takano32@gmail.com?part=1