From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp.codeaurora.org by pdx-caf-mail.web.codeaurora.org (Dovecot) with LMTP id d2+2HYf1GFt+PgAAmS7hNA ; Thu, 07 Jun 2018 09:06:37 +0000 Received: by smtp.codeaurora.org (Postfix, from userid 1000) id 9D363607E7; Thu, 7 Jun 2018 09:06:37 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on pdx-caf-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.9 required=2.0 tests=BAYES_00,MAILING_LIST_MULTI autolearn=ham autolearn_force=no version=3.4.0 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by smtp.codeaurora.org (Postfix) with ESMTP id 228C76063F; Thu, 7 Jun 2018 09:06:37 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 228C76063F Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=arm.com Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932756AbeFGJGf (ORCPT + 25 others); Thu, 7 Jun 2018 05:06:35 -0400 Received: from usa-sjc-mx-foss1.foss.arm.com ([217.140.101.70]:48670 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753175AbeFGJGd (ORCPT ); Thu, 7 Jun 2018 05:06:33 -0400 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id D75C515AB; Thu, 7 Jun 2018 02:06:32 -0700 (PDT) Received: from [0.0.0.0] (e107985-lin.cambridge.arm.com [10.1.210.41]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id BBBED3F59D; Thu, 7 Jun 2018 02:06:30 -0700 (PDT) Subject: Re: [PATCH v5 07/10] sched/irq: add irq utilization tracking To: Vincent Guittot Cc: Peter Zijlstra , Ingo Molnar , linux-kernel , "Rafael J. Wysocki" , Juri Lelli , Morten Rasmussen , viresh kumar , Valentin Schneider , Quentin Perret References: <1527253951-22709-1-git-send-email-vincent.guittot@linaro.org> <1527253951-22709-8-git-send-email-vincent.guittot@linaro.org> <72473e6f-8ade-8e26-3282-276fcae4c4c7@arm.com> <4368a36c-3df4-1454-1837-473e569b9080@arm.com> From: Dietmar Eggemann Message-ID: <5ff5d418-ed84-9747-3868-597223ceb7e2@arm.com> Date: Thu, 7 Jun 2018 11:06:29 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.8.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 06/07/2018 10:44 AM, Vincent Guittot wrote: > On 7 June 2018 at 10:29, Dietmar Eggemann wrote: >> On 06/06/2018 06:06 PM, Vincent Guittot wrote: >>> >>> Hi Dietmar, >>> >>> Sorry for the late answer >>> >>> On 31 May 2018 at 18:54, Dietmar Eggemann >>> wrote: >>>> >>>> On 05/30/2018 08:45 PM, Vincent Guittot wrote: >>>>> >>>>> Hi Dietmar, >>>>> >>>>> On 30 May 2018 at 17:55, Dietmar Eggemann >>>>> wrote: >>>>>> >>>>>> On 05/25/2018 03:12 PM, Vincent Guittot wrote: [...] >> Can't buy this argument though because this is true with the current >> implementation as well since the 'decay load sum' - 'accrue load sum' >> sequence is not atomic. > > it's not a problem that the _sum variable are updated in different > step because there are internal variable > Only util_avg is used "outside" and the latter is updated after both > idle and running steps have been applied You're right here! >> What about calling update_irq_load_avg(rq, 0) in update_rq_clock_task() if >> (irq_delta + steal) eq. 0 and sched_feat(NONTASK_CAPACITY) eq. true in this >> #ifdef CONFIG_XXX_TIME_ACCOUNTING block? > > update_irq_load_avg(rq, 0) is called in update_blocked_averages to > decay smoothly like other blocked signals and replace the need to call > update_irq_load_avg(rq, 0) for every call to update_rq_clock_task() > which can be significant OK. >> Maintaining a irq/steal time signal makes only sense if at least one of the >> CONFIG_XXX_TIME_ACCOUNTING is set and NONTASK_CAPACITY is true. The call to >> update_irq_load_avg() in update_blocked_averages() isn't guarded my them. > > good point [...]