From: Dietmar Eggemann <dietmar.eggemann@arm.com>
To: Patrick Bellasi <patrick.bellasi@arm.com>,
linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org
Cc: Ingo Molnar <mingo@redhat.com>,
Peter Zijlstra <peterz@infradead.org>, Tejun Heo <tj@kernel.org>,
"Rafael J . Wysocki" <rafael.j.wysocki@intel.com>,
Viresh Kumar <viresh.kumar@linaro.org>,
Vincent Guittot <vincent.guittot@linaro.org>,
Paul Turner <pjt@google.com>,
Morten Rasmussen <morten.rasmussen@arm.com>,
Juri Lelli <juri.lelli@redhat.com>, Todd Kjos <tkjos@google.com>,
Joel Fernandes <joelaf@google.com>,
Steve Muckle <smuckle@google.com>,
Suren Baghdasaryan <surenb@google.com>
Subject: Re: [PATCH v3 12/14] sched/core: uclamp: add system default clamps
Date: Mon, 20 Aug 2018 12:18:40 +0200 [thread overview]
Message-ID: <2816b340-ca91-4d09-1610-df05a9893da2@arm.com> (raw)
In-Reply-To: <20180806163946.28380-13-patrick.bellasi@arm.com>
On 08/06/2018 06:39 PM, Patrick Bellasi wrote:
> Clamp values cannot be tuned at the root cgroup level. Moreover, because
> of the delegation model requirements and how the parent clamps
> propagation works, if we want to enable subgroups to set a non null
> util.min, we need to be able to configure the root group util.min to the
> allow the maximum utilization (SCHED_CAPACITY_SCALE = 1024).
Why 1024 (100)? Would any non 0 value work here?
[...]
> @@ -1269,6 +1296,75 @@ static inline void uclamp_group_get(struct task_struct *p,
> uclamp_group_put(clamp_id, prev_group_id);
> }
>
> +int sched_uclamp_handler(struct ctl_table *table, int write,
> + void __user *buffer, size_t *lenp,
> + loff_t *ppos)
> +{
> + int group_id[UCLAMP_CNT] = { UCLAMP_NOT_VALID };
> + struct uclamp_se *uc_se;
> + int old_min, old_max;
> + int result;
> +
> + mutex_lock(&uclamp_mutex);
> +
> + old_min = sysctl_sched_uclamp_util_min;
> + old_max = sysctl_sched_uclamp_util_max;
> +
> + result = proc_dointvec(table, write, buffer, lenp, ppos);
> + if (result)
> + goto undo;
> + if (!write)
> + goto done;
> +
> + if (sysctl_sched_uclamp_util_min > sysctl_sched_uclamp_util_max)
> + goto undo;
> + if (sysctl_sched_uclamp_util_max > 1024)
> + goto undo;
> +
> + /* Find a valid group_id for each required clamp value */
> + if (old_min != sysctl_sched_uclamp_util_min) {
> + result = uclamp_group_find(UCLAMP_MIN, sysctl_sched_uclamp_util_min);
> + if (result == -ENOSPC) {
> + pr_err("Cannot allocate more than %d UTIL_MIN clamp groups\n",
> + CONFIG_UCLAMP_GROUPS_COUNT);
> + goto undo;
> + }
> + group_id[UCLAMP_MIN] = result;
> + }
> + if (old_max != sysctl_sched_uclamp_util_max) {
> + result = uclamp_group_find(UCLAMP_MAX, sysctl_sched_uclamp_util_max);
> + if (result == -ENOSPC) {
> + pr_err("Cannot allocate more than %d UTIL_MAX clamp groups\n",
> + CONFIG_UCLAMP_GROUPS_COUNT);
> + goto undo;
> + }
> + group_id[UCLAMP_MAX] = result;
> + }
> +
> + /* Update each required clamp group */
> + if (old_min != sysctl_sched_uclamp_util_min) {
> + uc_se = &uclamp_default[UCLAMP_MIN];
> + uclamp_group_get(NULL, UCLAMP_MIN, group_id[UCLAMP_MIN],
> + uc_se, sysctl_sched_uclamp_util_min);
> + }
> + if (old_max != sysctl_sched_uclamp_util_max) {
> + uc_se = &uclamp_default[UCLAMP_MAX];
> + uclamp_group_get(NULL, UCLAMP_MAX, group_id[UCLAMP_MAX],
> + uc_se, sysctl_sched_uclamp_util_max);
> + }
> +
> + if (result) {
> +undo:
> + sysctl_sched_uclamp_util_min = old_min;
> + sysctl_sched_uclamp_util_max = old_max;
> + }
This looks strange! In case uclamp_group_find() returns free_group_id
instead of -ENOSPC, the sysctl min/max values are reset?
I was under the assumption that I could specify:
sysctl_sched_uclamp_util_min = 40 (for boosting)
sysctl_sched_uclamp_util_max = 80 (for clamping)
with an empty cpu controller hierarchy and then those values become the
.effective values of (a first level) task group?
[...]
next prev parent reply other threads:[~2018-08-20 10:18 UTC|newest]
Thread overview: 82+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-08-06 16:39 [PATCH v3 00/14] Add utilization clamping support Patrick Bellasi
2018-08-06 16:39 ` [PATCH v3 01/14] sched/core: uclamp: extend sched_setattr to support utilization clamping Patrick Bellasi
2018-08-06 16:50 ` Randy Dunlap
2018-08-09 8:39 ` Patrick Bellasi
2018-08-09 15:20 ` Randy Dunlap
2018-08-07 9:59 ` Juri Lelli
2018-08-13 12:14 ` Patrick Bellasi
2018-08-13 12:27 ` Juri Lelli
2018-08-07 12:35 ` Juri Lelli
2018-08-09 9:14 ` Patrick Bellasi
2018-08-09 9:50 ` Juri Lelli
2018-08-09 15:23 ` Patrick Bellasi
2018-08-10 7:50 ` Juri Lelli
2018-08-17 10:34 ` Quentin Perret
2018-08-17 10:57 ` Patrick Bellasi
2018-08-17 11:14 ` Quentin Perret
2018-08-06 16:39 ` [PATCH v3 02/14] sched/core: uclamp: map TASK's clamp values into CPU's clamp groups Patrick Bellasi
2018-08-14 11:25 ` Pavan Kondeti
2018-08-14 15:21 ` Patrick Bellasi
2018-08-06 16:39 ` [PATCH v3 03/14] sched/core: uclamp: add CPU's clamp groups accounting Patrick Bellasi
2018-08-14 15:44 ` Dietmar Eggemann
2018-08-14 16:49 ` Patrick Bellasi
2018-08-15 9:37 ` Dietmar Eggemann
2018-08-15 10:54 ` Patrick Bellasi
2018-08-15 10:59 ` Dietmar Eggemann
2018-08-16 13:32 ` Patrick Bellasi
2018-08-16 13:37 ` Quentin Perret
2018-08-16 13:45 ` Dietmar Eggemann
2018-08-16 14:21 ` Quentin Perret
2018-08-16 15:00 ` Dietmar Eggemann
2018-08-17 11:04 ` Patrick Bellasi
2018-08-06 16:39 ` [PATCH v3 04/14] sched/core: uclamp: update CPU's refcount on clamp changes Patrick Bellasi
2018-08-15 15:02 ` Dietmar Eggemann
2018-08-16 13:22 ` Patrick Bellasi
2018-08-06 16:39 ` [PATCH v3 05/14] sched/cpufreq: uclamp: add utilization clamping for FAIR tasks Patrick Bellasi
2018-08-08 13:18 ` Vincent Guittot
2018-08-09 15:30 ` Patrick Bellasi
2018-08-15 15:30 ` Dietmar Eggemann
2018-08-16 13:53 ` Patrick Bellasi
2018-08-06 16:39 ` [PATCH v3 06/14] sched/cpufreq: uclamp: add utilization clamping for RT tasks Patrick Bellasi
2018-08-07 13:26 ` Juri Lelli
2018-08-09 15:34 ` Patrick Bellasi
2018-08-09 16:03 ` Vincent Guittot
2018-08-13 10:12 ` Patrick Bellasi
2018-08-13 10:50 ` Juri Lelli
2018-08-13 12:07 ` Vincent Guittot
2018-08-13 12:09 ` Vincent Guittot
2018-08-13 12:49 ` Patrick Bellasi
2018-08-13 14:06 ` Vincent Guittot
2018-08-13 15:01 ` Patrick Bellasi
2018-08-16 10:34 ` Dietmar Eggemann
2018-08-16 13:40 ` Patrick Bellasi
2018-08-07 13:54 ` Quentin Perret
2018-08-09 15:41 ` Patrick Bellasi
2018-08-09 15:55 ` Quentin Perret
2018-08-13 10:17 ` Patrick Bellasi
2018-08-06 16:39 ` [PATCH v3 07/14] sched/core: uclamp: enforce last task UCLAMP_MAX Patrick Bellasi
2018-08-16 15:43 ` Dietmar Eggemann
2018-08-16 16:47 ` Patrick Bellasi
2018-08-16 17:10 ` Dietmar Eggemann
2018-08-16 17:27 ` Patrick Bellasi
2018-08-16 17:20 ` Patrick Bellasi
2018-08-06 16:39 ` [PATCH v3 08/14] sched/core: uclamp: extend cpu's cgroup controller Patrick Bellasi
2018-08-17 12:21 ` Dietmar Eggemann
2018-08-17 14:24 ` Patrick Bellasi
2018-08-06 16:39 ` [PATCH v3 09/14] sched/core: uclamp: propagate parent clamps Patrick Bellasi
2018-08-16 9:09 ` Pavan Kondeti
2018-08-16 14:07 ` Patrick Bellasi
2018-08-17 13:43 ` Dietmar Eggemann
2018-08-17 14:45 ` Patrick Bellasi
2018-08-17 15:50 ` Dietmar Eggemann
2018-08-20 10:01 ` Dietmar Eggemann
2018-08-20 12:28 ` Patrick Bellasi
2018-08-06 16:39 ` [PATCH v3 10/14] sched/core: uclamp: map TG's clamp values into CPU's clamp groups Patrick Bellasi
2018-08-06 16:39 ` [PATCH v3 11/14] sched/core: uclamp: use TG's clamps to restrict Task's clamps Patrick Bellasi
2018-08-06 16:39 ` [PATCH v3 12/14] sched/core: uclamp: add system default clamps Patrick Bellasi
2018-08-16 9:13 ` Pavan Kondeti
2018-08-16 14:37 ` Patrick Bellasi
2018-08-20 10:18 ` Dietmar Eggemann [this message]
2018-08-20 12:27 ` Patrick Bellasi
2018-08-06 16:39 ` [PATCH v3 13/14] sched/core: uclamp: update CPU's refcount on TG's clamp changes Patrick Bellasi
2018-08-06 16:39 ` [PATCH v3 14/14] sched/core: uclamp: use percentage clamp values Patrick Bellasi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=2816b340-ca91-4d09-1610-df05a9893da2@arm.com \
--to=dietmar.eggemann@arm.com \
--cc=joelaf@google.com \
--cc=juri.lelli@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=mingo@redhat.com \
--cc=morten.rasmussen@arm.com \
--cc=patrick.bellasi@arm.com \
--cc=peterz@infradead.org \
--cc=pjt@google.com \
--cc=rafael.j.wysocki@intel.com \
--cc=smuckle@google.com \
--cc=surenb@google.com \
--cc=tj@kernel.org \
--cc=tkjos@google.com \
--cc=vincent.guittot@linaro.org \
--cc=viresh.kumar@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).