On Thu, 10 Oct 2019 15:38:20 +0200 (CEST)
Miroslav Benes <mbenes@xxxxxxx> wrote:
> On Thu, 10 Oct 2019, Steven Rostedt wrote:
>
> > On Thu, 10 Oct 2019 10:50:35 +0200
> > Petr Mladek <pmladek@xxxxxxxx> wrote:
> >
> > > It will make the flag unusable for other ftrace users. But it
> > > will be already be the case when it can't be disabled.
> >
> > Honestly, I hate that flag. Most people don't even know about it. It
> > was added in the beginning of ftrace as a way to stop function tracing
> > in the latency tracer. But that use case has been obsoleted by
> > 328df4759c03e ("tracing: Add function-trace option to disable function
> > tracing of latency tracers"). I may just remove the damn thing and only
> > add it back if somebody complains about it.
>
> That would of course solve the issue too and code removal is always
> better.
>
Yes, but let's still add the patch that does the permanent check. And
then I'll put the "remove this flag" patch on top (and revert
everything else). This way, if somebody complains, and Linus reverts
the removal patch, we don't end up breaking live kernel patching
again ;-)
-- Steve
