Re: [GIT PULL] RCU changes for v5.10
From: Paul E. McKenney
Date: Sun Oct 18 2020 - 23:24:04 EST
On Sun, Oct 18, 2020 at 02:39:56PM -0700, Linus Torvalds wrote:
> On Mon, Oct 12, 2020 at 7:14 AM Ingo Molnar <mingo@xxxxxxxxxx> wrote:
> >
> > Please pull the latest core/rcu git tree from:
> >
> > git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git core-rcu-2020-10-12
>
> I've pulled everything but that last merge and the PREEMPT_COUNT stuff
> that came with it.
>
> When Paul asked whether it was ok for RCU to use preempt_count() and I
> answered in the affirmative, I didn't mean it in the sense of "RCU
> wants to force it on everybody else too".
>
> I'm pretty convinced that the proper fix is to simply make sure that
> rcu_free() and friends aren't run under any raw spinlocks. So even if
> the cost of preempt-count isn't that noticeable, there just isn't a
> reason for RCU to say "screw everybody else, I want this" when there
> are other alternatives.
Thank you for pulling the other branches.
On CONFIG_PREEMPT_COUNT, got it. It would be OK for RCU to use
preempt_count() for some debugging or specialty kernel, but not across
the board. Thank you for bearing with me on this one.
There is more to it than just raw spinlocks, but regardless we will go
back to the drawing board and come up with a less intrusive fix for the
v5.11 merge window.
Thanx, Paul