From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755916AbcBVVl0 (ORCPT ); Mon, 22 Feb 2016 16:41:26 -0500 Received: from mail-lf0-f50.google.com ([209.85.215.50]:33535 "EHLO mail-lf0-f50.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755663AbcBVVlX (ORCPT ); Mon, 22 Feb 2016 16:41:23 -0500 MIME-Version: 1.0 In-Reply-To: <20160222094246.GC27380@e106622-lin> References: <3071836.JbNxX8hU6x@vostro.rjw.lan> <1455900129.7375.231.camel@linux.intel.com> <20160219172545.GA27380@e106622-lin> <1882532.OhsNMeyWOd@vostro.rjw.lan> <20160222094246.GC27380@e106622-lin> Date: Mon, 22 Feb 2016 22:41:21 +0100 X-Google-Sender-Auth: _TwEE5oAY0aYLbYvlJnxGum7KtM Message-ID: Subject: Re: [PATCH v10 1/3] cpufreq: Add mechanism for registering utilization update callbacks From: "Rafael J. Wysocki" To: Juri Lelli Cc: "Rafael J. Wysocki" , Srinivas Pandruvada , "Rafael J. Wysocki" , Linux PM list , Peter Zijlstra , Ingo Molnar , Linux Kernel Mailing List , Viresh Kumar , Steve Muckle , Thomas Gleixner Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Feb 22, 2016 at 10:42 AM, Juri Lelli wrote: > Hi Rafael, > > On 19/02/16 23:26, Rafael J. Wysocki wrote: >> On Friday, February 19, 2016 05:26:04 PM Juri Lelli wrote: >> > Hi Srinivas, [cut] >> --- >> From: Rafael J. Wysocki >> Subject: [PATCH] cpufreq: Rework the scheduler hooks for triggering updates >> >> Commit fe7034338ba0 (cpufreq: Add mechanism for registering >> utilization update callbacks) added cpufreq_update_util() to be >> called by the scheduler (from the CFS part) on utilization updates. >> The goal was to allow CFS to pass utilization information to cpufreq >> and to trigger it to evaluate the frequency/voltage configuration >> (P-state) of every CPU on a regular basis. >> >> However, the last two arguments of that function are never used by >> the current code, so CFS might simply call cpufreq_trigger_update() >> instead of it. >> >> For this reason, drop the last two arguments of cpufreq_update_util(), >> rename it to cpufreq_trigger_update() and modify CFS to call it. >> >> Moreover, since the utilization is not involved in that now, rename >> data types, functions and variables related to cpufreq_trigger_update() >> to reflect that (eg. struct update_util_data becomes struct >> freq_update_hook and so on). >> >> Signed-off-by: Rafael J. Wysocki > > This patch looks good to me. I didn't yet test it, but it shouldn't > break things AFAICT. > > Thanks a lot for taking the time for this cleanup. Alas, I don't think I will apply it. Peter says that he wants the arguments to stay and he has a point IMO. The very idea behind hooking up cpufreq to the scheduler through those hooks has always been to make it possible to use the utilization information provided by the scheduler in cpufreq. As it turns out, we can make significant improvements even *without* using that information, because just having the hooks in there alone makes it possible to simplify the code quite a bit in general and make it more straightforward, but that's a *bonus* and not the objective. :-) The objective still is to use the utilization numbers from the scheduler. Both sched-freq and my approach agree on that, so I don't quite see why I should pretend that this isn't the case now? Thanks, Rafael