Currently schedutil enforces a maximum frequency when RT/DL tasks are
RUNNABLE.  Such a mandatory policy can be made more tunable from
userspace thus allowing for example to define a max frequency which is
still reasonable for the execution of a specific RT/DL workload. This
will contribute to make the RT class more friendly for power/energy
sensitive use-cases.

This patch extends the usage of util_{min,max} to the RT/DL classes.
Whenever a task in these classes is RUNNABLE, the util required is
defined by the constraints of the CPU control group the task belongs to.

Signed-off-by: Patrick Bellasi <[email protected]>
Cc: Ingo Molnar <[email protected]>
Cc: Peter Zijlstra <[email protected]>
Cc: Rafael J. Wysocki <[email protected]>
Cc: [email protected]
Cc: [email protected]
---
 kernel/sched/cpufreq_schedutil.c | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

diff --git a/kernel/sched/cpufreq_schedutil.c b/kernel/sched/cpufreq_schedutil.c
index f67c26bbade4..feca60c107bc 100644
--- a/kernel/sched/cpufreq_schedutil.c
+++ b/kernel/sched/cpufreq_schedutil.c
@@ -227,7 +227,10 @@ static void sugov_update_single(struct update_util_data 
*hook, u64 time,
        busy = sugov_cpu_is_busy(sg_cpu);
 
        if (flags & SCHED_CPUFREQ_RT_DL) {
-               next_f = policy->cpuinfo.max_freq;
+               util = uclamp_util(smp_processor_id(), SCHED_CAPACITY_SCALE);
+               next_f = (uclamp_enabled && util < SCHED_CAPACITY_SCALE)
+                       ? get_next_freq(sg_policy, util, 
policy->cpuinfo.max_freq)
+                       : policy->cpuinfo.max_freq;
        } else {
                sugov_get_util(&util, &max);
                sugov_iowait_boost(sg_cpu, &util, &max);
@@ -276,10 +279,15 @@ static unsigned int sugov_next_freq_shared(struct 
sugov_cpu *sg_cpu, u64 time)
                        j_sg_cpu->iowait_boost = 0;
                        continue;
                }
-               if (j_sg_cpu->flags & SCHED_CPUFREQ_RT_DL)
-                       return policy->cpuinfo.max_freq;
 
-               j_util = j_sg_cpu->util;
+               if (j_sg_cpu->flags & SCHED_CPUFREQ_RT_DL) {
+                       if (!uclamp_enabled)
+                               return policy->cpuinfo.max_freq;
+                       j_util = uclamp_util(j, SCHED_CAPACITY_SCALE);
+               } else {
+                       j_util = j_sg_cpu->util;
+               }
+
                j_max = j_sg_cpu->max;
                if (j_util * max > j_max * util) {
                        util = j_util;
-- 
2.14.1

Reply via email to