Re: [PATCH 0/6 v1] perf test attr: Fix the re-enabled test

[Ian Rogers]

On Wed, Oct 7, 2026 at 1:31 AM Arnaldo Carvalho de Melo
<arnaldo.melo@xxxxxxxxx> wrote:
>
> On Tue, Oct 06, 2026 at 05:00:26PM -0700, Namhyung Kim wrote:
> > On Tue, Oct 06, 2026 at 06:13:03PM +0200, Arnaldo Carvalho de Melo wrote:
> > > From: Arnaldo Carvalho de Melo <acme@xxxxxxxxxx>
>  > The test passes with 'perf test -vv attribute' on x86_64.  The AMD
> > > encodings come from the amd_pmu_event_map tables in
> > > arch/x86/events/amd/core.c.
> >
> > On my machine, the current tmp.perf-tools-next which contains these
> > changes fails like this.
> >
> >   $ tools/perf/perf test -v 82
> >   ---- start ----
> >   test child forked, pid 931585
> >   /home/namhyung/project/linux/tools/perf/tests/shell/attr.sh: line 20: python: command not found
> >   ---- end(-1) ----
> >    82: Perf attribute expectations test                                                 : FAILED!
> >
> >   === Test Summary ===
> >   Passed main tests : 0
> >   Passed subtests   : 0
> >   Skipped tests     : 0
> >   Failed tests      : 1
> >   List of failed tests:
> >      82: Perf attribute expectations test
> >
> > The below patch fixes the problem.
>
> still without your patch:
>
> root@number:~# perf test attribute
>  82: Perf attribute expectations t: Ok
>
>  === Test Summary ===
>  Passed main tests : 1
>  Passed subtests   : 0
>  Skipped tests     : 0
>  Failed tests      : 0
>  root@number:~# grep "model name" -m1 /proc/cpuinfo
>  model name      : AMD Ryzen 9 9950X3D 16-Core Processor
>  root@number:~# uname -a
>  Linux number 7.2.7-200.fc44.x86_64 #1 SMP PREEMPT_DYNAMIC Mon Sep 21
>  18:25:57 UTC 2026 x86_64 GNU/Linux
>  root@number:~#
>
>  I reproduced the build failure in an ARM64 Debian Trixie machine,
>  applying your patch to try again.

For the change to use PYTHON:
Reviewed-by: Ian Rogers <irogers@xxxxxxxxxx>
I think we can fix the attribute test expectations separately.

Thanks,
Ian

>  - Arnaldo