Re: [RFC PATCH 6/6] perf cs-etm: Add warnings for missing DSOs

From: Arnaldo Carvalho de Melo
Date: Fri Aug 06 2021 - 14:36:42 EST


Em Thu, Aug 05, 2021 at 01:59:49PM +0100, James Clark escreveu:
>
>
> On 03/08/2021 15:01, Leo Yan wrote:
> >> one can also use debuginfod-client, which, as time passes, probably will
> >> be the main way of finding DSOs now that we have build-ids in
> >> PERF_RECORD_MMAP2 and debuginfod servers such as:
> >>
> >> export DEBUGINFOD_URLS=https://debuginfod.fedoraproject.org/
> > If so, maybe should use more general description for missing DSO.
> >
> >> https://fedoraproject.org/wiki/Debuginfod
> >>
> >>> With this improvement, the patch looks good to me:
> >>>
> >>> Reviewed-by: Leo Yan <leo.yan@xxxxxxxxxx>
> >> Does this apply to the other 5 patches in this series?
> > I finished to reivew patches 01, 04, 05, 06/06, so my review tag can
> > apply on these patches. Current patch 06/06 needs James to improve for
> > the comments.
>
> Thanks for the reviews, I've submitted v2.
>
> I also had a play around with debuginfod, it looks promising.
> Especially if it's integrated with perf report, which I assume is the plan?

Yeah, code is needed for that in the underlying symbols library so that
we can tune its usage, i.e. the threshold of samples to trigger asking
for the auto-download of debuginfo files.

- Arnaldo