From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S934093AbbDVOaS (ORCPT <rfc822;w@1wt.eu>);
	Wed, 22 Apr 2015 10:30:18 -0400
Received: from e39.co.us.ibm.com ([32.97.110.160]:41934 "EHLO
	e39.co.us.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S933219AbbDVOaL (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Wed, 22 Apr 2015 10:30:11 -0400
Date: Wed, 22 Apr 2015 07:30:03 -0700
From: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
To: Pranith Kumar <bobby.prani@gmail.com>
Cc: LKML <linux-kernel@vger.kernel.org>, Ingo Molnar <mingo@kernel.org>,
        Lai Jiangshan <laijs@cn.fujitsu.com>,
        Dipankar Sarma <dipankar@in.ibm.com>,
        Andrew Morton <akpm@linux-foundation.org>,
        Mathieu Desnoyers <mathieu.desnoyers@efficios.com>,
        Josh Triplett <josh@joshtriplett.org>,
        Thomas Gleixner <tglx@linutronix.de>,
        Peter Zijlstra <peterz@infradead.org>,
        Steven Rostedt <rostedt@goodmis.org>,
        David Howells <dhowells@redhat.com>,
        Eric Dumazet <edumazet@google.com>,
        Darren Hart <dvhart@linux.intel.com>,
        =?iso-8859-1?Q?Fr=E9d=E9ric?= Weisbecker <fweisbec@gmail.com>,
        Oleg Nesterov <oleg@redhat.com>
Subject: Re: [PATCH tip/core/rcu 09/12] rcu: Break dependency of
 RCU_FANOUT_LEAF on RCU_FANOUT
Message-ID: <20150422143003.GX5561@linux.vnet.ibm.com>
Reply-To: paulmck@linux.vnet.ibm.com
References: <20150421195529.GA17864@linux.vnet.ibm.com>
 <1429646153-20677-1-git-send-email-paulmck@linux.vnet.ibm.com>
 <1429646153-20677-9-git-send-email-paulmck@linux.vnet.ibm.com>
 <CAJhHMCC9f76-33VAQH11gDenBo7uukswAN2SrCPKUfxLXewdhg@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <CAJhHMCC9f76-33VAQH11gDenBo7uukswAN2SrCPKUfxLXewdhg@mail.gmail.com>
User-Agent: Mutt/1.5.21 (2010-09-15)
X-TM-AS-MML: disable
X-Content-Scanned: Fidelis XPS MAILER
x-cbid: 15042214-0033-0000-0000-0000044FB0CB
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

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@linux.vnet.ibm.com> wrote:
> > From: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
> >
> > 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@linux.vnet.ibm.com>
> > ---
> >  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