From mboxrd@z Thu Jan 1 00:00:00 1970 From: Viresh Kumar Subject: Re: [RFC/RFT][PATCH 1/2] cpufreq: schedutil: Use policy-dependent transition delays Date: Tue, 11 Apr 2017 16:44:43 +0530 Message-ID: <20170411111443.GE13627@vireshk-i7> References: <3498238.liCqOyIkGA@aspire.rjw.lan> <2407280.n9qVSLCrF5@aspire.rjw.lan> <71055380.gdY33coAtF@aspire.rjw.lan> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail-pf0-f169.google.com ([209.85.192.169]:35910 "EHLO mail-pf0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752291AbdDKLOr (ORCPT ); Tue, 11 Apr 2017 07:14:47 -0400 Received: by mail-pf0-f169.google.com with SMTP id o126so47401816pfb.3 for ; Tue, 11 Apr 2017 04:14:47 -0700 (PDT) Content-Disposition: inline In-Reply-To: <71055380.gdY33coAtF@aspire.rjw.lan> Sender: linux-pm-owner@vger.kernel.org List-Id: linux-pm@vger.kernel.org To: "Rafael J. Wysocki" Cc: Linux PM , Juri Lelli , LKML , Peter Zijlstra , Srinivas Pandruvada , Vincent Guittot , Patrick Bellasi , Joel Fernandes , Morten Rasmussen , Brendan Jackman On 11-04-17, 00:20, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > Make the schedutil governor take the initial (default) value of the > rate_limit_us sysfs attribute from the (new) transition_delay_us > policy parameter (to be set by the scaling driver). > > That will allow scaling drivers to make schedutil use smaller default > values of rate_limit_us and reduce the default average time interval > between consecutive frequency changes. > > Make intel_pstate set transition_delay_us to 500. > > Signed-off-by: Rafael J. Wysocki > --- > > This is a replacement for https://patchwork.kernel.org/patch/9671831/ > > --- > drivers/cpufreq/intel_pstate.c | 2 ++ > include/linux/cpufreq.h | 7 +++++++ > kernel/sched/cpufreq_schedutil.c | 15 ++++++++++----- > 3 files changed, 19 insertions(+), 5 deletions(-) Should we use this new value for the ondemand/conservative governors as well? -- viresh