Re: [PATCH 09/13] perf tools: Add perf download to download event files

From: Michael Ellerman
Date: Tue Aug 05 2014 - 21:09:08 EST


On Thu, 2014-07-17 at 12:47 +0200, Ingo Molnar wrote:
> this bit needs more discussion:
>
> * Jiri Olsa <jolsa@xxxxxxxxxx> wrote:
>
> > From: Andi Kleen <ak@xxxxxxxxxxxxxxx>
> >
> > Add a downloader to automatically download the right
> > files from a download site.
>
> 'A download site' is way too vague and detached from the perf project.
>
> > + echo "Downloading models file"
> > + URLBASE=${URLBASE:-https://download.01.org/perfmon}
>
> We want these description files to be in the perf source code,
> somewhere in tools/perf/live-config/arch/x86/ or so, and installed
> during 'make install' - i.e. part of perf project and installed in
> ~/.debug or ~/.perf or so.

Surely the shipped versions should go in /usr/share/perf or something like
that? Rather than a particular users home directory?

> Those files could be refreshed via 'perf download' and could be
> accessible via kernel.org as well, 'perf download' should pick up
> these files from Linus's latest git repository (via the HTTP
> namespace).

This version would go in ~/.perf or similar.

cheers


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/