[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20150209204019.GA13991@two.firstfloor.org>
Date: Mon, 9 Feb 2015 21:40:19 +0100
From: Andi Kleen <andi@...stfloor.org>
To: Ingo Molnar <mingo@...nel.org>
Cc: Jiri Olsa <jolsa@...hat.com>,
Peter Zijlstra <peterz@...radead.org>,
Andi Kleen <andi@...stfloor.org>,
Tom Huynh <tom.huynh@...escale.com>, benh@...nel.crashing.org,
paulus@...ba.org, mpe@...erman.id.au, mingo@...hat.com,
acme@...nel.org, Kim.Phillips@...escale.com,
linuxppc-dev@...ts.ozlabs.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/3] perf/e6500: Make event translations available in
sysfs
> I'll NAK any external 'download area' (and I told that Andi
> before): tools/perf/event-tables/ or so is a good enough
> 'download area' with fast enough update cycles.
The proposal was to put it on kernel.org, similar to how
external firmware blobs are distributed. CPU event lists
are data sheets, so are like firmware. They do not
follow the normal kernel code licenses. They are not
source code. They cannot be reviewed in the normal way.
> If any 'update' of event descriptions is needed it can
> happen through the distro package mechanism, or via a
> simple 'git pull' if it's compiled directly.
>
> Lets not overengineer this with any dependence on an
> external site and with a separate update mechanism - lets
> just get the tables into tools/ and see it from there...
That experiment has been already done for oprofile,
didn't work very well.
-Andi
--
ak@...ux.intel.com -- Speaking for myself only.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists