Subject: Re: [RFC][PATCH 10/13] perf: Per PMU disable
From: Peter Zijlstra <peterz@infradead.org>
To: Paul Mackerras <paulus@samba.org>
Cc: stephane eranian <eranian@googlemail.com>,
       Robert Richter <robert.richter@amd.com>,
       Will Deacon <will.deacon@arm.com>, Paul Mundt <lethal@linux-sh.org>,
       Frederic Weisbecker <fweisbec@gmail.com>,
       Cyrill Gorcunov <gorcunov@gmail.com>, Lin Ming <ming.m.lin@intel.com>,
       Yanmin <yanmin_zhang@linux.intel.com>,
       Deng-Cheng Zhu <dengcheng.zhu@gmail.com>,
       David Miller <davem@davemloft.net>, Ingo Molnar <mingo@elte.hu>,
       linux-kernel@vger.kernel.org
In-Reply-To: <1278671458.1900.193.camel@laptop>
References: <20100709082117.631541128@chello.nl>
	 <20100709083248.156796575@chello.nl>
	 <20100709101703.GR7771@brick.ozlabs.ibm.com>
	 <1278671458.1900.193.camel@laptop>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 8BIT
Date: Fri, 09 Jul 2010 12:39:52 +0200
Message-ID: <1278671992.1900.197.camel@laptop>
Mime-Version: 1.0
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 994
Lines: 26

On Fri, 2010-07-09 at 12:30 +0200, Peter Zijlstra wrote:
> On Fri, 2010-07-09 at 20:17 +1000, Paul Mackerras wrote:
> > On Fri, Jul 09, 2010 at 10:21:27AM +0200, Peter Zijlstra wrote:
> > 
> > > -void hw_perf_disable(void)
> > > +static void powerpc_pmu_pmu_disable(struct pmu *pmu)
> > 
> > > -void hw_perf_enable(void)
> > > +static void powerpc_pmu_pmu_enable(struct pmu *pmu)
> > 
> > >  struct pmu power_pmu = {
> > > -	.event_init	= power_pmu_event_init,
> > > +	.pmu_enable	= power_pmu_pmu_enable,
> > > +	.pmu_disable	= power_pmu_pmu_disable,
> > > +	.event_init	= pmwer_pmu_event_init,
> > 
> > Doh!  Think you posted the old version of the patch again...
> 
> *sigh*

The hunks ended up in the last patch.. I've pushed out a new git tree.
--
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/