Re: [PATCH 2/6] perf annotate-data: Collect global variables in advance

From: Namhyung Kim
Date: Thu May 02 2024 - 14:23:39 EST


On Thu, May 2, 2024 at 6:50 AM Arnaldo Carvalho de Melo <acme@xxxxxxxxxx> wrote:
>
> On Wed, May 01, 2024 at 11:00:07PM -0700, Namhyung Kim wrote:
> > Currently it looks up global variables from the current CU using address
> > and name. But it sometimes fails to find a variable as the variable can
> > come from a different CU - but it's still strange it failed to find a
> > declaration for some reason.
> >
> > Anyway, it can collect all global variables from all CU once and then
> > lookup them later on. This slightly improves the success rate of my
> > test data set.
>
> It would be interesting you could provide examples from your test data
> set, i.e. after this patch these extra global variables were considered
> in the test results, with some tool output, etc.
>
> This would help intersested parties to reproduce your results,
> validate the end result, etc.

Hmm.. ok. I'll try to find public data that I can share.
But it's just a perf data file recorded with perf mem.

Thanks,
Namhyung