Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754910AbZGEEa1 (ORCPT ); Sun, 5 Jul 2009 00:30:27 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751293AbZGEEaQ (ORCPT ); Sun, 5 Jul 2009 00:30:16 -0400 Received: from hera.kernel.org ([140.211.167.34]:37388 "EHLO hera.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750882AbZGEEaP (ORCPT ); Sun, 5 Jul 2009 00:30:15 -0400 Subject: Re: [PATCH 4/6 -tip] perf_counter: Add Generalized Hardware interrupt support for AMD From: Jaswinder Singh Rajput To: Ingo Molnar Cc: Peter Zijlstra , Arjan van de Ven , =?ISO-8859-1?Q?Fr=E9d=E9ric?= Weisbecker , Arnaldo Carvalho de Melo , Paul Mackerras , Anton Blanchard , Thomas Gleixner , Peter Zijlstra , x86 maintainers , LKML , Alan Cox In-Reply-To: <20090705011123.GA31673@elte.hu> References: <1246440815.3403.3.camel@hpdv5.satnam> <1246440909.3403.5.camel@hpdv5.satnam> <1246440977.3403.7.camel@hpdv5.satnam> <1246441043.3403.9.camel@hpdv5.satnam> <1246441094.3403.10.camel@hpdv5.satnam> <20090701112406.GE15958@elte.hu> <1246622515.2322.30.camel@jaswinder.satnam> <20090704102230.GE2139@elte.hu> <1246717039.2329.26.camel@jaswinder.satnam> <20090705011123.GA31673@elte.hu> Content-Type: text/plain Date: Sun, 05 Jul 2009 09:59:26 +0530 Message-Id: <1246768166.2325.8.camel@jaswinder.satnam> Mime-Version: 1.0 X-Mailer: Evolution 2.26.1 (2.26.1-2.fc11) Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2624 Lines: 71 On Sun, 2009-07-05 at 03:11 +0200, Ingo Molnar wrote: > * Jaswinder Singh Rajput wrote: > > > On Sat, 2009-07-04 at 12:22 +0200, Ingo Molnar wrote: > > > * Jaswinder Singh Rajput wrote: > > > > > > > On Wed, 2009-07-01 at 13:24 +0200, Ingo Molnar wrote: > > > > > * Jaswinder Singh Rajput wrote: > > > > > > > > > > > > > > > > > $ ./perf stat -e interrupts -e masked -e int-pending-mask-cycles -- ls -lR /usr/include/ > /dev/null > > > > > > > > > > > > Performance counter stats for 'ls -lR /usr/include/': > > > > > > > > > > > > 377 interrupts > > > > > > 53429936 int-mask-cycles > > > > > > 1119 int-pending-mask-cycles > > > > > > > > > > > > 0.371457539 seconds time elapsed > > > > > > > > > > Agreed, this is another useful generalization - and the 'cycles > > > > > pending' metrics are not retrievable via any software means. > > > > > > > > > > We could and should probably add a software counter for hardirqs > > > > > as wel. That would allow the vector/irqnr information to be > > > > > passed in, and it would allow architectures without irq metrics > > > > > in the PMU to have this counter too. > > > > > > > > > > > > > Please let me know that addition of software counter will be > > > > in this patch or we can do it incrementally after this patch. > > > > > > It should be in this series. That way we can cross-check whether > > > the soft counts and the hard counts match up and find potential > > > bugs that way, etc. > > > > > > > You want to cross check performance counter events ? > > Yes. The events are also more complete if we add per IRQ source > counts as well, not just summary counts. > If you ask me about 'complete', I will say : "No-one is 'complete' except God". Let me know what you mean by 'complete' and 'more complete'. This is a hardware performance interrupt event patch. If you want to add IRQ source, of course you can add it in another patch, it is a never ending task. I do not understand why you behave like this : 1. Is today the last day of the creation. 2. Or you will not collect any further patches. Of course answer is "no" then what is the problem with you. Stop this complete-ness madness. You will never complete atleast in this life no matter what you will do. Thanks, -- JSR -- 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/