Re: [PATCH] arm: Set hardirq tracing to on when idling

2014-05-28 Thread Arnd Bergmann
On Tuesday 27 May 2014 19:28:06 Corey Minyard wrote: > On 05/27/2014 02:27 PM, Arnd Bergmann wrote: > > On Tuesday 27 May 2014 11:53:59 Stephen Boyd wrote: > >> On 05/27/14 11:49, Arnd Bergmann wrote: > >>> You also commented in that thread about stop_critical_timings()/ > >>>

Re: [PATCH] arm: Set hardirq tracing to on when idling

2014-05-28 Thread Arnd Bergmann
On Tuesday 27 May 2014 19:28:06 Corey Minyard wrote: On 05/27/2014 02:27 PM, Arnd Bergmann wrote: On Tuesday 27 May 2014 11:53:59 Stephen Boyd wrote: On 05/27/14 11:49, Arnd Bergmann wrote: You also commented in that thread about stop_critical_timings()/ start_critical_timings(). Corey,

Re: [PATCH] arm: Set hardirq tracing to on when idling

2014-05-27 Thread Corey Minyard
On 05/27/2014 02:27 PM, Arnd Bergmann wrote: > On Tuesday 27 May 2014 11:53:59 Stephen Boyd wrote: >> On 05/27/14 11:49, Arnd Bergmann wrote: >>> You also commented in that thread about stop_critical_timings()/ >>> start_critical_timings(). Corey, can you look at that, too? I >>> think it's

Re: [PATCH] arm: Set hardirq tracing to on when idling

2014-05-27 Thread Stephen Boyd
On 05/27/14 12:39, Arnd Bergmann wrote: > On Tuesday 27 May 2014 12:33:38 Stephen Boyd wrote: >> On 05/27/14 12:27, Arnd Bergmann wrote: >>> On Tuesday 27 May 2014 11:53:59 Stephen Boyd wrote: On 05/27/14 11:49, Arnd Bergmann wrote: > You also commented in that thread about

Re: [PATCH] arm: Set hardirq tracing to on when idling

2014-05-27 Thread Arnd Bergmann
On Tuesday 27 May 2014 12:33:38 Stephen Boyd wrote: > On 05/27/14 12:27, Arnd Bergmann wrote: > > On Tuesday 27 May 2014 11:53:59 Stephen Boyd wrote: > >> On 05/27/14 11:49, Arnd Bergmann wrote: > >>> You also commented in that thread about stop_critical_timings()/ > >>> start_critical_timings().

Re: [PATCH] arm: Set hardirq tracing to on when idling

2014-05-27 Thread Stephen Boyd
On 05/27/14 12:27, Arnd Bergmann wrote: > On Tuesday 27 May 2014 11:53:59 Stephen Boyd wrote: >> On 05/27/14 11:49, Arnd Bergmann wrote: >>> You also commented in that thread about stop_critical_timings()/ >>> start_critical_timings(). Corey, can you look at that, too? I >>> think it's designed to

Re: [PATCH] arm: Set hardirq tracing to on when idling

2014-05-27 Thread Arnd Bergmann
On Tuesday 27 May 2014 11:53:59 Stephen Boyd wrote: > On 05/27/14 11:49, Arnd Bergmann wrote: > > > > You also commented in that thread about stop_critical_timings()/ > > start_critical_timings(). Corey, can you look at that, too? I > > think it's designed to avoid the issue you are seeing but > >

Re: [PATCH] arm: Set hardirq tracing to on when idling

2014-05-27 Thread Stephen Boyd
On 05/27/14 11:49, Arnd Bergmann wrote: > > You also commented in that thread about stop_critical_timings()/ > start_critical_timings(). Corey, can you look at that, too? I > think it's designed to avoid the issue you are seeing but > for some reason doesn't. I sent a patch last week to "solve"

Re: [PATCH] arm: Set hardirq tracing to on when idling

2014-05-27 Thread Corey Minyard
On 05/27/2014 11:16 AM, Arnd Bergmann wrote: > On Tuesday 27 May 2014, Corey Minyard wrote: >> On 05/26/2014 04:26 AM, Arnd Bergmann wrote: >>> On Sunday 25 May 2014 14:15:23 miny...@acm.org wrote: From: Corey Minyard The CPU will go to idle with interrupts off, but the interrupts

