1. Patchew
  2. linux
  3. View series

[PATCH v2] perf test: Run test 126 exclusive

Thomas Richter posted 1 patch 2 months, 1 week ago 
tools/perf/tests/shell/trace_btf_general.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH v2] perf test: Run test 126 exclusive
Posted by Thomas Richter 2 months, 1 week ago 
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
 #

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
 #

Signed-off-by: Thomas Richter <tmricht@linux.ibm.com>
Cc: Howard Chu <howardchu95@gmail.com>
---
 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
Re: [PATCH v2] perf test: Run test 126 exclusive
Posted by Namhyung Kim 2 months ago 
On Wed, 08 Apr 2026 13:31:43 +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
>  #
> 
> [...]
Applied to perf-tools-next, thanks!

Best regards,
Namhyung
Re: [PATCH v2] perf test: Run test 126 exclusive
Posted by Namhyung Kim 2 months, 1 week ago 
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@kernel.org/

Until then, I think it's ok to have this.

Thanks,
Namhyung

> 
> Signed-off-by: Thomas Richter <tmricht@linux.ibm.com>
> Cc: Howard Chu <howardchu95@gmail.com>
> ---
>  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
>
Re: [PATCH v2] perf test: Run test 126 exclusive
Posted by Howard Chu 2 months, 1 week ago 
Hi Namhyung and Thomas,

On Wed, Apr 8, 2026 at 7:31 PM Namhyung Kim <namhyung@kernel.org> 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@kernel.org/
>
> Until then, I think it's ok to have this.

Agreed.

Thanks,
Howard

>
> Thanks,
> Namhyung
>
> >
> > Signed-off-by: Thomas Richter <tmricht@linux.ibm.com>
> > Cc: Howard Chu <howardchu95@gmail.com>

Acked-by: Howard Chu <howardchu95@gmail.com>

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
> >

Patchew 2.1-devel-ea86937

© 2016 - 2026 Red Hat, Inc.