Re: [PATCH V4 7/9] cpuidle/powernv: Add "Fast-Sleep" CPU idle state

2013-12-02 Thread Thomas Gleixner
On Mon, 2 Dec 2013, Preeti U Murthy wrote: > On 11/29/2013 08:09 PM, Thomas Gleixner wrote: > > This supports HIGHRES and NO_HZ if done right, without polling at > > all. So you can even let the last CPU which handles the broadcast > > hrtimer go for a long sleep, just not in the deepest idle state

Re: [PATCH V4 7/9] cpuidle/powernv: Add "Fast-Sleep" CPU idle state

2013-12-02 Thread Preeti U Murthy
Hi Thomas, On 11/29/2013 08:09 PM, Thomas Gleixner wrote: > On Fri, 29 Nov 2013, Preeti U Murthy wrote: >> +static enum hrtimer_restart handle_broadcast(struct hrtimer *hrtimer) >> +{ >> +struct clock_event_device *bc_evt = &bc_timer; >> +ktime_t interval, next_bc_tick, now; >> + >> +n

Re: [PATCH V4 7/9] cpuidle/powernv: Add "Fast-Sleep" CPU idle state

2013-11-29 Thread Thomas Gleixner
On Fri, 29 Nov 2013, Preeti U Murthy wrote: > +static enum hrtimer_restart handle_broadcast(struct hrtimer *hrtimer) > +{ > + struct clock_event_device *bc_evt = &bc_timer; > + ktime_t interval, next_bc_tick, now; > + > + now = ktime_get(); > + > + if (!restart_broadcast(bc_evt)) >

[PATCH V4 7/9] cpuidle/powernv: Add "Fast-Sleep" CPU idle state

2013-11-29 Thread Preeti U Murthy
Fast sleep is one of the deep idle states on Power8 in which local timers of CPUs stop. Now that the basic support for fast sleep has been added, enable it in the cpuidle framework on PowerNV. On ppc, since we do not have an external device that can wakeup cpus in deep idle, the local timer of one