lkml.org 
[lkml]   [2018]   [Nov]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v5 03/15] sched/core: uclamp: map TASK's clamp values into CPU's clamp groups
On Mon, Oct 29, 2018 at 06:32:57PM +0000, Patrick Bellasi wrote:
> +static int __setscheduler_uclamp(struct task_struct *p,
> + const struct sched_attr *attr)
> +{
> + unsigned int lower_bound = p->uclamp[UCLAMP_MIN].value;
> + unsigned int upper_bound = p->uclamp[UCLAMP_MAX].value;
> + int result = 0;
> +
> + if (attr->sched_flags & SCHED_FLAG_UTIL_CLAMP_MIN)
> + lower_bound = attr->sched_util_min;
> +
> + if (attr->sched_flags & SCHED_FLAG_UTIL_CLAMP_MAX)
> + upper_bound = attr->sched_util_max;
> +
> + if (lower_bound > upper_bound ||
> + upper_bound > SCHED_CAPACITY_SCALE)
> return -EINVAL;
>
> + mutex_lock(&uclamp_mutex);
>
> + /* Update each required clamp group */
> + if (attr->sched_flags & SCHED_FLAG_UTIL_CLAMP_MIN) {
> + uclamp_group_get(&p->uclamp[UCLAMP_MIN],
> + UCLAMP_MIN, lower_bound);
> + }
> + if (attr->sched_flags & SCHED_FLAG_UTIL_CLAMP_MAX) {
> + uclamp_group_get(&p->uclamp[UCLAMP_MAX],
> + UCLAMP_MAX, upper_bound);
> + }
> +
> + mutex_unlock(&uclamp_mutex);
> +
> + return result;
> +}

I'm missing where we error due to lack of groups.

\
 
 \ /
  Last update: 2018-11-07 15:38    [W:0.224 / U:0.144 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site