Re: [PATCH] ARM: OMAP2+: PM: Denote the cpuidle tracepoints as _rcuidle()

2015-10-12 Thread Tony Lindgren 
* Jisheng Zhang  [150923 20:26]:
> Dear Tony,
> 
> On Wed, 23 Sep 2015 11:08:15 -0700
> Tony Lindgren  wrote:
> 
> > * Kevin Hilman  [150923 11:03]:
> > > Jisheng Zhang  writes:
> > > 
> > > > The cpuidle tracepoints are called within a rcu_idle_exit() section, and
> > > > must be denoted with the _rcuidle() version of the tracepoint.
> > > >
> > > > Signed-off-by: Jisheng Zhang 
> > >  
> > > Acked-by: Kevin Hilman 
> > 
> > Hmm is this needed as a fix for the -rc cycle or can this wait
> > for v4.4?
> 
> IMHO, this is a fix. But it can wait for v4.4 merge window.

Applying into omap-for-v4.4/fixes thanks.

Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] ARM: OMAP2+: PM: Denote the cpuidle tracepoints as _rcuidle()

2015-09-23 Thread Kevin Hilman 
Jisheng Zhang  writes:

> The cpuidle tracepoints are called within a rcu_idle_exit() section, and
> must be denoted with the _rcuidle() version of the tracepoint.
>
> Signed-off-by: Jisheng Zhang 
 
Acked-by: Kevin Hilman 
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] ARM: OMAP2+: PM: Denote the cpuidle tracepoints as _rcuidle()

2015-09-23 Thread Tony Lindgren 
* Kevin Hilman  [150923 11:03]:
> Jisheng Zhang  writes:
> 
> > The cpuidle tracepoints are called within a rcu_idle_exit() section, and
> > must be denoted with the _rcuidle() version of the tracepoint.
> >
> > Signed-off-by: Jisheng Zhang 
>  
> Acked-by: Kevin Hilman 

Hmm is this needed as a fix for the -rc cycle or can this wait
for v4.4?

Regards,

Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] ARM: OMAP2+: PM: Denote the cpuidle tracepoints as _rcuidle()

2015-09-23 Thread Jisheng Zhang 
Dear Tony,

On Wed, 23 Sep 2015 11:08:15 -0700
Tony Lindgren  wrote:

> * Kevin Hilman  [150923 11:03]:
> > Jisheng Zhang  writes:
> > 
> > > The cpuidle tracepoints are called within a rcu_idle_exit() section, and
> > > must be denoted with the _rcuidle() version of the tracepoint.
> > >
> > > Signed-off-by: Jisheng Zhang 
> >  
> > Acked-by: Kevin Hilman 
> 
> Hmm is this needed as a fix for the -rc cycle or can this wait
> for v4.4?

IMHO, this is a fix. But it can wait for v4.4 merge window.

Thanks,
Jisheng

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH] ARM: OMAP2+: PM: Denote the cpuidle tracepoints as _rcuidle()

2015-09-17 Thread Jisheng Zhang 
The cpuidle tracepoints are called within a rcu_idle_exit() section, and
must be denoted with the _rcuidle() version of the tracepoint.

Signed-off-by: Jisheng Zhang 
---
 arch/arm/mach-omap2/pm34xx.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/mach-omap2/pm34xx.c b/arch/arm/mach-omap2/pm34xx.c
index 87b98bf9..2dbd378 100644
--- a/arch/arm/mach-omap2/pm34xx.c
+++ b/arch/arm/mach-omap2/pm34xx.c
@@ -301,11 +301,11 @@ static void omap3_pm_idle(void)
if (omap_irq_pending())
return;
 
-   trace_cpu_idle(1, smp_processor_id());
+   trace_cpu_idle_rcuidle(1, smp_processor_id());
 
omap_sram_idle();
 
-   trace_cpu_idle(PWR_EVENT_EXIT, smp_processor_id());
+   trace_cpu_idle_rcuidle(PWR_EVENT_EXIT, smp_processor_id());
 }
 
 #ifdef CONFIG_SUSPEND
-- 
2.5.1

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html