Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Tony Lindgren
* Paul E. McKenney [160426 14:27]: > Done! These are at -rcu: > > 6eeec2b4a50c (arm: Use _rcuidle tracepoint to allow use from idle) > 0efee3da3b29 (arm: Use more _rcuidle tracepoints to allow use from idle) > b6cdf6d22f52 (arm: Additional _rcuidle tracepoints to allow use from idle) > 70da6fb72

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Paul E. McKenney
On Tue, Apr 26, 2016 at 02:12:59PM -0700, Tony Lindgren wrote: > * Paul E. McKenney [160426 14:08]: > > On Tue, Apr 26, 2016 at 01:46:15PM -0700, Tony Lindgren wrote: > > > * Paul E. McKenney [160426 13:42]: > > > > On Tue, Apr 26, 2016 at 01:25:39PM -0700, Tony Lindgren wrote: > > > > > * Paul E

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Tony Lindgren
* Paul E. McKenney [160426 14:08]: > On Tue, Apr 26, 2016 at 01:46:15PM -0700, Tony Lindgren wrote: > > * Paul E. McKenney [160426 13:42]: > > > On Tue, Apr 26, 2016 at 01:25:39PM -0700, Tony Lindgren wrote: > > > > * Paul E. McKenney [160426 13:12]: > > > > > On Tue, Apr 26, 2016 at 01:06:13PM

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Paul E. McKenney
On Tue, Apr 26, 2016 at 01:46:15PM -0700, Tony Lindgren wrote: > * Paul E. McKenney [160426 13:42]: > > On Tue, Apr 26, 2016 at 01:25:39PM -0700, Tony Lindgren wrote: > > > * Paul E. McKenney [160426 13:12]: > > > > On Tue, Apr 26, 2016 at 01:06:13PM -0700, Paul E. McKenney wrote: > > > > > On Tu

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Paul E. McKenney
On Tue, Apr 26, 2016 at 04:52:17PM -0400, Steven Rostedt wrote: > On Tue, 26 Apr 2016 13:46:15 -0700 > Tony Lindgren wrote: > > > > [ INFO: suspicious RCU usage. ] > > 4.6.0-rc5-next-20160426+ #1127 Not tainted > > --- > > include/trace/events/clk.h:45 suspicious rcu_

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Steven Rostedt
On Tue, 26 Apr 2016 13:46:15 -0700 Tony Lindgren wrote: > [ INFO: suspicious RCU usage. ] > 4.6.0-rc5-next-20160426+ #1127 Not tainted > --- > include/trace/events/clk.h:45 suspicious rcu_dereference_check() usage! > > other info that might help us debug this: > >

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Tony Lindgren
* Paul E. McKenney [160426 13:42]: > On Tue, Apr 26, 2016 at 01:25:39PM -0700, Tony Lindgren wrote: > > * Paul E. McKenney [160426 13:12]: > > > On Tue, Apr 26, 2016 at 01:06:13PM -0700, Paul E. McKenney wrote: > > > > On Tue, Apr 26, 2016 at 12:51:59PM -0700, Tony Lindgren wrote: > > > > > * Ste

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Paul E. McKenney
On Tue, Apr 26, 2016 at 01:25:39PM -0700, Tony Lindgren wrote: > * Paul E. McKenney [160426 13:12]: > > On Tue, Apr 26, 2016 at 01:06:13PM -0700, Paul E. McKenney wrote: > > > On Tue, Apr 26, 2016 at 12:51:59PM -0700, Tony Lindgren wrote: > > > > * Steven Rostedt [160426 12:45]: > > > > > *Whack*

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Paul E. McKenney
On Tue, Apr 26, 2016 at 04:16:21PM -0400, Steven Rostedt wrote: > On Tue, 26 Apr 2016 13:06:13 -0700 > "Paul E. McKenney" wrote: > > > On Tue, Apr 26, 2016 at 12:51:59PM -0700, Tony Lindgren wrote: > > > * Steven Rostedt [160426 12:45]: > > > > *Whack* *Whack* *Whack*!!! > > > > > > > > Signe

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Tony Lindgren
* Paul E. McKenney [160426 13:12]: > On Tue, Apr 26, 2016 at 01:06:13PM -0700, Paul E. McKenney wrote: > > On Tue, Apr 26, 2016 at 12:51:59PM -0700, Tony Lindgren wrote: > > > * Steven Rostedt [160426 12:45]: > > > > *Whack* *Whack* *Whack*!!! > > > > > > > > Signed-off-by: Steven Rostedt > > >

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Steven Rostedt
On Tue, 26 Apr 2016 13:06:13 -0700 "Paul E. McKenney" wrote: > On Tue, Apr 26, 2016 at 12:51:59PM -0700, Tony Lindgren wrote: > > * Steven Rostedt [160426 12:45]: > > > *Whack* *Whack* *Whack*!!! > > > > > > Signed-off-by: Steven Rostedt > > > --- > > > diff --git a/drivers/clk/clk.c b/drive

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Paul E. McKenney
On Tue, Apr 26, 2016 at 01:06:13PM -0700, Paul E. McKenney wrote: > On Tue, Apr 26, 2016 at 12:51:59PM -0700, Tony Lindgren wrote: > > * Steven Rostedt [160426 12:45]: > > > *Whack* *Whack* *Whack*!!! > > > > > > Signed-off-by: Steven Rostedt > > > --- > > > diff --git a/drivers/clk/clk.c b/driv

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Paul E. McKenney
On Tue, Apr 26, 2016 at 03:52:29PM -0400, Steven Rostedt wrote: > On Tue, 26 Apr 2016 12:51:21 -0700 > "Paul E. McKenney" wrote: > > > On Tue, Apr 26, 2016 at 03:44:14PM -0400, Steven Rostedt wrote: > > > On Tue, 26 Apr 2016 12:39:50 -0700 > > > Tony Lindgren wrote: > > > > > > > > > > With

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Paul E. McKenney
On Tue, Apr 26, 2016 at 12:51:59PM -0700, Tony Lindgren wrote: > * Steven Rostedt [160426 12:45]: > > *Whack* *Whack* *Whack*!!! > > > > Signed-off-by: Steven Rostedt > > --- > > diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c > > index fb74dc1f7520..4fa43c02d682 100644 > > --- a/drivers/clk/

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Tony Lindgren
* Steven Rostedt [160426 12:45]: > *Whack* *Whack* *Whack*!!! > > Signed-off-by: Steven Rostedt > --- > diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c > index fb74dc1f7520..4fa43c02d682 100644 > --- a/drivers/clk/clk.c > +++ b/drivers/clk/clk.c > @@ -682,12 +682,12 @@ static void clk_core_di

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Paul E. McKenney
On Tue, Apr 26, 2016 at 03:44:14PM -0400, Steven Rostedt wrote: > On Tue, 26 Apr 2016 12:39:50 -0700 > Tony Lindgren wrote: > > > > With these omap4 is now OK. But omap3 produces yet another warning, > > See below :) > > > > Regards, > > > > Tony > > > > 8< -- > >

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Steven Rostedt
On Tue, 26 Apr 2016 12:51:21 -0700 "Paul E. McKenney" wrote: > On Tue, Apr 26, 2016 at 03:44:14PM -0400, Steven Rostedt wrote: > > On Tue, 26 Apr 2016 12:39:50 -0700 > > Tony Lindgren wrote: > > > > > > > With these omap4 is now OK. But omap3 produces yet another warning, > > > See below :)

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Steven Rostedt
On Tue, 26 Apr 2016 12:39:50 -0700 Tony Lindgren wrote: > With these omap4 is now OK. But omap3 produces yet another warning, > See below :) > > Regards, > > Tony > > 8< -- > === > [ INFO: suspicious RCU usage. ] > 4.6.0-rc5-next-20160426+ #1114 Not

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Steven Rostedt
On Tue, 26 Apr 2016 12:42:15 -0700 "Paul E. McKenney" wrote: > Given the similarity with the following: > > http://lkml.kernel.org/g/20160425171239.ge3...@linux.vnet.ibm.com > > May I apply your Reviewed-by? > Yep. -- Steve

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Paul E. McKenney
On Tue, Apr 26, 2016 at 12:39:50PM -0700, Tony Lindgren wrote: > * Paul E. McKenney [160426 12:08]: > > On Tue, Apr 26, 2016 at 11:29:39AM -0700, Tony Lindgren wrote: > > > * Paul E. McKenney [160426 10:53]: > > > > Does the following patch help? > > > > > > It just changes the output.. See belo

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Paul E. McKenney
On Tue, Apr 26, 2016 at 03:17:25PM -0400, Steven Rostedt wrote: > On Tue, 26 Apr 2016 11:29:39 -0700 > Tony Lindgren wrote: > > > * Paul E. McKenney [160426 10:53]: > > > Does the following patch help? > > > > It just changes the output.. See below. > > > > > It is quite possible that there

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Tony Lindgren
* Paul E. McKenney [160426 12:08]: > On Tue, Apr 26, 2016 at 11:29:39AM -0700, Tony Lindgren wrote: > > * Paul E. McKenney [160426 10:53]: > > > Does the following patch help? > > > > It just changes the output.. See below. > > Ah -- you need both patches. Oops sorry, I though I had the first

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Tony Lindgren
* Steven Rostedt [160426 12:18]: > Time to play "Whack-a-Mole"(TM) :) > Signed-off-by: Steven Rostedt > --- > diff --git a/arch/arm/mach-omap2/powerdomain.c > b/arch/arm/mach-omap2/powerdomain.c > index 78af6d8cf2e2..12b66b5bcc55 100644 > --- a/arch/arm/mach-omap2/powerdomain.c > +++ b/arch/ar

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Steven Rostedt
On Tue, 26 Apr 2016 11:29:39 -0700 Tony Lindgren wrote: > * Paul E. McKenney [160426 10:53]: > > Does the following patch help? > > It just changes the output.. See below. > > > It is quite possible that there are quite a few more of these. If this > > is the case, then one way to make the

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Paul E. McKenney
On Tue, Apr 26, 2016 at 11:29:39AM -0700, Tony Lindgren wrote: > * Paul E. McKenney [160426 10:53]: > > Does the following patch help? > > It just changes the output.. See below. Ah -- you need both patches. These are on -rcu at: 501889db6fee (arm: Use _rcuidle tracepoint to allow use from idl

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Tony Lindgren
* Paul E. McKenney [160426 10:53]: > Does the following patch help? It just changes the output.. See below. > It is quite possible that there are quite a few more of these. If this > is the case, then one way to make the kernel list more of them on a > given boot is to build with CONFIG_PROVE_R

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Paul E. McKenney
On Tue, Apr 26, 2016 at 08:00:31AM -0700, Tony Lindgren wrote: > Hi, > > * Paul E. McKenney [160425 13:49]: > > Testing on ARM encountered the following pair of lockdep-RCU splats: > > > > > > > > =

Re: [PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-26 Thread Tony Lindgren
Hi, * Paul E. McKenney [160425 13:49]: > Testing on ARM encountered the following pair of lockdep-RCU splats: > > > > === > [ INFO: suspicious RCU usage. ] > 4.6.0-rc4-next-20160422 #1 Not taint

[PATCH arm 1/1] arm: Use _rcuidle tracepoint to allow use from idle

2016-04-25 Thread Paul E. McKenney
Testing on ARM encountered the following pair of lockdep-RCU splats: === [ INFO: suspicious RCU usage. ] 4.6.0-rc4-next-20160422 #1 Not tainted --- include/trace/events