diff options
author | Nicolas Pitre <nicolas.pitre@linaro.org> | 2014-01-29 15:31:36 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2014-02-11 03:58:20 -0500 |
commit | af8cd8ef726f335815233d03b8723e9c52041edd (patch) | |
tree | c3f928607c04a8144215aab0f9737f011c3df59e /kernel | |
parent | 37e6bae8395a94b4dd934c92b02b9408be992365 (diff) |
sched/idle: Move the cpuidle entry point to the generic idle loop
In order to integrate cpuidle with the scheduler, we must have a better
proximity in the core code with what cpuidle is doing and not delegate
such interaction to arch code.
Architectures implementing arch_cpu_idle() should simply enter
a cheap idle mode in the absence of a proper cpuidle driver.
In both cases i.e. whether it is a cpuidle driver or the default
arch_cpu_idle(), the calling convention expects IRQs to be disabled
on entry and enabled on exit. There is a warning in place already but
let's add a forced IRQ enable here as well. This will allow for
removing the forced IRQ enable some implementations do locally and
allowing for the warning to trig.
Signed-off-by: Nicolas Pitre <nico@linaro.org>
Acked-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: Preeti U Murthy <preeti@linux.vnet.ibm.com>
Cc: Paul Mundt <lethal@linux-sh.org>
Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net>
Cc: Olof Johansson <olof@lixom.net>
Cc: Russell King <linux@arm.linux.org.uk>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/alpine.LFD.2.11.1401291526320.1652@knanqh.ubzr
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Diffstat (limited to 'kernel')
-rw-r--r-- | kernel/cpu/idle.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/kernel/cpu/idle.c b/kernel/cpu/idle.c index 277f494c2a9a..b7976a127178 100644 --- a/kernel/cpu/idle.c +++ b/kernel/cpu/idle.c | |||
@@ -3,6 +3,7 @@ | |||
3 | */ | 3 | */ |
4 | #include <linux/sched.h> | 4 | #include <linux/sched.h> |
5 | #include <linux/cpu.h> | 5 | #include <linux/cpu.h> |
6 | #include <linux/cpuidle.h> | ||
6 | #include <linux/tick.h> | 7 | #include <linux/tick.h> |
7 | #include <linux/mm.h> | 8 | #include <linux/mm.h> |
8 | #include <linux/stackprotector.h> | 9 | #include <linux/stackprotector.h> |
@@ -95,8 +96,10 @@ static void cpu_idle_loop(void) | |||
95 | if (!current_clr_polling_and_test()) { | 96 | if (!current_clr_polling_and_test()) { |
96 | stop_critical_timings(); | 97 | stop_critical_timings(); |
97 | rcu_idle_enter(); | 98 | rcu_idle_enter(); |
98 | arch_cpu_idle(); | 99 | if (cpuidle_idle_call()) |
99 | WARN_ON_ONCE(irqs_disabled()); | 100 | arch_cpu_idle(); |
101 | if (WARN_ON_ONCE(irqs_disabled())) | ||
102 | local_irq_enable(); | ||
100 | rcu_idle_exit(); | 103 | rcu_idle_exit(); |
101 | start_critical_timings(); | 104 | start_critical_timings(); |
102 | } else { | 105 | } else { |