Re: [PATCH] arm: Set hardirq tracing to on when idling

2014-05-27 Thread Arnd Bergmann
On Tuesday 27 May 2014 18:38:33 Stanislav Meduna wrote: > On 26.05.2014 11:26, Arnd Bergmann wrote: > > > It seems like the right thing to do, I just don't understand > > why nobody hit this before. > > Looks like this is what I did hit a month ago and > was not able to find the culprit: >

Re: [PATCH] arm: Set hardirq tracing to on when idling

2014-05-27 Thread Stanislav Meduna
On 26.05.2014 11:26, Arnd Bergmann wrote: > It seems like the right thing to do, I just don't understand > why nobody hit this before. Looks like this is what I did hit a month ago and was not able to find the culprit: http://www.spinics.net/lists/linux-rt-users/msg11656.html > How exactly do

Re: [PATCH] arm: Set hardirq tracing to on when idling

2014-05-27 Thread Arnd Bergmann
On Tuesday 27 May 2014, Corey Minyard wrote: > On 05/26/2014 04:26 AM, Arnd Bergmann wrote: > > On Sunday 25 May 2014 14:15:23 miny...@acm.org wrote: > >> From: Corey Minyard > >> > >> The CPU will go to idle with interrupts off, but the interrupts > >> will wake up the idle. This was causing

Re: [PATCH] arm: Set hardirq tracing to on when idling

2014-05-27 Thread Corey Minyard
On 05/26/2014 04:26 AM, Arnd Bergmann wrote: > On Sunday 25 May 2014 14:15:23 miny...@acm.org wrote: >> From: Corey Minyard >> >> The CPU will go to idle with interrupts off, but the interrupts >> will wake up the idle. This was causing very long irqsoff trace >> values because, basically, the

Re: [PATCH] arm: Set hardirq tracing to on when idling

2014-05-27 Thread Corey Minyard
On 05/26/2014 04:26 AM, Arnd Bergmann wrote: On Sunday 25 May 2014 14:15:23 miny...@acm.org wrote: From: Corey Minyard cminy...@mvista.com The CPU will go to idle with interrupts off, but the interrupts will wake up the idle. This was causing very long irqsoff trace values because,

Re: [PATCH] arm: Set hardirq tracing to on when idling

2014-05-27 Thread Arnd Bergmann
On Tuesday 27 May 2014, Corey Minyard wrote: On 05/26/2014 04:26 AM, Arnd Bergmann wrote: On Sunday 25 May 2014 14:15:23 miny...@acm.org wrote: From: Corey Minyard cminy...@mvista.com The CPU will go to idle with interrupts off, but the interrupts will wake up the idle. This was

Re: [PATCH] arm: Set hardirq tracing to on when idling

2014-05-27 Thread Stanislav Meduna
On 26.05.2014 11:26, Arnd Bergmann wrote: It seems like the right thing to do, I just don't understand why nobody hit this before. Looks like this is what I did hit a month ago and was not able to find the culprit: http://www.spinics.net/lists/linux-rt-users/msg11656.html How exactly do you

Re: [PATCH] arm: Set hardirq tracing to on when idling

2014-05-27 Thread Arnd Bergmann
On Tuesday 27 May 2014 18:38:33 Stanislav Meduna wrote: On 26.05.2014 11:26, Arnd Bergmann wrote: It seems like the right thing to do, I just don't understand why nobody hit this before. Looks like this is what I did hit a month ago and was not able to find the culprit:

Re: [PATCH] arm: Set hardirq tracing to on when idling

2014-05-27 Thread Corey Minyard
On 05/27/2014 11:16 AM, Arnd Bergmann wrote: On Tuesday 27 May 2014, Corey Minyard wrote: On 05/26/2014 04:26 AM, Arnd Bergmann wrote: On Sunday 25 May 2014 14:15:23 miny...@acm.org wrote: From: Corey Minyard cminy...@mvista.com The CPU will go to idle with interrupts off, but the interrupts

Re: [PATCH] arm: Set hardirq tracing to on when idling

