Re: [PATCH tip/core/rcu 09/12] rcu: Break dependency of RCU_FANOUT_LEAF on RCU_FANOUT

[Paul E. McKenney]

On Wed, Apr 22, 2015 at 01:22:23AM -0400, Pranith Kumar wrote:
> On Tue, Apr 21, 2015 at 3:55 PM, Paul E. McKenney
> <paulmck@xxxxxxxxxxxxxxxxxx> wrote:
> > From: "Paul E. McKenney" <paulmck@xxxxxxxxxxxxxxxxxx>
> >
> > RCU_FANOUT_LEAF's range and default values depend on the value of
> > RCU_FANOUT, which at the time seemed like a cute way to save two lines
> > of Kconfig code.  However, adding a dependency from both of these
> > Kconfig parameters on RCU_EXPERT requires that RCU_FANOUT_LEAF operate
> > correctly even if RCU_FANOUT is undefined.  This commit therefore
> > allows RCU_FANOUT_LEAF to take on the full range of permitted values,
> > even in cases where RCU_FANOUT is undefined.
> >
> > Signed-off-by: Paul E. McKenney <paulmck@xxxxxxxxxxxxxxxxxx>
> > ---
> >  init/Kconfig | 6 ++++--
> >  1 file changed, 4 insertions(+), 2 deletions(-)
> >
> > diff --git a/init/Kconfig b/init/Kconfig
> > index aa68d72fe051..35559e07db18 100644
> > --- a/init/Kconfig
> > +++ b/init/Kconfig
> > @@ -599,9 +599,11 @@ config RCU_FANOUT
> >
> >  config RCU_FANOUT_LEAF
> >         int "Tree-based hierarchical RCU leaf-level fanout value"
> > -       range 2 RCU_FANOUT if 64BIT
> > -       range 2 RCU_FANOUT if !64BIT
> > +       range 2 64 if 64BIT
> > +       range 2 32 if !64BIT
> >         depends on TREE_RCU || PREEMPT_RCU
> > +       default 64 if 64BIT
> > +       default 32 if !64BIT
> >         default 16
> 
> What is the effect of this 'default 16' line? It is either 64 or 32
> based on 64BIT, not sure when it is 16...

Good catch, fixes to delete the two added "default" statements.
The default thus is now 16, as it was before.

							Thanx, Paul

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/