Re: [PATCH v4 4/4] perf test amd ibs: Add sample period unit test

[Arnaldo Carvalho de Melo <acme@kernel.org>]

On Wed, Apr 30, 2025 at 02:32:12PM +0530, Ravi Bangoria wrote:
> On 30-Apr-25 6:52 AM, Arnaldo Carvalho de Melo wrote:
> > On Tue, Apr 29, 2025 at 10:13:53PM -0300, Arnaldo Carvalho de Melo wrote:
> >> Adding the patch below cures it, still need to test on a Zen 5 system.

> >> These issues were just in the regression test.
 
> > BTW, all is at the tmp.perf-tools-next branch at:
 
> > https://git.kernel.org/pub/scm/linux/kernel/git/perf/perf-tools-next.git

> I tested with few simple perf mem/c2c commands and it seems to be working
> fine.

Thanks for checking!

While testing it I noticed that locally built kernels using O= to
separate the build files from source are ending up with:

root@number:/home/acme/git/linux# readelf -wi ../build/v6.15.0-rc4+/vmlinux | grep -m1 DW_AT_comp_dir
    <17>   DW_AT_comp_dir    : (indirect line string, offset: 0): /home/acme/git/build/v6.15.0-rc4+
root@number:/home/acme/git/linux# readelf -wi ../build/v6.15.0-rc4+/vmlinux | grep DW_AT_comp_dir | cut -d: -f4 | sort | uniq -c
     49  /home/acme/git/build/v6.15.0-rc2+
   3104  /home/acme/git/build/v6.15.0-rc4+
root@number:/home/acme/git/linux#

I reused a previous build dir, something got reused, but then tools like
annotate, objdump -dS can't find the sources.

A distraction while testing your patches :-\

- Arnaldo