lkml.org 
[lkml]   [2018]   [Jun]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH v6 08/11] cpufreq/schedutil: take into account interrupt
On 12 June 2018 at 10:54, Dietmar Eggemann <dietmar.eggemann@arm.com> wrote:
> On 06/08/2018 02:09 PM, Vincent Guittot wrote:
>
> [...]
>
>> @@ -182,21 +183,30 @@ static void sugov_get_util(struct sugov_cpu *sg_cpu)
>> sg_cpu->util_dl = cpu_util_dl(rq);
>> sg_cpu->bw_dl = cpu_bw_dl(rq);
>> sg_cpu->util_rt = cpu_util_rt(rq);
>> + sg_cpu->util_irq = cpu_util_irq(rq);
>> }
>> static unsigned long sugov_aggregate_util(struct sugov_cpu *sg_cpu)
>> {
>> struct rq *rq = cpu_rq(sg_cpu->cpu);
>> - unsigned long util;
>> + unsigned long util, max = sg_cpu->max;
>> if (rq->rt.rt_nr_running)
>> return sg_cpu->max;
>> + if (unlikely(sg_cpu->util_irq >= max))
>> + return max;
>> +
>> + /* Sum rq utilization */
>> util = sg_cpu->util_cfs;
>> util += sg_cpu->util_rt;
>> - if ((util + sg_cpu->util_dl) >= sg_cpu->max)
>> - return sg_cpu->max;
>> :confirm b9
>
>
> This didn't let me apply the patch ;-) After removing this line it worked.

Argh ... i have done something wrong
I'm going to resend it
>
> [...]

\
 
 \ /
  Last update: 2018-06-12 11:11    [W:0.062 / U:0.840 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site