From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754918Ab0F2SVc (ORCPT ); Tue, 29 Jun 2010 14:21:32 -0400 Received: from mail-wy0-f174.google.com ([74.125.82.174]:51750 "EHLO mail-wy0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754084Ab0F2SVb (ORCPT ); Tue, 29 Jun 2010 14:21:31 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=date:from:to:cc:subject:message-id:references:mime-version :content-type:content-disposition:in-reply-to:user-agent; b=mKU3puAZUG6E9gZza3zNzEVuNcXT3uMuL86EYmNs11wX8CoukGk9RejQ/7RH3K+PXw c5FNRsBjEa4gjA3Agqx/fg4Nnscj0vHBQCZ/TnwvxP4nua1Ps097icWvwf0+6MvPsfPR qu9YEh3zdYf8ut8jUPigMLxINtR0X7ZIaSyLc= Date: Tue, 29 Jun 2010 20:21:41 +0200 From: Frederic Weisbecker To: Peter Zijlstra Cc: paulus , stephane eranian , Robert Richter , Will Deacon , Paul Mundt , Cyrill Gorcunov , Lin Ming , Yanmin , Deng-Cheng Zhu , David Miller , linux-kernel@vger.kernel.org Subject: Re: [RFC][PATCH 09/11] perf: Default PMU ops Message-ID: <20100629182139.GD11967@nowhere> References: <20100624142804.431553874@chello.nl> <20100624143406.993794468@chello.nl> <20100629145806.GE5318@nowhere> <1277823591.1868.43.camel@laptop> <20100629150305.GG5318@nowhere> <1277829250.1868.46.camel@laptop> <20100629180745.GA11967@nowhere> <1277834950.1868.56.camel@laptop> <20100629181104.GC11967@nowhere> <1277835543.1868.57.camel@laptop> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1277835543.1868.57.camel@laptop> User-Agent: Mutt/1.5.18 (2008-05-17) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jun 29, 2010 at 08:19:03PM +0200, Peter Zijlstra wrote: > On Tue, 2010-06-29 at 20:11 +0200, Frederic Weisbecker wrote: > > On Tue, Jun 29, 2010 at 08:09:10PM +0200, Peter Zijlstra wrote: > > > On Tue, 2010-06-29 at 20:07 +0200, Frederic Weisbecker wrote: > > > > On Tue, Jun 29, 2010 at 06:34:10PM +0200, Peter Zijlstra wrote: > > > > > On Tue, 2010-06-29 at 17:03 +0200, Frederic Weisbecker wrote: > > > > > > > You mean, if (!pmu->start_txn && pmu->pmu_enable) { /* install defaults > > > > > > > */ } ? > > > > > > > neglected pmu implementations shouldn't impact software pmus. > > > > > > With the above and not providing ->pmu_enable you get that. > > > > > > But we don't need to provide a pmu_enable for software events, right? > > My point exactly. Which I understand late. Agreed.