perf test: Make perf trace enum augmentation tests exclusive

[PATCH] perf test: Make perf trace enum augmentation tests exclusive

Posted by Thomas Richter 2 months ago

Running both tests cases 'perf trace enum augmentation tests'
and 'perf trace BTF general tests' together causes the first test case
to fail:
 # for i in $(seq 5); do \
	./perf test 'perf trace enum augmentation tests' \
	'perf trace BTF general tests'; \
   done
 125: perf trace enum augmentation tests  : Ok
 126: perf trace BTF general tests        : FAILED!
 125: perf trace enum augmentation tests  : Ok
 126: perf trace BTF general tests        : FAILED!
 125: perf trace enum augmentation tests  : Ok
 126: perf trace BTF general tests        : FAILED!
 125: perf trace enum augmentation tests  : Ok
 126: perf trace BTF general tests        : FAILED!
 125: perf trace enum augmentation tests  : Ok
 126: perf trace BTF general tests        : FAILED!
 #

Test case perf trace enum augmentation tests' fails because  the second
test case 'perf trace BTF general tests' 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 enum augmentation tests' exclusive.

 # for i in $(seq 5); do \
	./perf test 'perf trace enum augmentation tests' \
	'perf trace BTF general tests'; \
   done
 125: perf trace BTF general tests        : Ok
 155: perf trace enum augmentation tests  : Ok
 125: perf trace BTF general tests        : Ok
 155: perf trace enum augmentation tests  : Ok
 125: perf trace BTF general tests        : Ok
 155: perf trace enum augmentation tests  : Ok
 125: perf trace BTF general tests        : Ok
 155: perf trace enum augmentation tests  : Ok
 125: perf trace BTF general tests        : Ok
 155: perf trace enum augmentation tests  : Ok
 #

Signed-off-by: Thomas Richter <tmricht@linux.ibm.com>
Cc: Howard Chu <howardchu95@gmail.com>
---
 tools/perf/tests/shell/trace_btf_enum.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/perf/tests/shell/trace_btf_enum.sh b/tools/perf/tests/shell/trace_btf_enum.sh
index 03e9f680a4a6..437510d46f58 100755
--- a/tools/perf/tests/shell/trace_btf_enum.sh
+++ b/tools/perf/tests/shell/trace_btf_enum.sh
@@ -1,5 +1,5 @@
 #!/bin/bash
-# perf trace enum augmentation tests
+# perf trace enum augmentation tests (exclusive)
 # SPDX-License-Identifier: GPL-2.0
 
 err=0
-- 
2.53.0

Re: [PATCH] perf test: Make perf trace enum augmentation tests exclusive

Posted by Howard Chu 2 months ago

Hi Thomas,

On Thu, Apr 9, 2026 at 1:12 AM Thomas Richter <tmricht@linux.ibm.com> wrote:
>
> Running both tests cases 'perf trace enum augmentation tests'
> and 'perf trace BTF general tests' together causes the first test case
> to fail:
>  # for i in $(seq 5); do \
>         ./perf test 'perf trace enum augmentation tests' \
>         'perf trace BTF general tests'; \
>    done
>  125: perf trace enum augmentation tests  : Ok
>  126: perf trace BTF general tests        : FAILED!
>  125: perf trace enum augmentation tests  : Ok
>  126: perf trace BTF general tests        : FAILED!
>  125: perf trace enum augmentation tests  : Ok
>  126: perf trace BTF general tests        : FAILED!
>  125: perf trace enum augmentation tests  : Ok
>  126: perf trace BTF general tests        : FAILED!
>  125: perf trace enum augmentation tests  : Ok
>  126: perf trace BTF general tests        : FAILED!
>  #
>
> Test case perf trace enum augmentation tests' fails because  the second
> test case 'perf trace BTF general tests' 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 enum augmentation tests' exclusive.
>
>  # for i in $(seq 5); do \
>         ./perf test 'perf trace enum augmentation tests' \
>         'perf trace BTF general tests'; \
>    done
>  125: perf trace BTF general tests        : Ok
>  155: perf trace enum augmentation tests  : Ok
>  125: perf trace BTF general tests        : Ok
>  155: perf trace enum augmentation tests  : Ok
>  125: perf trace BTF general tests        : Ok
>  155: perf trace enum augmentation tests  : Ok
>  125: perf trace BTF general tests        : Ok
>  155: perf trace enum augmentation tests  : Ok
>  125: perf trace BTF general tests        : Ok
>  155: perf trace enum augmentation tests  : Ok
>  #
>
> 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_enum.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/perf/tests/shell/trace_btf_enum.sh b/tools/perf/tests/shell/trace_btf_enum.sh
> index 03e9f680a4a6..437510d46f58 100755
> --- a/tools/perf/tests/shell/trace_btf_enum.sh
> +++ b/tools/perf/tests/shell/trace_btf_enum.sh
> @@ -1,5 +1,5 @@
>  #!/bin/bash
> -# perf trace enum augmentation tests
> +# perf trace enum augmentation tests (exclusive)
>  # SPDX-License-Identifier: GPL-2.0
>
>  err=0
> --
> 2.53.0
>