Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752794AbbHZQvu (ORCPT ); Wed, 26 Aug 2015 12:51:50 -0400 Received: from foss.arm.com ([217.140.101.70]:46048 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751818AbbHZQvt (ORCPT ); Wed, 26 Aug 2015 12:51:49 -0400 Date: Wed, 26 Aug 2015 17:51:49 +0100 From: Will Deacon To: Drew Richardson Cc: Peter Zijlstra , Ingo Molnar , Arnaldo Carvalho de Melo , Russell King , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , Wade Cherry Subject: Re: [PATCH] arm: perf: Add event descriptions Message-ID: <20150826165148.GF30466@arm.com> References: <20150804001522.GA13214@dreric01-gentoo.localdomain> <20150817205809.GA16506@dreric01-gentoo.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150817205809.GA16506@dreric01-gentoo.localdomain> 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 Content-Length: 1241 Lines: 27 On Mon, Aug 17, 2015 at 09:58:09PM +0100, Drew Richardson wrote: > On Tue, Aug 04, 2015 at 01:15:23AM +0100, Drew Richardson wrote: > > Add additional information about hardware events to make counters self > > describing. This makes the hardware PMUs easier to use as perf list > > contains the possible events instead of users having to refer to > > documentation like the ARM TRMs. This could also allow tools like > > oprofile to support PMUs without requiring an update. > > > > Signed-off-by: Drew Richardson > > --- > > arch/arm/kernel/perf_event.c | 1 + > > arch/arm/kernel/perf_event_v7.c | 617 ++++++++++++++++++++++++++++++++++++++++ > > 2 files changed, 618 insertions(+) > > Any concerns with merging this? We should revisit this after the merge window (since the ARM perf code is moving out to drivers/), but I do have reservations about putting all these strings into the kernel. Why can't we put this into something like libpfm instead? Will -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/