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

From: Thomas Gleixner
Date: Mon Dec 02 2013 - 13:00:41 EST


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.
>
> Thank you for the review. The above points are all valid. I will rework
> the design to:
>
> 1. Eliminate the concept of a broadcast CPU and integrate its
> functionality in the timekeeping CPU.
>
> 2. Avoid polling by using IPIs to communicate the next wakeup of the
> CPUs in deep idle state so as to reprogram the broadcast hrtimer.
>
> 3. Make this feature generic and not arch-specific.

Great. If you need help with the generic bits, please let me know.

Thanks,

tglx
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/