Re: [PATCH V4 2/3] tick/cpuidle: Initialize hrtimer mode of broadcast

2014-02-11 Thread Daniel Lezcano
On 02/07/2014 09:06 AM, Preeti U Murthy wrote: From: Thomas Gleixner t...@linutronix.de On some architectures, in certain CPU deep idle states the local timers stop. An external clock device is used to wakeup these CPUs. The kernel support for the wakeup of these CPUs is provided by the tick

Re: [PATCH V4 2/3] tick/cpuidle: Initialize hrtimer mode of broadcast

2014-02-11 Thread Thomas Gleixner
On Tue, 11 Feb 2014, Daniel Lezcano wrote: On 02/07/2014 09:06 AM, Preeti U Murthy wrote: Setting the smp affinity on the earliest timer should be handled automatically with the CLOCK_EVT_FEAT_DYNIRQ flag. Did you look at using this flag ? How should this flag help? Not at all, because the

Re: [PATCH V4 2/3] tick/cpuidle: Initialize hrtimer mode of broadcast

2014-02-11 Thread Preeti U Murthy
Hi Daniel, Thank you very much for the review. On 02/11/2014 03:46 PM, Daniel Lezcano wrote: On 02/07/2014 09:06 AM, Preeti U Murthy wrote: From: Thomas Gleixner t...@linutronix.de On some architectures, in certain CPU deep idle states the local timers stop. An external clock device is

Re: [PATCH V4 2/3] tick/cpuidle: Initialize hrtimer mode of broadcast

2014-02-11 Thread Daniel Lezcano
On 02/11/2014 04:58 PM, Thomas Gleixner wrote: On Tue, 11 Feb 2014, Daniel Lezcano wrote: On 02/07/2014 09:06 AM, Preeti U Murthy wrote: Setting the smp affinity on the earliest timer should be handled automatically with the CLOCK_EVT_FEAT_DYNIRQ flag. Did you look at using this flag ? How

Re: [PATCH V4 2/3] tick/cpuidle: Initialize hrtimer mode of broadcast

2014-02-11 Thread Daniel Lezcano
On 02/11/2014 05:09 PM, Preeti U Murthy wrote: Hi Daniel, Thank you very much for the review. On 02/11/2014 03:46 PM, Daniel Lezcano wrote: On 02/07/2014 09:06 AM, Preeti U Murthy wrote: From: Thomas Gleixner t...@linutronix.de On some architectures, in certain CPU deep idle states the

[PATCH V4 2/3] tick/cpuidle: Initialize hrtimer mode of broadcast

2014-02-07 Thread Preeti U Murthy
From: Thomas Gleixner t...@linutronix.de On some architectures, in certain CPU deep idle states the local timers stop. An external clock device is used to wakeup these CPUs. The kernel support for the wakeup of these CPUs is provided by the tick broadcast framework by using the external clock