Re: [PATCH] rcu,ftrace: Fix ftrace recursion
From: Paul E. McKenney
Date: Sun Oct 04 2020 - 20:19:14 EST
On Tue, Sep 29, 2020 at 01:04:49PM -0400, Steven Rostedt wrote:
> On Tue, 29 Sep 2020 09:56:40 -0700
> "Paul E. McKenney" <paulmck@xxxxxxxxxx> wrote:
>
> > > Well, I think we should actually apply both, but the comment needs to be
> > > updated, as it will no longer be outside recursion. And the comment is
> > > wrong now as well, as its only outside recursion protection for the
> > > assist_func().
> > >
> > > But it does prevent it from being always called for perf.
> > >
> > > * Make notrace because it can be called by the internal functions of
> > > * ftrace, and making this notrace removes unnecessary recursion calls.
> >
> > Fair enough. ;-)
> >
> > If I don't hear otherwise by late today (Tuesday), Pacific Time, I will
> > update the comment and pull it into -rcu. If you guys have some other
> > route to mainline in mind, you have my Reviewed-by. Either way, just
> > let me know.
>
> I'm currently testing the recursion fix and will push that to Linus when
> done. But you can take the comment update through your tree.
>
> Peter, are you OK if Paul changes your comment to what I suggested?
Hearing no objections, I have queued the patch with the comment updated
as suggested by Steven. Thank you all!
Thanx, Paul