Re: [PATCH 24/26] perf tools: Add buildid-list --store option

From: Jiri Olsa
Date: Mon Sep 14 2020 - 16:43:02 EST


On Mon, Sep 14, 2020 at 12:14:13PM -0300, Arnaldo Carvalho de Melo wrote:
> Em Mon, Sep 14, 2020 at 03:42:55PM +0900, Namhyung Kim escreveu:
> > On Mon, Sep 14, 2020 at 6:05 AM Jiri Olsa <jolsa@xxxxxxxxxx> wrote:
> > >
> > > Adding buildid-list --store option to populate
> > > .debug data with build id files.
> >
> > Hmm.. isn't it better to add it to the buildid-cache command?
>
> Yes, that is the right place. 'buildid-list' is about perf.data files,
> buildid-cache is about .debug cache.

I saw it in buildid-list, because it works over perf.data
by default and --store option made it obvious for me

but I guess we could have the same, like:

$ perf buildid-cache --store[=path]

with path being perf.data by default

thanks,
jirka