linux-kernel - Re: [PATCH v3] sysctl: Add a group of macro functions to initcall the sysctl table of each feature

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] [day] [month] [year] [list]

Message-ID: <Yb+lCVrP3eGfKjE+@bombadil.infradead.org>
Date:   Sun, 19 Dec 2021 13:32:57 -0800
From:   Luis Chamberlain <mcgrof@...nel.org>
To:     Xiaoming Ni <nixiaoming@...wei.com>
Cc:     linux-kernel@...r.kernel.org, viro@...iv.linux.org.uk,
        ebiederm@...ssion.com, keescook@...omium.org, jlayton@...nel.org,
        bfields@...ldses.org, yzaikin@...gle.com, apw@...onical.com,
        joe@...ches.com, dwaipayanray1@...il.com, lukas.bulwahn@...il.com,
        julia.lawall@...ia.fr, akpm@...ux-foundation.org,
        wangle6@...wei.com
Subject: Re: [PATCH v3] sysctl: Add a group of macro functions to initcall
 the sysctl table of each feature

On Mon, Dec 13, 2021 at 11:31:19AM +0800, Xiaoming Ni wrote:
> To avoid duplicated code, add a set of macro functions to initialize the
> sysctl table for each feature.
> 
> The system initialization process is as follows:

See Eric's comments, we don't need to take care here on order as if the
path does not exist, it will be created. So please updates the commit to
to instead reflect that reality as I was just lazy and assumed the worst
and was just being cautious. If you get to add a few tests on
lib/test_sysctl.c and tools/testing/selftests/sysctl/sysctl.sh  to
verify it would be great too.

  Luis