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

Groups > linux.kernel > #1417438 > unrolled thread

Re: [GIT PULL arm] Use _rcuidle tracepoints to allow tracing from idle

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [GIT PULL arm] Use _rcuidle tracepoints to allow tracing from  idle Ingo Molnar <mingo@kernel.org> - 2016-06-08 15:40 +0200
    Re: [GIT PULL arm] Use _rcuidle tracepoints to allow tracing from  idle "Paul E. McKenney" <paulmck@linux.vnet.ibm.com> - 2016-06-08 17:20 +0200

#1417438 — Re: [GIT PULL arm] Use _rcuidle tracepoints to allow tracing from idle

* Paul E. McKenney <paulmck@linux.vnet.ibm.com> wrote:

> On Thu, May 26, 2016 at 09:26:06AM -0700, Paul E. McKenney wrote:
> > Hello, Ingo,
> > 
> > This series changes a number of event tracepoints to their _rcuidle() form
> > to allow use from idle without lockdep-RCU complaints, a straightforward
> > modification that has been successfully applied many times.  These splats
> > were found in testing by Guenter Roeck and Tony Lindgren, who have
> > both successfully tested the full series.  Tony asked that I carry these
> > in -rcu:
> > 
> > 	http://lkml.kernel.org/g/20160426213630.GV5995@atomide.com
> > 
> > These have been posted to LKML, CCing relevant maintainers, who have
> > not objected to the proposed changes:
> > 
> > 	http://lkml.kernel.org/g/20160516184844.GA20144@linux.vnet.ibm.com
> > 
> > These maintainers are also CCed on this pull request.
> > 
> > They have also been subjected to 0day Test Robot and -next testing.
> 
> And this time actually including the pull request...  :-/
> 
> 							Thanx, Paul
> 
> These changes are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git for-mingo
> 
> for you to fetch changes up to e7c38dda94b23965a3eb46ef4656bb8cb921933d:
> 
>   arm: Use _rcuidle suffix to allow clk_core_enable() to used from idle (2016-05-18 11:55:29 -0700)
> 
> ----------------------------------------------------------------
> Paul E. McKenney (6):
>       arm: Use _rcuidle tracepoint to allow use from idle
>       arm: Use _rcuidle for suspend/resume tracepoints
>       arm: Add _rcuidle tracepoints to allow clk_core_disable() use from idle
>       arm: Add _rcuidle suffix to allow rpm_idle() use from idle
>       arm: Add _rcuidle suffix to allow rpm_resume() to be called from idle
>       arm: Use _rcuidle suffix to allow clk_core_enable() to used from idle
> 
>  arch/arm/kernel/smp.c             |  2 +-
>  arch/arm/mach-omap2/powerdomain.c |  9 +++++----
>  drivers/base/power/runtime.c      | 14 +++++++-------
>  drivers/clk/clk.c                 |  8 ++++----
>  4 files changed, 17 insertions(+), 16 deletions(-)

Just wondering, wouldn't it be better for Russell to pull/apply, as it appears to 
be a mostly ARM patch-set?

Thanks,

	Ingo

[toc] | [next] | [standalone]

#1417569

On Wed, Jun 08, 2016 at 03:33:50PM +0200, Ingo Molnar wrote:
> 
> * Paul E. McKenney <paulmck@linux.vnet.ibm.com> wrote:
> 
> > On Thu, May 26, 2016 at 09:26:06AM -0700, Paul E. McKenney wrote:
> > > Hello, Ingo,
> > > 
> > > This series changes a number of event tracepoints to their _rcuidle() form
> > > to allow use from idle without lockdep-RCU complaints, a straightforward
> > > modification that has been successfully applied many times.  These splats
> > > were found in testing by Guenter Roeck and Tony Lindgren, who have
> > > both successfully tested the full series.  Tony asked that I carry these
> > > in -rcu:
> > > 
> > > 	http://lkml.kernel.org/g/20160426213630.GV5995@atomide.com
> > > 
> > > These have been posted to LKML, CCing relevant maintainers, who have
> > > not objected to the proposed changes:
> > > 
> > > 	http://lkml.kernel.org/g/20160516184844.GA20144@linux.vnet.ibm.com
> > > 
> > > These maintainers are also CCed on this pull request.
> > > 
> > > They have also been subjected to 0day Test Robot and -next testing.
> > 
> > And this time actually including the pull request...  :-/
> > 
> > 							Thanx, Paul
> > 
> > These changes are available in the git repository at:
> > 
> >   git://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git for-mingo
> > 
> > for you to fetch changes up to e7c38dda94b23965a3eb46ef4656bb8cb921933d:
> > 
> >   arm: Use _rcuidle suffix to allow clk_core_enable() to used from idle (2016-05-18 11:55:29 -0700)
> > 
> > ----------------------------------------------------------------
> > Paul E. McKenney (6):
> >       arm: Use _rcuidle tracepoint to allow use from idle
> >       arm: Use _rcuidle for suspend/resume tracepoints
> >       arm: Add _rcuidle tracepoints to allow clk_core_disable() use from idle
> >       arm: Add _rcuidle suffix to allow rpm_idle() use from idle
> >       arm: Add _rcuidle suffix to allow rpm_resume() to be called from idle
> >       arm: Use _rcuidle suffix to allow clk_core_enable() to used from idle
> > 
> >  arch/arm/kernel/smp.c             |  2 +-
> >  arch/arm/mach-omap2/powerdomain.c |  9 +++++----
> >  drivers/base/power/runtime.c      | 14 +++++++-------
> >  drivers/clk/clk.c                 |  8 ++++----
> >  4 files changed, 17 insertions(+), 16 deletions(-)
> 
> Just wondering, wouldn't it be better for Russell to pull/apply, as it appears to 
> be a mostly ARM patch-set?

Fair enough -- I can always repost the patches to their respective
maintainers if you are in any way uncomfortable taking them.  The patches
do not depend on each other.

							Thanx, Paul

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web