Skip to content

Commit 7cb7fb5

Browse files
cloehleIngo Molnar
authored andcommitted
sched/fair: Remove stale FREQUENCY_UTIL comment
On 05/03/2024 15:05, Vincent Guittot wrote: I'm fine with either and that was my first thought here, too, but it did seem like the comment was mostly placed there to justify the 'unexpected' high utilization when explicitly passing FREQUENCY_UTIL and the need to clamp it then. So removing did feel slightly more natural to me anyway. So alternatively: From: Christian Loehle <christian.loehle@arm.com> Date: Tue, 5 Mar 2024 09:34:41 +0000 Subject: [PATCH] sched/fair: Remove stale FREQUENCY_UTIL mention effective_cpu_util() flags were removed, so remove mentioning of the flag. commit 9c0b4bb ("sched/cpufreq: Rework schedutil governor performance estimation") reworked effective_cpu_util() removing enum cpu_util_type. Modify the comment accordingly. Signed-off-by: Christian Loehle <christian.loehle@arm.com> Signed-off-by: Ingo Molnar <mingo@kernel.org> Reviewed-by: Vincent Guittot <vincent.guittot@linaro.org> Link: https://lore.kernel.org/r/0e2833ee-0939-44e0-82a2-520a585a0153@arm.com
1 parent 72bffbf commit 7cb7fb5

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

kernel/sched/fair.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7900,8 +7900,8 @@ eenv_pd_max_util(struct energy_env *eenv, struct cpumask *pd_cpus,
79007900
* Performance domain frequency: utilization clamping
79017901
* must be considered since it affects the selection
79027902
* of the performance domain frequency.
7903-
* NOTE: in case RT tasks are running, by default the
7904-
* FREQUENCY_UTIL's utilization can be max OPP.
7903+
* NOTE: in case RT tasks are running, by default the min
7904+
* utilization can be max OPP.
79057905
*/
79067906
eff_util = effective_cpu_util(cpu, util, &min, &max);
79077907

0 commit comments

Comments
 (0)