Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1495891

Re: [PATCH] rcu: Reword help of RCU_TRACE option

From "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
Newsgroups linux.kernel
Subject Re: [PATCH] rcu: Reword help of RCU_TRACE option
Date 2016-10-05 16:10 +0200
Message-ID <soV3s-209-23@gated-at.bofh.it> (permalink)
References <soOuZ-5Wx-3@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


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
> 

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH] rcu: Reword help of RCU_TRACE option Nikolay Borisov <kernel@kyup.com> - 2016-10-05 09:10 +0200
  Re: [PATCH] rcu: Reword help of RCU_TRACE option "Paul E. McKenney" <paulmck@linux.vnet.ibm.com> - 2016-10-05 16:10 +0200
    Re: [PATCH] rcu: Reword help of RCU_TRACE option Nikolay Borisov <kernel@kyup.com> - 2016-10-05 16:20 +0200
      Re: [PATCH] rcu: Reword help of RCU_TRACE option "Paul E. McKenney" <paulmck@linux.vnet.ibm.com> - 2016-10-06 14:50 +0200
        Re: [PATCH] rcu: Reword help of RCU_TRACE option Nikolay Borisov <kernel@kyup.com> - 2016-10-06 15:00 +0200

csiph-web