Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752233AbdGGHVK (ORCPT ); Fri, 7 Jul 2017 03:21:10 -0400 Received: from mail-pf0-f172.google.com ([209.85.192.172]:34917 "EHLO mail-pf0-f172.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750904AbdGGHVI (ORCPT ); Fri, 7 Jul 2017 03:21:08 -0400 Date: Fri, 7 Jul 2017 12:51:04 +0530 From: Viresh Kumar To: Juri Lelli Cc: peterz@infradead.org, mingo@redhat.com, rjw@rjwysocki.net, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, tglx@linutronix.de, vincent.guittot@linaro.org, rostedt@goodmis.org, luca.abeni@santannapisa.it, claudio@evidence.eu.com, tommaso.cucinotta@santannapisa.it, bristot@redhat.com, mathieu.poirier@linaro.org, tkjos@android.com, joelaf@google.com, andresoportus@google.com, morten.rasmussen@arm.com, dietmar.eggemann@arm.com, patrick.bellasi@arm.com, Ingo Molnar , "Rafael J . Wysocki" Subject: Re: [RFC PATCH v1 2/8] sched/deadline: move cpu frequency selection triggering points Message-ID: <20170707072104.GG1523@vireshk-i7> References: <20170705085905.6558-1-juri.lelli@arm.com> <20170705085905.6558-3-juri.lelli@arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170705085905.6558-3-juri.lelli@arm.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 963 Lines: 27 On 05-07-17, 09:58, Juri Lelli wrote: > Since SCHED_DEADLINE doesn't track utilization signal (but reserves a > fraction of CPU bandwidth to tasks admitted to the system), there is no > point in evaluating frequency changes during each tick event. > > Move frequency selection triggering points to where running_bw changes. > > Co-authored-by: Claudio Scordino > Signed-off-by: Juri Lelli > Cc: Peter Zijlstra > Cc: Ingo Molnar > Cc: Rafael J. Wysocki > Cc: Viresh Kumar > Cc: Luca Abeni > --- > Changes from RFCv0: > > - modify comment regarding periodic RT updates (Claudio) > --- > kernel/sched/deadline.c | 7 ++++--- > kernel/sched/sched.h | 12 ++++++------ > 2 files changed, 10 insertions(+), 9 deletions(-) Reviewed-by: Viresh Kumar -- viresh