Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754576Ab0HXJJ1 (ORCPT ); Tue, 24 Aug 2010 05:09:27 -0400 Received: from casper.infradead.org ([85.118.1.10]:52328 "EHLO casper.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754188Ab0HXJJ0 convert rfc822-to-8bit (ORCPT ); Tue, 24 Aug 2010 05:09:26 -0400 Subject: Re: [PATCH 0/4] Finer granularity and task/cgroup irq time accounting From: Peter Zijlstra To: balbir@linux.vnet.ibm.com Cc: Venkatesh Pallipadi , Martin Schwidefsky , Ingo Molnar , "H. Peter Anvin" , Thomas Gleixner , Paul Menage , linux-kernel@vger.kernel.org, Paul Turner , Heiko Carstens , Paul Mackerras , Tony Luck In-Reply-To: <20100824080515.GK4684@balbir.in.ibm.com> References: <1279583835-22854-1-git-send-email-venki@google.com> <20100720095546.2f899e04@mschwide.boeblingen.de.ibm.com> <20100722131239.208d9501@mschwide.boeblingen.de.ibm.com> <1282636286.2605.2307.camel@laptop> <20100824080515.GK4684@balbir.in.ibm.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8BIT Date: Tue, 24 Aug 2010 11:09:13 +0200 Message-ID: <1282640953.2605.2428.camel@laptop> Mime-Version: 1.0 X-Mailer: Evolution 2.28.3 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1910 Lines: 42 On Tue, 2010-08-24 at 13:35 +0530, Balbir Singh wrote: > * Peter Zijlstra [2010-08-24 09:51:26]: > > > On Thu, 2010-07-22 at 19:12 -0700, Venkatesh Pallipadi wrote: > > > > > > > > Well, the task and cgroup information is there but what does it really > > > > tell me? As long as the irq & softirq time can be caused by any other > > > > process I don't see the value of this incorrect data point. > > > > > > > > > > Data point will be correct. How it gets used is a different qn. This > > > interface will be useful for Alert/Paranoid/Annoyed user/admin who > > > sees that the job exec_time is high but it is not doing any useful > > > work. > > > > I'm very sympathetic with Martin's POV. irq/softirq times per task don't > > really make sense. In the case you provide above the solution would be > > to subtract these times from the task execution time, not break it out. > > In that case he would see his task not do much, and end up with the same > > action list. > > > > cgroup level info does make sense, assuming that tasks that share the > costs being mentioned here belong to the same cgroup. I don't think that's a valid assumption. If its not true for tasks, then its not true for groups of tasks either. It might be slightly less wrong due to the larger number of entities reducing the error bounds, but its still wrong in principle. The whole attribution mess can only be solved by actually splitting out the entries that do work, like per-cgroup workqueue threads and similar things. System wide entities like IRQs are very hard to attribute correctly like Martin already argued, and I don't think its worth doing. -- 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/