lkml.org 
[lkml]   [2022]   [Apr]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH sysctl-next] ftrace: fix building with SYSCTL=n but DYNAMIC_FTRACE=y
On Wed, Apr 20, 2022 at 07:28:07PM -0400, Steven Rostedt wrote:
> On Fri, 15 Apr 2022 14:54:04 -0700
> Luis Chamberlain <mcgrof@kernel.org> wrote:
>
> > Steven,
> >
> > As with the other fixes I can take this in through sysctl-next if you
> > are OK with that, please let me know.
>
> Honestly I would love to just nuke the ftrace_enabled sysctl, it's totally
> obsolete. Perhaps I should have it trigger some kind of warning message
> that it will be going away soon, and perhaps we can remove it?

I'm not sure if we can remove proc sysctls, that can break userpace
interaction. But deprecating it.. yeah I think it's fine to strive
for it, and just make the code do at least what it used to. In fact
sadly I don't see this documented well, we do have the documentation in
Documentation/admin-guide/sysctl/ but we don't really spell it out if
we can nuke some. Eric, do you have any thoughts on eventually
*removing* sysctls? Maybe we should document this.

BTW yeah another thing I noticed while looking at ftrace.c file was it could
probably use some spring cleaning by splitting the file up into multiple
files. Hey I'm not volunteering :) just an idea.

> Anyway, yeah, take this through your tree.

OK thanks, 0day found one last issue, I'll send a patch for that just
now. Let me know if you're OK for me to just squash that into this
patch.

> Acked-by: Steven Rostedt (Google) <rostedt@goodmis.org>

Luis

\
 
 \ /
  Last update: 2022-04-21 20:56    [W:0.043 / U:0.116 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site