From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-ea0-f177.google.com (mail-ea0-f177.google.com [209.85.215.177]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority" (not verified)) by ozlabs.org (Postfix) with ESMTPS id CF8272C0293 for ; Fri, 1 Feb 2013 21:26:35 +1100 (EST) Received: by mail-ea0-f177.google.com with SMTP id n13so1627764eaa.8 for ; Fri, 01 Feb 2013 02:26:32 -0800 (PST) Sender: Ingo Molnar Date: Fri, 1 Feb 2013 11:26:29 +0100 From: Ingo Molnar To: Arnaldo Carvalho de Melo , Stephane Eranian Subject: Re: [PATCH 22/25] perf: Make EVENT_ATTR global Message-ID: <20130201102629.GA24485@gmail.com> References: <1359653128-10433-1-git-send-email-acme@infradead.org> <1359653128-10433-23-git-send-email-acme@infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1359653128-10433-23-git-send-email-acme@infradead.org> Cc: Andi Kleen , Peter Zijlstra , Robert Richter , Anton Blanchard , linux-kernel@vger.kernel.org, Stephane Eranian , Arnaldo Carvalho de Melo , linuxppc-dev@ozlabs.org, Ingo Molnar , Paul Mackerras , Sukadev Bhattiprolu , Jiri Olsa List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , * Arnaldo Carvalho de Melo wrote: > From: Sukadev Bhattiprolu > > Rename EVENT_ATTR() to PMU_EVENT_ATTR() and make it global so it is > available to all architectures. > > Further to allow architectures flexibility, have PMU_EVENT_ATTR() pass > in the variable name as a parameter. > > Changelog[v2] > - [Jiri Olsa] No need to define PMU_EVENT_PTR() > > Signed-off-by: Sukadev Bhattiprolu > Acked-by: Jiri Olsa > Cc: Andi Kleen > Cc: Anton Blanchard > Cc: Ingo Molnar > Cc: Jiri Olsa > Cc: Paul Mackerras > Cc: Peter Zijlstra > Cc: Robert Richter > Cc: Stephane Eranian > Cc: linuxppc-dev@ozlabs.org > Link: http://lkml.kernel.org/r/20130123062422.GC13720@us.ibm.com > Signed-off-by: Arnaldo Carvalho de Melo > --- > arch/x86/kernel/cpu/perf_event.c | 13 +++---------- > include/linux/perf_event.h | 11 +++++++++++ > 2 files changed, 14 insertions(+), 10 deletions(-) so this one started conflicting non-trivially with tip:perf/x86 - the pending memory profiling kernel-side bits. Can we merge the memory profiling tooling side bits together with the kernel side bits - or does it need more work? For now I've excluded perf/x86 from tip:master until this is resolved. Thanks, Ingo