On Thu, Sep 22, 2022 at 06:54:42AM -0700, Paul E. McKenney wrote:
> On Thu, Sep 22, 2022 at 05:29:32PM +0800, Pingfan Liu wrote:
> > My understanding is after removing the tick dep by
> > diff --git a/kernel/rcu/tree.c b/kernel/rcu/tree.c
> > index 79aea7df4345..cbfc884f04a4 100644
> > --- a/kernel/rcu/tree.c
> > +++ b/kernel/rcu/tree.c
> > @@ -2171,8 +2171,6 @@ int rcutree_dead_cpu(unsigned int cpu)
> > WRITE_ONCE(rcu_state.n_online_cpus, rcu_state.n_online_cpus - 1);
> > /* Adjust any no-longer-needed kthreads. */
> > rcu_boost_kthread_setaffinity(rnp, -1);
> > - // Stop-machine done, so allow nohz_full to disable tick.
> > - tick_dep_clear(TICK_DEP_BIT_RCU);
> > return 0;
> > }
> >
> > @@ -4008,8 +4006,6 @@ int rcutree_online_cpu(unsigned int cpu)
> > sync_sched_exp_online_cleanup(cpu);
> > rcutree_affinity_setting(cpu, -1);
> >
> > - // Stop-machine done, so allow nohz_full to disable tick.
> > - tick_dep_clear(TICK_DEP_BIT_RCU);
> > return 0;
> > }
> >
> > @@ -4031,8 +4027,6 @@ int rcutree_offline_cpu(unsigned int cpu)
> >
> > rcutree_affinity_setting(cpu, cpu);
> >
> > - // nohz_full CPUs need the tick for stop-machine to work quickly
> > - tick_dep_set(TICK_DEP_BIT_RCU);
> > return 0;
> > }
> >
> > If the TREE04 can success, then move on to revert the commit(s)
> > identified by Frederic, and do test again.
> >
> > At this time, a TREE04 failure is expected.
> >
> > If the above two results are observed, TICK_DEP_BIT_RCU can be
> > removed.
> >
> > Is my understanding right?
>
> Seems plausible to me, but I again defer to Frederic.
Right, more precisely the commit to revert is:
96926686deab ("rcu: Make CPU-hotplug removal operations enable tick")
which is the diff Pingfan Liu proposed above. The other uses of TICK_DEP_BIT_RCU
will still be needed though.
Thanks.
