Messages in this thread | | | Subject | Re: [PATCH 09/13] perf tools: Add perf download to download event files | From | Michael Ellerman <> | Date | Wed, 06 Aug 2014 11:09:00 +1000 |
| |
On Thu, 2014-07-17 at 12:47 +0200, Ingo Molnar wrote: > this bit needs more discussion: > > * Jiri Olsa <jolsa@kernel.org> wrote: > > > From: Andi Kleen <ak@linux.intel.com> > > > > 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
| |