linux-kernel - Re: [PATCH] sched: Add a new version sysctl to control child runs first

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 for Android: free password hash cracker in your pocket

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

Message-ID: <613f37fc.1c69fb81.9092.a4f5@mx.google.com>
Date:   Mon, 13 Sep 2021 11:37:31 +0000
From:   CGEL <cgel.zte@...il.com>
To:     Peter Zijlstra <peterz@...radead.org>
Cc:     yzaikin@...gle.com, liu.hailong6@....com.cn, mingo@...hat.com,
        juri.lelli@...hat.com, vincent.guittot@...aro.org,
        dietmar.eggemann@....com, rostedt@...dmis.org, bsegall@...gle.com,
        mgorman@...e.de, bristot@...hat.com, mcgrof@...nel.org,
        keescook@...omium.org, pjt@...gle.com, yang.yang29@....com.cn,
        joshdon@...gle.com, linux-kernel@...r.kernel.org,
        linux-fsdevel@...r.kernel.org, Zeal Robot <zealci@....com.cm>
Subject: Re: [PATCH] sched: Add a new version sysctl to control child runs
 first

On Mon, Sep 13, 2021 at 10:13:54AM +0200, Peter Zijlstra wrote:
> On Sun, Sep 12, 2021 at 04:12:23AM +0000, cgel.zte@...il.com wrote:
> > From: Yang Yang <yang.yang29@....com.cn>
> > 
> > The old version sysctl has some problems. First, it allows set value
> > bigger than 1, which is unnecessary. Second, it didn't follow the
> > rule of capabilities. Thirdly, it didn't use static key. This new
> > version fixes all the problems.
> 
> Does any of that actually matter?

For the first problem, I think the reason why sysctl_schedstats() only
accepts 0 or 1, is suitbale for sysctl_child_runs_first(). Since
task_fork_fair() only need sysctl_sched_child_runs_first to be
zero or non-zero.

For the second problem, I remember there is a rule: try to
administration system through capilities but not depends on
root identity. Just like sysctl_schedstats() or other
sysctl_xx().

For the thirdly problem, sysctl_child_runs_first maynot changes
often, but may accessed often, like static_key delayacct_key
controlled by sysctl_delayacct().

Thanks!