Re: [PATCH v10 00/16] Add utilization clamping support

Patrick Bellasi
2019-06-21T14:55:39Z

On 21-Jun 09:42, Patrick Bellasi wrote:
> Hi all, this is a respin of:
>
> https://lore.kernel.org/lkml/20190515094459.10317-1-patrick.bellasi@arm.com/
>
> which addresses all Tejun's concerns:
>
> - rename cgroup attributes to be cpu.uclamp.{min,max}
> - update initialization of subgroups clamps to be "no clamps" by default
> - use percentage rational numbers for clamp attributes, e.g. "12.34" for 12.34%.
>
> by introducing modifications impacting only patches:
>
> [PATCH v10 12/16] sched/core: uclamp: Extend CPU's cgroup controller
> [PATCH v10 13/16] sched/core: uclamp: Propagate parent clamps
>
> The rest of the patches are the same as per in v9, they have been just rebased
> on top of:
>
> tj/cgroup.git for-5.3
> tip/tip.git sched/core
>
> AFAIU, all the first 11 patches have been code reviewed and should be at a
> "ready to merge" quality level. Please let me know if I'm wrong and there
> is something else I need/can to do on those patches.
>
> Otherwise, now that we should have settled all the behavioral aspects, I'm

Regarding the behavioral aspects, here I have a report with some
simple tests for the current implementation:

https://gist.github.com/derkling/519459b5a2be35d8681fbaf1d6efe225

There are a couple of sections at the end to test the "Delegation
Model" with both CGroups v1 and v2.

I'm sharing the link just in case it can be helpful to verify if what
has been implemented is actually matching what Tejun expects as a sane
cgroups interface.

Cheers,
Patrick

--
#include <best/regards.h>

Patrick Bellasi