Re: [PATCH v2] perf test: Run test 126 exclusive

[Howard Chu]

Hi Namhyung and Thomas,

On Wed, Apr 8, 2026 at 7:31 PM Namhyung Kim <namhyung@xxxxxxxxxx> wrote:
>
> On Wed, Apr 08, 2026 at 01:31:43PM +0200, Thomas Richter wrote:
> > V1 --> V2 Fixed commit message
> >
> > Running both tests cases 126 128 together causes the first test case
> > 126 to fail:
> >  # for i in $(seq 3); do ./perf test 126 128; done
> >  126: perf trace BTF general tests    : FAILED!
> >  128: perf trace record and replay    : Ok
> >  126: perf trace BTF general tests    : FAILED!
> >  128: perf trace record and replay    : Ok
> >  126: perf trace BTF general tests    : FAILED!
> >  128: perf trace record and replay    : Ok
> >  #
>
> The test numbers can be different on other platforms, let's use the test
> names like below.
>
> >
> > Test case 126 fails because test case 128 runs concurrently as can
> > be observed using a ps -ef | grep perf output list on a different
> > window. Both do a perf trace command concurrently.
> > Make test case 'perf trace BTF general tests' exclusive.
> >
> > Output after:
> >  # for i in $(seq 3); do ./perf test 'perf trace BTF general tests' \
> >       'perf trace record and replay'; done
> >  127: perf trace BTF general tests                   : Ok
> >  155: perf trace record and replay                   : Ok
> >  127: perf trace BTF general tests                   : Ok
> >  155: perf trace record and replay                   : Ok
> >  127: perf trace BTF general tests                   : Ok
> >  155: perf trace record and replay                   : Ok
> >  #
>
> I believe I have a fix for this problem but it's blocked by a BPF
> verifier issue.  I need to find time to work on it again..
>
> https://lore.kernel.org/linux-perf-users/20250814071754.193265-1-namhyung@xxxxxxxxxx/
>
> Until then, I think it's ok to have this.

Agreed.

Thanks,
Howard

>
> Thanks,
> Namhyung
>
> >
> > Signed-off-by: Thomas Richter <tmricht@xxxxxxxxxxxxx>
> > Cc: Howard Chu <howardchu95@xxxxxxxxx>

Acked-by: Howard Chu <howardchu95@xxxxxxxxx>

Thanks,
Howard

> > ---
> >  tools/perf/tests/shell/trace_btf_general.sh | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/tools/perf/tests/shell/trace_btf_general.sh b/tools/perf/tests/shell/trace_btf_general.sh
> > index ef2da806be6b..7a94a5743924 100755
> > --- a/tools/perf/tests/shell/trace_btf_general.sh
> > +++ b/tools/perf/tests/shell/trace_btf_general.sh
> > @@ -1,5 +1,5 @@
> >  #!/bin/bash
> > -# perf trace BTF general tests
> > +# perf trace BTF general tests (exclusive)
> >  # SPDX-License-Identifier: GPL-2.0
> >
> >  err=0
> > --
> > 2.53.0
> >