Re: too many timer retries happen when do local timer swtich with broadcast timer

2013-02-25 Thread Lorenzo Pieralisi
On Fri, Feb 22, 2013 at 06:52:14PM +, Thomas Gleixner wrote: On Fri, 22 Feb 2013, Lorenzo Pieralisi wrote: On Fri, Feb 22, 2013 at 03:03:02PM +, Thomas Gleixner wrote: On Fri, 22 Feb 2013, Lorenzo Pieralisi wrote: On Fri, Feb 22, 2013 at 12:07:30PM +, Thomas Gleixner wrote:

Re: too many timer retries happen when do local timer swtich with broadcast timer

2013-02-24 Thread Santosh Shilimkar
On Saturday 23 February 2013 12:22 AM, Thomas Gleixner wrote: On Fri, 22 Feb 2013, Lorenzo Pieralisi wrote: On Fri, Feb 22, 2013 at 03:03:02PM +, Thomas Gleixner wrote: On Fri, 22 Feb 2013, Lorenzo Pieralisi wrote: On Fri, Feb 22, 2013 at 12:07:30PM +, Thomas Gleixner wrote: Now we

Re: too many timer retries happen when do local timer swtich with broadcast timer

2013-02-24 Thread Jason Liu
Thomas, 2013/2/23 Thomas Gleixner t...@linutronix.de: On Fri, 22 Feb 2013, Lorenzo Pieralisi wrote: On Fri, Feb 22, 2013 at 03:03:02PM +, Thomas Gleixner wrote: On Fri, 22 Feb 2013, Lorenzo Pieralisi wrote: On Fri, Feb 22, 2013 at 12:07:30PM +, Thomas Gleixner wrote: Now we

Re: too many timer retries happen when do local timer swtich with broadcast timer

2013-02-22 Thread Santosh Shilimkar
Thomas, On Friday 22 February 2013 03:49 AM, Thomas Gleixner wrote: On Thu, 21 Feb 2013, Santosh Shilimkar wrote: On Thursday 21 February 2013 07:18 PM, Thomas Gleixner wrote: find below a completely untested patch, which should address that issue. After looking at the thread, I tried to

Re: too many timer retries happen when do local timer swtich with broadcast timer

