From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760509AbbBIN0E (ORCPT ); Mon, 9 Feb 2015 08:26:04 -0500 Received: from mail-we0-f170.google.com ([74.125.82.170]:56629 "EHLO mail-we0-f170.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1760249AbbBIN0C (ORCPT ); Mon, 9 Feb 2015 08:26:02 -0500 Date: Mon, 9 Feb 2015 14:25:57 +0100 From: Ingo Molnar To: Jiri Olsa Cc: Peter Zijlstra , Andi Kleen , Tom Huynh , benh@kernel.crashing.org, paulus@samba.org, mpe@ellerman.id.au, mingo@redhat.com, acme@kernel.org, Kim.Phillips@freescale.com, linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/3] perf/e6500: Make event translations available in sysfs Message-ID: <20150209132557.GA8099@gmail.com> References: <1423262636-12053-1-git-send-email-tom.huynh@freescale.com> <1423262636-12053-2-git-send-email-tom.huynh@freescale.com> <20150209100242.GM23123@twins.programming.kicks-ass.net> <20150209100738.GA5461@gmail.com> <20150209121132.GB3952@krava.redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150209121132.GB3952@krava.redhat.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org * Jiri Olsa wrote: > On Mon, Feb 09, 2015 at 11:07:38AM +0100, Ingo Molnar wrote: > > > > * Peter Zijlstra wrote: > > > > > On Fri, Feb 06, 2015 at 04:43:54PM -0600, Tom Huynh wrote: > > > > arch/powerpc/perf/e6500-events-list.h | 289 ++++++++++++++++++++++++++ > > > > > > That's a lot of events to stuff in the kernel, would a > > > userspace list not be more convenient? > > > > > > ISTR there being various discussions on providing support > > > for that in tools/perf, Jiri? > > > > As long as it's in a single well organized place in tools/, > > I'd be fine with that solution as well. > > > > What doesn't work very well is disjunct, disorganized, > > inconsistent event descriptions all across the tooling and > > platform landscape - putting static tables into sysfs is a > > marked improvement over that, despite its memory usage. > > the last version is in here: > http://marc.info/?l=linux-kernel&m=140676269017820&w=2 > > AFAIK Andi is setting up the download area as discussed > in the thread and should repost at some point 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. 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... Thanks, Ingo