On Thu, 28 Mar 2013, Srivatsa S. Bhat wrote:
> (In all my replies, I'm referring to the code in your v2 on your git
> tree, but replying to the corresponding patches in your v1).
>
> So, in alpha, we poll in cpu idle. In the generic implementation in
> kernel/cpu/idle.c, arch_cpu_idle() is defined as:
>
> void __weak arch_cpu_idle(void)
> {
> cpu_idle_poll();
> }
>
> Should it not have been:
>
> void __weak arch_cpu_idle(void)
> {
> current_set_polling();
> cpu_idle_poll();
> current_clr_polling();
> }
>
> instead?
Hmm, we should not clear it at all, because we are always polling and
never leave polling mode. So instead of calling cpu_idle_poll from the
weak arch_cpu_idle() we should set cpu_idle_force_poll so we drop into
if (cpu_idle_force_poll) {
cpu_idle_poll();
} else ...
which keeps the polling flag set.
Thanks,
tglx
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