2013-02-22 Thread Thomas Gleixner
On Fri, 22 Feb 2013, Santosh Shilimkar wrote: BTW, Lorenzo off-list mentioned to me about warning in boot-up which I missed while testing your patch. It will take bit more time for me to look into it and hence thought of reporting it. [2.186126] [ cut here ] [

Re: too many timer retries happen when do local timer swtich with broadcast timer

2013-02-22 Thread Jason Liu
Thomas, 2013/2/21 Thomas Gleixner t...@linutronix.de: Jason, On Thu, 21 Feb 2013, Jason Liu wrote: 2013/2/21 Thomas Gleixner t...@linutronix.de: Now your explanation makes sense. I have no fast solution for this, but I think that I have an idea how to fix it. Stay tuned. Thanks

Re: too many timer retries happen when do local timer swtich with broadcast timer

2013-02-22 Thread Lorenzo Pieralisi
On Thu, Feb 21, 2013 at 10:19:18PM +, Thomas Gleixner wrote: On Thu, 21 Feb 2013, Santosh Shilimkar wrote: On Thursday 21 February 2013 07:18 PM, Thomas Gleixner wrote: find below a completely untested patch, which should address that issue. After looking at the thread, I tried to

Re: too many timer retries happen when do local timer swtich with broadcast timer

2013-02-22 Thread Santosh Shilimkar
On Friday 22 February 2013 03:54 PM, Thomas Gleixner wrote: On Fri, 22 Feb 2013, Santosh Shilimkar wrote: BTW, Lorenzo off-list mentioned to me about warning in boot-up which I missed while testing your patch. It will take bit more time for me to look into it and hence thought of reporting it.

Re: too many timer retries happen when do local timer swtich with broadcast timer

2013-02-22 Thread Lorenzo Pieralisi
On Fri, Feb 22, 2013 at 10:24:00AM +, Thomas Gleixner wrote: On Fri, 22 Feb 2013, Santosh Shilimkar wrote: BTW, Lorenzo off-list mentioned to me about warning in boot-up which I missed while testing your patch. It will take bit more time for me to look into it and hence thought of

Re: too many timer retries happen when do local timer swtich with broadcast timer

2013-02-22 Thread Santosh Shilimkar
On Friday 22 February 2013 04:01 PM, Lorenzo Pieralisi wrote: On Fri, Feb 22, 2013 at 10:24:00AM +, Thomas Gleixner wrote: On Fri, 22 Feb 2013, Santosh Shilimkar wrote: BTW, Lorenzo off-list mentioned to me about warning in boot-up which I missed while testing your patch. It will take bit

Re: too many timer retries happen when do local timer swtich with broadcast timer

2013-02-22 Thread Thomas Gleixner
On Fri, 22 Feb 2013, Santosh Shilimkar wrote: On Friday 22 February 2013 04:01 PM, Lorenzo Pieralisi wrote: On Fri, Feb 22, 2013 at 10:24:00AM +, Thomas Gleixner wrote: On Fri, 22 Feb 2013, Santosh Shilimkar wrote: BTW, Lorenzo off-list mentioned to me about warning in boot-up

Re: too many timer retries happen when do local timer swtich with broadcast timer

2013-02-22 Thread Lorenzo Pieralisi
On Fri, Feb 22, 2013 at 12:07:30PM +, Thomas Gleixner wrote: On Fri, 22 Feb 2013, Santosh Shilimkar wrote: On Friday 22 February 2013 04:01 PM, Lorenzo Pieralisi wrote: On Fri, Feb 22, 2013 at 10:24:00AM +, Thomas Gleixner wrote: On Fri, 22 Feb 2013, Santosh Shilimkar wrote:

Re: too many timer retries happen when do local timer swtich with broadcast timer

2013-02-22 Thread Thomas Gleixner
On Fri, 22 Feb 2013, Lorenzo Pieralisi wrote: On Fri, Feb 22, 2013 at 12:07:30PM +, Thomas Gleixner wrote: Now we could make use of that and avoid going deep idle just to come back right away via the IPI. Unfortunately the notification thingy has no return value, but we can fix that.

Re: too many timer retries happen when do local timer swtich with broadcast timer

2013-02-22 Thread Lorenzo Pieralisi
On Fri, Feb 22, 2013 at 03:03:02PM +, Thomas Gleixner wrote: On Fri, 22 Feb 2013, Lorenzo Pieralisi wrote: On Fri, Feb 22, 2013 at 12:07:30PM +, Thomas Gleixner wrote: Now we could make use of that and avoid going deep idle just to come back right away via the IPI. Unfortunately

Re: too many timer retries happen when do local timer swtich with broadcast timer

2013-02-22 Thread Thomas Gleixner
On Fri, 22 Feb 2013, Lorenzo Pieralisi wrote: On Fri, Feb 22, 2013 at 03:03:02PM +, Thomas Gleixner wrote: On Fri, 22 Feb 2013, Lorenzo Pieralisi wrote: On Fri, Feb 22, 2013 at 12:07:30PM +, Thomas Gleixner wrote: Now we could make use of that and avoid going deep idle just to

Re: too many timer retries happen when do local timer swtich with broadcast timer

2013-02-21 Thread Thomas Gleixner
On Thu, 21 Feb 2013, Jason Liu wrote: 2013/2/20 Thomas Gleixner t...@linutronix.de: On Wed, 20 Feb 2013, Jason Liu wrote: void arch_idle(void) { clockevents_notify(CLOCK_EVT_NOTIFY_BROADCAST_ENTER, cpu); enter_the_wait_mode();

Re: too many timer retries happen when do local timer swtich with broadcast timer

2013-02-21 Thread Lorenzo Pieralisi
Hi Jason, On Thu, Feb 21, 2013 at 06:16:51AM +, Jason Liu wrote: 2013/2/20 Thomas Gleixner t...@linutronix.de: On Wed, 20 Feb 2013, Jason Liu wrote: void arch_idle(void) { clockevents_notify(CLOCK_EVT_NOTIFY_BROADCAST_ENTER, cpu); enter_the_wait_mode();

Re: too many timer retries happen when do local timer swtich with broadcast timer

2013-02-21 Thread Jason Liu
2013/2/21 Lorenzo Pieralisi lorenzo.pieral...@arm.com: Hi Jason, On Thu, Feb 21, 2013 at 06:16:51AM +, Jason Liu wrote: 2013/2/20 Thomas Gleixner t...@linutronix.de: On Wed, 20 Feb 2013, Jason Liu wrote: void arch_idle(void) {

Re: too many timer retries happen when do local timer swtich with broadcast timer

2013-02-21 Thread Jason Liu
2013/2/21 Thomas Gleixner t...@linutronix.de: On Thu, 21 Feb 2013, Jason Liu wrote: 2013/2/20 Thomas Gleixner t...@linutronix.de: On Wed, 20 Feb 2013, Jason Liu wrote: void arch_idle(void) { clockevents_notify(CLOCK_EVT_NOTIFY_BROADCAST_ENTER, cpu); enter_the_wait_mode();

Re: too many timer retries happen when do local timer swtich with broadcast timer

2013-02-21 Thread Thomas Gleixner
Jason, On Thu, 21 Feb 2013, Jason Liu wrote: 2013/2/21 Thomas Gleixner t...@linutronix.de: Now your explanation makes sense. I have no fast solution for this, but I think that I have an idea how to fix it. Stay tuned. Thanks Thomas, wait for your fix. :) find below a completely

Re: too many timer retries happen when do local timer swtich with broadcast timer

2013-02-21 Thread Santosh Shilimkar
Thomas, On Thursday 21 February 2013 07:18 PM, Thomas Gleixner wrote: Jason, On Thu, 21 Feb 2013, Jason Liu wrote: 2013/2/21 Thomas Gleixner t...@linutronix.de: Now your explanation makes sense. I have no fast solution for this, but I think that I have an idea how to fix it. Stay tuned.

Re: too many timer retries happen when do local timer swtich with broadcast timer

2013-02-21 Thread Thomas Gleixner
On Thu, 21 Feb 2013, Santosh Shilimkar wrote: On Thursday 21 February 2013 07:18 PM, Thomas Gleixner wrote: find below a completely untested patch, which should address that issue. After looking at the thread, I tried to see the issue on OMAP and could see the same issue as Jason. That's

too many timer retries happen when do local timer swtich with broadcast timer

2013-02-20 Thread Jason Liu
Hi, sorry for so long email, please be patient... thanks, I have seen too many timer retries happen when do local timer switch with broadcast timeron ARM Cortex A9 SMP(4 cores), see the following log such as: retries: 36383 root@~$ cat /proc/timer_list Timer List Version: v0.6

Re: too many timer retries happen when do local timer swtich with broadcast timer

2013-02-20 Thread Thomas Gleixner
On Wed, 20 Feb 2013, Jason Liu wrote: void arch_idle(void) { clockevents_notify(CLOCK_EVT_NOTIFY_BROADCAST_ENTER, cpu); enter_the_wait_mode(); clockevents_notify(CLOCK_EVT_NOTIFY_BROADCAST_EXIT, cpu); } when the broadcast timer interrupt arrives(this interrupt just wakeup the

Re: too many timer retries happen when do local timer swtich with broadcast timer

2013-02-20 Thread Jason Liu
2013/2/20 Thomas Gleixner t...@linutronix.de: On Wed, 20 Feb 2013, Jason Liu wrote: void arch_idle(void) { clockevents_notify(CLOCK_EVT_NOTIFY_BROADCAST_ENTER, cpu); enter_the_wait_mode(); clockevents_notify(CLOCK_EVT_NOTIFY_BROADCAST_EXIT, cpu); } when the broadcast timer