2014-05-27 Thread Stephen Boyd
On 05/27/14 11:49, Arnd Bergmann wrote: You also commented in that thread about stop_critical_timings()/ start_critical_timings(). Corey, can you look at that, too? I think it's designed to avoid the issue you are seeing but for some reason doesn't. I sent a patch last week to solve this

Re: [PATCH] arm: Set hardirq tracing to on when idling

2014-05-27 Thread Arnd Bergmann
On Tuesday 27 May 2014 11:53:59 Stephen Boyd wrote: On 05/27/14 11:49, Arnd Bergmann wrote: You also commented in that thread about stop_critical_timings()/ start_critical_timings(). Corey, can you look at that, too? I think it's designed to avoid the issue you are seeing but for some

Re: [PATCH] arm: Set hardirq tracing to on when idling

2014-05-27 Thread Stephen Boyd
On 05/27/14 12:27, Arnd Bergmann wrote: On Tuesday 27 May 2014 11:53:59 Stephen Boyd wrote: On 05/27/14 11:49, Arnd Bergmann wrote: You also commented in that thread about stop_critical_timings()/ start_critical_timings(). Corey, can you look at that, too? I think it's designed to avoid the

Re: [PATCH] arm: Set hardirq tracing to on when idling

2014-05-27 Thread Arnd Bergmann
On Tuesday 27 May 2014 12:33:38 Stephen Boyd wrote: On 05/27/14 12:27, Arnd Bergmann wrote: On Tuesday 27 May 2014 11:53:59 Stephen Boyd wrote: On 05/27/14 11:49, Arnd Bergmann wrote: You also commented in that thread about stop_critical_timings()/ start_critical_timings(). Corey, can you

Re: [PATCH] arm: Set hardirq tracing to on when idling

2014-05-27 Thread Stephen Boyd
On 05/27/14 12:39, Arnd Bergmann wrote: On Tuesday 27 May 2014 12:33:38 Stephen Boyd wrote: On 05/27/14 12:27, Arnd Bergmann wrote: On Tuesday 27 May 2014 11:53:59 Stephen Boyd wrote: On 05/27/14 11:49, Arnd Bergmann wrote: You also commented in that thread about stop_critical_timings()/

Re: [PATCH] arm: Set hardirq tracing to on when idling

2014-05-27 Thread Corey Minyard
On 05/27/2014 02:27 PM, Arnd Bergmann wrote: On Tuesday 27 May 2014 11:53:59 Stephen Boyd wrote: On 05/27/14 11:49, Arnd Bergmann wrote: You also commented in that thread about stop_critical_timings()/ start_critical_timings(). Corey, can you look at that, too? I think it's designed to avoid

Re: [PATCH] arm: Set hardirq tracing to on when idling

2014-05-26 Thread Arnd Bergmann
On Sunday 25 May 2014 14:15:23 miny...@acm.org wrote: > From: Corey Minyard > > The CPU will go to idle with interrupts off, but the interrupts > will wake up the idle. This was causing very long irqsoff trace > values because, basically, the whole idle time was traces with > irqs off, even

Re: [PATCH] arm: Set hardirq tracing to on when idling

2014-05-26 Thread Arnd Bergmann
On Sunday 25 May 2014 14:15:23 miny...@acm.org wrote: From: Corey Minyard cminy...@mvista.com The CPU will go to idle with interrupts off, but the interrupts will wake up the idle. This was causing very long irqsoff trace values because, basically, the whole idle time was traces with irqs

[PATCH] arm: Set hardirq tracing to on when idling

2014-05-25 Thread minyard
From: Corey Minyard The CPU will go to idle with interrupts off, but the interrupts will wake up the idle. This was causing very long irqsoff trace values because, basically, the whole idle time was traces with irqs off, even though they weren't really off. Rework the idle code to turn hardirq

[PATCH] arm: Set hardirq tracing to on when idling

2014-05-25 Thread minyard
From: Corey Minyard cminy...@mvista.com The CPU will go to idle with interrupts off, but the interrupts will wake up the idle. This was causing very long irqsoff trace values because, basically, the whole idle time was traces with irqs off, even though they weren't really off. Rework the idle