linux-kernel - Re: [PATCH v2 sysctl-next] ftrace: move sysctl_ftrace

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: <YhqPIvyDtMM7Ehpl@bombadil.infradead.org>
Date:   Sat, 26 Feb 2022 12:35:46 -0800
From:   Luis Chamberlain <mcgrof@...nel.org>
To:     Wei Xiao <xiaowei66@...wei.com>
Cc:     rostedt@...dmis.org, mingo@...hat.com, keescook@...omium.org,
        yzaikin@...gle.com, linux-kernel@...r.kernel.org,
        linux-fsdevel@...r.kernel.org, young.liuyang@...wei.com,
        zengweilin@...wei.com, nixiaoming@...wei.com
Subject: Re: [PATCH v2 sysctl-next] ftrace: move sysctl_ftrace_enabled to
 ftrace.c

On Wed, Feb 23, 2022 at 07:11:53PM +0800, Wei Xiao wrote:
> This moves ftrace_enabled to trace/ftrace.c.
> 
> We move sysctls to places where features actually belong to improve
> the readability of the code and reduce the risk of code merge conflicts.
> At the same time, the proc-sysctl maintainers do not want to know what
> sysctl knobs you wish to add for your owner piece of code, we just care
> about the core logic.

Thanks for using a commit log message which helps folks understand why
we are doing this.

> 
> Signed-off-by: Wei Xiao <xiaowei66@...wei.com>

Thanks, queued on sysctl-next!

  Luis