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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Fri, 21 Jun 2019 15:55:32 +0100
From:   Patrick Bellasi <patrick.bellasi@....com>
To:     linux-kernel@...r.kernel.org, linux-pm@...r.kernel.org
Cc:     Ingo Molnar <mingo@...hat.com>,
        Peter Zijlstra <peterz@...radead.org>,
        Tejun Heo <tj@...nel.org>,
        "Rafael J . Wysocki" <rafael.j.wysocki@...el.com>,
        Vincent Guittot <vincent.guittot@...aro.org>,
        Viresh Kumar <viresh.kumar@...aro.org>,
        Paul Turner <pjt@...gle.com>,
        Quentin Perret <quentin.perret@....com>,
        Dietmar Eggemann <dietmar.eggemann@....com>,
        Morten Rasmussen <morten.rasmussen@....com>,
        Juri Lelli <juri.lelli@...hat.com>,
        Todd Kjos <tkjos@...gle.com>,
        Joel Fernandes <joelaf@...gle.com>,
        Steve Muckle <smuckle@...gle.com>,
        Suren Baghdasaryan <surenb@...gle.com>,
        Alessio Balsini <balsini@...roid.com>
Subject: Re: [PATCH v10 00/16] Add utilization clamping support

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