lkml.org 
[lkml]   [2020]   [Oct]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH v2] sched/fair: prefer prev cpu in asymmetric wakeup path
Date

On 29/10/20 12:17, Tao Zhou wrote:
> Hi Vincent,
>
> On Wed, Oct 28, 2020 at 06:44:12PM +0100, Vincent Guittot wrote:
>> @@ -6173,20 +6173,20 @@ static int
>> select_idle_capacity(struct task_struct *p, struct sched_domain *sd, int target)
>> {
>> unsigned long best_cap = 0;
>> - int cpu, best_cpu = -1;
>> + int task_util, cpu, best_cpu = -1;
>> struct cpumask *cpus;
>>
>> - sync_entity_load_avg(&p->se);
>> -
>> cpus = this_cpu_cpumask_var_ptr(select_idle_mask);
>> cpumask_and(cpus, sched_domain_span(sd), p->cpus_ptr);
>>
>> + task_util = uclamp_task_util(p);
>
> The return type of uclamp_task_util() is *unsigned long*.
> But, task_util is *int*.
>
> @Valentin: I checked that the return type of uclamp_task_util()
> is changed from unsigned int to unsigned long by you
>

Indeed, IIRC I stated in that changelog that the canonical type for
utilization / capacity values is unsigned long.

\
 
 \ /
  Last update: 2020-10-29 13:25    [W:0.102 / U:1.584 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site