* [PATCH] rcu: Reword help of RCU_TRACE option
@ 2016-10-05 7:06 Nikolay Borisov
2016-10-05 14:03 ` Paul E. McKenney
0 siblings, 1 reply; 5+ messages in thread
From: Nikolay Borisov @ 2016-10-05 7:06 UTC (permalink / raw)
To: paulmck; +Cc: linux-kernel, Nikolay Borisov
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.
Say Y here if you want to enable RCU tracing
Say N if you are unsure.
--
2.5.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] rcu: Reword help of RCU_TRACE option
2016-10-05 7:06 [PATCH] rcu: Reword help of RCU_TRACE option Nikolay Borisov
@ 2016-10-05 14:03 ` Paul E. McKenney
2016-10-05 14:18 ` Nikolay Borisov
0 siblings, 1 reply; 5+ messages in thread
From: Paul E. McKenney @ 2016-10-05 14:03 UTC (permalink / raw)
To: Nikolay Borisov; +Cc: linux-kernel
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
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] rcu: Reword help of RCU_TRACE option
2016-10-05 14:03 ` Paul E. McKenney
@ 2016-10-05 14:18 ` Nikolay Borisov
2016-10-06 12:45 ` Paul E. McKenney
0 siblings, 1 reply; 5+ messages in thread
From: Nikolay Borisov @ 2016-10-05 14:18 UTC (permalink / raw)
To: paulmck; +Cc: linux-kernel
On 10/05/2016 05:03 PM, Paul E. McKenney wrote:
> 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.
So long as it's explicitly stated that it also enables tracepoints it's
all fine by me. Apparently I'm not the only one who got confused. In a
question re. CONFIG_RCU_TRACE here is what someone said on IRC:
"what's tracepoints do you mean? this option enables the additional
statistics permanently."
Care to apply it with the revised wording or shall I resend?
>
> Thanx, Paul
>
>>
>> Say Y here if you want to enable RCU tracing
>> Say N if you are unsure.
>> --
>> 2.5.0
>>
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] rcu: Reword help of RCU_TRACE option
2016-10-05 14:18 ` Nikolay Borisov
@ 2016-10-06 12:45 ` Paul E. McKenney
2016-10-06 12:51 ` Nikolay Borisov
0 siblings, 1 reply; 5+ messages in thread
From: Paul E. McKenney @ 2016-10-06 12:45 UTC (permalink / raw)
To: Nikolay Borisov; +Cc: linux-kernel
On Wed, Oct 05, 2016 at 05:18:09PM +0300, Nikolay Borisov wrote:
>
>
> On 10/05/2016 05:03 PM, Paul E. McKenney wrote:
> > 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.
>
> So long as it's explicitly stated that it also enables tracepoints it's
> all fine by me. Apparently I'm not the only one who got confused. In a
> question re. CONFIG_RCU_TRACE here is what someone said on IRC:
>
> "what's tracepoints do you mean? this option enables the additional
> statistics permanently."
>
> Care to apply it with the revised wording or shall I resend?
Please see below for the current state.
Thanx, Paul
------------------------------------------------------------------------
commit a30cd69bdcb1ae12cb243e75eddd608c1798d438
Author: Nikolay Borisov <kernel@kyup.com>
Date: Thu Oct 6 05:42:39 2016 -0700
rcu: RCU_TRACE enables event tracing as well as debugfs
The commit brings the RCU_TRACE Kconfig option's help text up to date
by noting that it enables additional event tracing as well as debugfs.
Signed-off-by: Nikolay Borisov <kernel@kyup.com>
[ paulmck: Do some wordsmithing. ]
Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
index 2b5441812214..7f8e8f3c72ea 100644
--- a/lib/Kconfig.debug
+++ b/lib/Kconfig.debug
@@ -1450,7 +1450,8 @@ config RCU_TRACE
select TRACE_CLOCK
help
This option provides tracing in RCU which presents stats
- in debugfs for debugging RCU implementation.
+ in debugfs for debugging RCU implementation. It also enables
+ additional tracepoints for ftrace-style event tracing.
Say Y here if you want to enable RCU tracing
Say N if you are unsure.
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH] rcu: Reword help of RCU_TRACE option
2016-10-06 12:45 ` Paul E. McKenney
@ 2016-10-06 12:51 ` Nikolay Borisov
0 siblings, 0 replies; 5+ messages in thread
From: Nikolay Borisov @ 2016-10-06 12:51 UTC (permalink / raw)
To: paulmck; +Cc: linux-kernel
On 10/06/2016 03:45 PM, Paul E. McKenney wrote:
> On Wed, Oct 05, 2016 at 05:18:09PM +0300, Nikolay Borisov wrote:
>>
>>
>> On 10/05/2016 05:03 PM, Paul E. McKenney wrote:
>>> 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.
>>
>> So long as it's explicitly stated that it also enables tracepoints it's
>> all fine by me. Apparently I'm not the only one who got confused. In a
>> question re. CONFIG_RCU_TRACE here is what someone said on IRC:
>>
>> "what's tracepoints do you mean? this option enables the additional
>> statistics permanently."
>>
>> Care to apply it with the revised wording or shall I resend?
>
> Please see below for the current state.
>
> Thanx, Paul
Splendid, ACK.
>
> ------------------------------------------------------------------------
>
> commit a30cd69bdcb1ae12cb243e75eddd608c1798d438
> Author: Nikolay Borisov <kernel@kyup.com>
> Date: Thu Oct 6 05:42:39 2016 -0700
>
> rcu: RCU_TRACE enables event tracing as well as debugfs
>
> The commit brings the RCU_TRACE Kconfig option's help text up to date
> by noting that it enables additional event tracing as well as debugfs.
>
> Signed-off-by: Nikolay Borisov <kernel@kyup.com>
> [ paulmck: Do some wordsmithing. ]
> Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
>
> diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
> index 2b5441812214..7f8e8f3c72ea 100644
> --- a/lib/Kconfig.debug
> +++ b/lib/Kconfig.debug
> @@ -1450,7 +1450,8 @@ config RCU_TRACE
> select TRACE_CLOCK
> help
> This option provides tracing in RCU which presents stats
> - in debugfs for debugging RCU implementation.
> + in debugfs for debugging RCU implementation. It also enables
> + additional tracepoints for ftrace-style event tracing.
>
> Say Y here if you want to enable RCU tracing
> Say N if you are unsure.
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-10-06 12:51 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-10-05 7:06 [PATCH] rcu: Reword help of RCU_TRACE option Nikolay Borisov
2016-10-05 14:03 ` Paul E. McKenney
2016-10-05 14:18 ` Nikolay Borisov
2016-10-06 12:45 ` Paul E. McKenney
2016-10-06 12:51 ` Nikolay Borisov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox