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

[Howard Chu]

Hi Thomas,

On Thu, Apr 9, 2026 at 1:12 AM Thomas Richter <tmricht@xxxxxxxxxxxxx> 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@xxxxxxxxxxxxx>
> Cc: Howard Chu <howardchu95@xxxxxxxxx>

Acked-by: Howard Chu <howardchu95@xxxxxxxxx>

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
>