linux-kernel - Re: [PATCH v4 0/2] perf trace: Add more tests for BTF-augmented perf trace

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <Z2m8qzyLdm1BEYuF@x1>
Date: Mon, 23 Dec 2024 16:40:27 -0300
From: Arnaldo Carvalho de Melo <acme@...nel.org>
To: Namhyung Kim <namhyung@...nel.org>
Cc: Howard Chu <howardchu95@...il.com>, mark.rutland@....com,
	alexander.shishkin@...ux.intel.com, jolsa@...nel.org,
	irogers@...gle.com, adrian.hunter@...el.com,
	kan.liang@...ux.intel.com, linux-perf-users@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v4 0/2] perf trace: Add more tests for BTF-augmented perf
 trace

On Mon, Dec 23, 2024 at 04:02:43PM -0300, Arnaldo Carvalho de Melo wrote:
> On Mon, Dec 16, 2024 at 11:58:46AM -0800, Namhyung Kim wrote:
> > On Sun, Dec 15, 2024 at 11:07:09AM -0800, Howard Chu wrote:
> > > Changes in v4:
> > >  - Use if -f to check the existence of vmlinux BTF, and exit if it
> > >    doesn't, so trace_test_string will not overwrite $err, and keep
> > >    running the test.
> > > 
> > > Changes in v3:
> > >  - Add vmlinux BTF check, and skip the tests if it doesn't exist
> > > 
> > > v1, v2:
> > > 
> > > The previous version of the perf trace BTF general augmentation tests
> > > didn't pass Shellcheck (thanks to Arnaldo Carvalho de Melo
> > > <acme@...nel.org> for pointing this out), this version uses bash instead
> > > of POSIX shell to pass Shellcheck.
> > > 
> > > This patch series also adds documentation for the new option
> > > --force-btf, which is used in the tests.
> > > 
> > > Link: https://lore.kernel.org/linux-perf-users/Zt9yiQq-n-W6I274@x1/
> > > 
> > > Howard Chu (2):
> > >   perf trace: Add tests for BTF general augmentation
> > >   perf docs: Add documentation for --force-btf option
> > 
> > Acked-by: Namhyung Kim <namhyung@...nel.org>
> 
> Thanks, applied to perf-tools-next,

It fails when running in parallel mode, sometimes:

109: perf trace BTF general tests                                    : FAILED!

Then:

root@...ber:~# perf stat --null -r 10 perf test "BTF general"
109: perf trace BTF general tests                                    : Ok
109: perf trace BTF general tests                                    : Ok
109: perf trace BTF general tests                                    : Ok
109: perf trace BTF general tests                                    : Ok
109: perf trace BTF general tests                                    : FAILED!
109: perf trace BTF general tests                                    : Ok
109: perf trace BTF general tests                                    : Ok
109: perf trace BTF general tests                                    : FAILED!
109: perf trace BTF general tests                                    : Ok
109: perf trace BTF general tests                                    : Ok

 Performance counter stats for 'perf test BTF general' (10 runs):

             2.148 +- 0.293 seconds time elapsed  ( +- 13.63% )

root@...ber:~#

So its not just when running in paralell, anyway, its merged, we can go
on from what we got there.

- Arnaldo