Date: Wed, 5 Oct 2016 07:03:07 -0700
From: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
To: Nikolay Borisov <kernel@kyup.com>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [PATCH] rcu: Reword help of RCU_TRACE option
Reply-To: paulmck@linux.vnet.ibm.com
References: <1475651181-5512-1-git-send-email-kernel@kyup.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1475651181-5512-1-git-send-email-kernel@kyup.com>
User-Agent: Mutt/1.5.21 (2010-09-15)
Message-Id: <20161005140307.GE3640@linux.vnet.ibm.com>
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1581
Lines: 49

On Wed, Oct 05, 2016 at 10:06:21AM +0300, Nikolay Borisov wrote:
> Explicitly state that enabling RCU_TRACE enables more
> tracepoints and not just "additional tracing".
> 
> Signed-off-by: Nikolay Borisov <kernel@kyup.com>
> ---
> 
> Hello Paul, 
> 
> Following our latest conversation re. enabling RCU tracing
> I had to actually go and look into the code to see which 
> option enables the additional tracepoints. I wasn't sure
> while reading the help of TRACE_RCU. So here is a patch
> which explicitly states that this enables additional 
> tracepoints. 
> 
> Apply it if you deem necessary it might be just me that 
> thinks the description is not verbose enough. 
> 
>  lib/Kconfig.debug | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
> index cab7405f48d2..f8c3468ec7ef 100644
> --- a/lib/Kconfig.debug
> +++ b/lib/Kconfig.debug
> @@ -1429,8 +1429,8 @@ config RCU_TRACE
>  	depends on DEBUG_KERNEL
>  	select TRACE_CLOCK
>  	help
> -	  This option provides tracing in RCU which presents stats
> -	  in debugfs for debugging RCU implementation.
> +	  This option enables additional tracepoints in RCU which present
> +	  stats in debugfs for debugging RCU implementation.

How about something like this?

	This option provides tracing in RCU which presents stats
	in debugfs for debugging RCU implementation.  It also enables
	additional tracepoints for ftrace-style event tracing.

							Thanx, Paul

> 
>  	  Say Y here if you want to enable RCU tracing
>  	  Say N if you are unsure.
> -- 
> 2.5.0
>