diff options
author | Daniel Lezcano <daniel.lezcano@linaro.org> | 2014-02-03 17:27:10 +0100 |
---|---|---|
committer | Daniel Lezcano <daniel.lezcano@linaro.org> | 2014-02-06 11:15:42 +0000 |
commit | 6653a4e8c69577a169dc9c6e4deceeb653457666 (patch) | |
tree | da7413a46424524a7188a1aa43c25c25939a1d81 | |
parent | 2558ba5d0ea2d0ab8e35a29e298d1e3d69f31035 (diff) |
idle: Add more comments to the codesched/idle-balance
The idle main function is a complex and a critical function. Added more
comments to the code.
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
-rw-r--r-- | kernel/sched/idle.c | 37 |
1 files changed, 33 insertions, 4 deletions
diff --git a/kernel/sched/idle.c b/kernel/sched/idle.c index b8e54f6c7d15..d5dcff0e644c 100644 --- a/kernel/sched/idle.c +++ b/kernel/sched/idle.c @@ -85,19 +85,34 @@ static int cpuidle_idle_call(void) if (cpu_idle_force_poll || tick_check_broadcast_expired()) return cpu_idle_poll(); + /* + * Check if the idle task must rescheduled. If it is the case, + * exit the function after re-enabling the local irq and set + * again the polling flag + */ if (current_clr_polling_and_test()) { local_irq_enable(); __current_set_polling(); return 0; } + /* + * During the idle period, stop measuring the disabled irqs + * critical sections latencies + */ stop_critical_timings(); + + /* + * Tell the RCU framework we are entering an idle section, + * so no more rcu read side critical sections and one more + * step to the grace period + */ rcu_idle_enter(); - /* Ask the governor for the next state, this call can fail for - * different reasons: cpuidle is not enabled or an idle state - * fulfilling the constraints was not found. In this case, we fall - * back to the default idle function + /* + * Ask the governor to choose an idle state it thinks it is + * convenient to go to. There is *always* a convenient idle + * state but the call could fail if cpuidle is not enabled */ next_state = cpuidle_select(drv, dev); if (next_state < 0) { @@ -105,6 +120,10 @@ static int cpuidle_idle_call(void) goto out; } + /* + * The idle task must be scheduled, it is pointless to go to idle, + * just update no idle residency and get out of this function + */ if (need_resched()) { dev->last_residency = 0; /* give the governor an opportunity to reflect on the outcome */ @@ -115,6 +134,12 @@ static int cpuidle_idle_call(void) trace_cpu_idle_rcuidle(next_state, dev->cpu); + /* + * Enter the idle state previously returned by the governor + * decision. This function will block until an interrupt + * occurs and will take care of re-enabling the local + * interrupts + */ entered_state = cpuidle_enter(drv, dev, next_state); trace_cpu_idle_rcuidle(PWR_EVENT_EXIT, dev->cpu); @@ -150,6 +175,10 @@ static void cpu_idle_loop(void) local_irq_disable(); arch_cpu_idle_enter(); + /* + * It is up to the underlying functions to + * enable the local interrupts again + */ cpuidle_idle_call(); arch_cpu_idle_exit(); |