Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751356AbaBZI35 (ORCPT ); Wed, 26 Feb 2014 03:29:57 -0500 Received: from merlin.infradead.org ([205.233.59.134]:37558 "EHLO merlin.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751233AbaBZI34 (ORCPT ); Wed, 26 Feb 2014 03:29:56 -0500 Date: Wed, 26 Feb 2014 09:29:43 +0100 From: Peter Zijlstra To: Cody P Schafer Cc: Michael Ellerman , Linux PPC , Arnaldo Carvalho de Melo , Ingo Molnar , Paul Mackerras , LKML Subject: Re: [PATCH v2 02/11] perf core: export swevent hrtimer helpers Message-ID: <20140226082943.GC18404@twins.programming.kicks-ass.net> References: <1392415338-16288-3-git-send-email-cody@linux.vnet.ibm.com> <20140225033326.7BB942C0228@ozlabs.org> <20140225102008.GI9987@twins.programming.kicks-ass.net> <530D0D57.4030704@linux.vnet.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <530D0D57.4030704@linux.vnet.ibm.com> User-Agent: Mutt/1.5.21 (2012-12-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Feb 25, 2014 at 01:38:31PM -0800, Cody P Schafer wrote: > On 02/25/2014 02:20 AM, Peter Zijlstra wrote: > >On Tue, Feb 25, 2014 at 02:33:26PM +1100, Michael Ellerman wrote: > >>On Fri, 2014-14-02 at 22:02:06 UTC, Cody P Schafer wrote: > >>>Export the swevent hrtimer helpers currently only used in events/core.c > >>>to allow the addition of architecture specific sw-like pmus. > >> > >>Peter, Ingo, can we get your ACK on this please? > > > >How are they used? I saw some usage in patch 9 or so; but its not > >explained anywhere. All patches have non-existent Changelogs and the few > >comments that are there are pretty hardware specific. > > > >So please do tell; what do you need this for? > > From this patch's change log: > > >Export the swevent hrtimer helpers currently only used in events/core.c to allow the addition of architecture specific sw-like pmus. > > The key part here is "architecture specific sw-like pmus", where the > announcement explains why these pmus are sw-like: I don't read announcements for crucial patch details; announcements are lost and therefore unimportant. > >The counters supplied by these interfaces are continually counting and never > >need to be (and cannot be) disabled or enabled. They additionally do not > >generate any interrupts. This makes them in some regards similar to software > >counters, and as a result their implimentation shares some common code (which > >an initial patch exposes) with the sw counters. > > Essentially, these pmus just provide access to a big array of counters which > don't generate interrupts, and are all 64bit (and assumed to never > overflow). Rather than duplicate the code that we already have for managing > timing when reading from counters that don't have interrupts (the functions > that are exposed by this patch), I've reused it. So note that all the software counters generate interrupts in their own measuring domain. The hrtimer ones measure time and generate time based interrupts, the event based ones generate 'interrupts' on their events. What you have here is a hw pmu without interrupt capability. That's fine, they don't get to generate interrupt. We have plenty of those already. But what you propose to do is add interrupt in another domain entirely. That's not fine. Don't do that. You also try and conceal this information; so you suck. -- 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/