Re: [patch 20/20] rcu: Make CPU_DYING_IDLE an explicit call
From: Thomas Gleixner
Date: Sat Feb 27 2016 - 02:49:24 EST
On Fri, 26 Feb 2016, Paul E. McKenney wrote:
> > > --- a/kernel/cpu.c
> > > +++ b/kernel/cpu.c
> > > @@ -762,6 +762,7 @@ void cpuhp_report_idle_dead(void)
> > > BUG_ON(st->state != CPUHP_AP_OFFLINE);
> > > st->state = CPUHP_AP_IDLE_DEAD;
> > > complete(&st->done);
> > What prevents the other CPU from killing this CPU at this point, so
> > that this CPU does not tell RCU that it is dead?
> > I agree that the odds should be low, but there are all manner of things
> > that might delay a CPU for just a little bit too long...
> > Or am I missing something subtle here?
No. The reason why I moved the rcu call past the complete is, that otherwise
complete() complains about rcu being dead already. Hmm, but you are right. In
theory the other side could allow physical removal before it actually told rcu
that it's gone.
> Just in case I am not missing anything...
> One approach is to go back to the spinning, but to do rcu_report_dead()
> just before kicking the other CPU. This would also fix some issues with
> use of RCU of the offline path, so would definitely be better than my
> earlier approach of notifying RCU from within the idle loop.
> This assumes that all the offline paths have been consolidated into
> this path. (Yes, I was too lazy and cowardly to consolidate them all
> last I touched this code, but perhaps that has happened elsewise?)
The question is whether the rcu dead notification has to happen
instantaniously and needs to be done on the dead cpu. If we can avoid both,
then there is a very simple solution.