From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wi0-x232.google.com (mail-wi0-x232.google.com [IPv6:2a00:1450:400c:c05::232]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id A8C0C1A17F6 for ; Thu, 28 May 2015 22:10:02 +1000 (AEST) Received: by wicmx19 with SMTP id mx19so144085783wic.0 for ; Thu, 28 May 2015 05:09:59 -0700 (PDT) Sender: Ingo Molnar Date: Thu, 28 May 2015 14:09:54 +0200 From: Ingo Molnar To: Jiri Olsa Cc: Namhyung Kim , Andi Kleen , Sukadev Bhattiprolu , Ingo Molnar , Michael Ellerman , Arnaldo Carvalho de Melo , Paul Mackerras , linuxppc-dev@lists.ozlabs.org, "linux-kernel@vger.kernel.org" Subject: Re: [PATCH 2/4] perf: jevents: Program to convert JSON file to C style file Message-ID: <20150528120954.GA8478@gmail.com> References: <1432080130-6678-1-git-send-email-sukadev@linux.vnet.ibm.com> <1432080130-6678-3-git-send-email-sukadev@linux.vnet.ibm.com> <20150527135402.GA29557@danjae.kornet> <20150527144007.GL7484@tassilo.jf.intel.com> <20150528115223.GD12392@krava.redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20150528115223.GD12392@krava.redhat.com> List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , * Jiri Olsa wrote: > On Wed, May 27, 2015 at 11:59:04PM +0900, Namhyung Kim wrote: > > Hi Andi, > > > > On Wed, May 27, 2015 at 11:40 PM, Andi Kleen wrote: > > >> So we build tables of all models in the architecture, and choose > > >> matching one when compiling perf, right? Can't we do that when > > >> building the tables? IOW, why don't we check the VFM and discard > > >> non-matching tables? Those non-matching tables are also needed? > > > > > > We build it for all cpus in an architecture, not all architectures. > > > So e.g. for an x86 binary power is not included, and vice versa. > > > > OK. > > > > > It always includes all CPUs for a given architecture, so it's possible > > > to use the perf binary on other systems than just the one it was > > > build on. > > > > So it selects one at run-time not build-time, good. But I worry about > > the size of the intel tables. How large are they? Maybe we can make > > it dynamic-loadable if needed.. > > just compiled Sukadev's new version with Andi's events list > and stripped binary size is: > > [jolsa@krava perf]$ ls -l perf > -rwxrwxr-x 1 jolsa jolsa 2772640 May 28 13:49 perf > > > while perf on Arnaldo's perf/core is: > > [jolsa@krava perf]$ ls -l perf > -rwxrwxr-x 1 jolsa jolsa 2334816 May 28 13:49 perf > > seems not that bad It's not bad at all. Do you have a Git tree URI where I could take a look at its current state? A tree would be nice that has as many of these patches integrated as possible. Thanks, Ingo