* Len Brown <lenb@xxxxxxxxxx> wrote: > acpi_state_timer_broadcast(pr, cx, 0); > - cx->time += ticks_elapsed(t1, t2); > + > + /* Do not account our idle-switching overhead: */ > + sleep_ticks -= cx->latency_ticks + C2_OVERHEAD; > + > + cx->time += sleep_ticks; the scheduler related bits look correct to me. note that the cx->time code is changed by your patch - it now subtracts 'cx->latency_tick + C2/3_OVERHEAD' from it (like it did in the old code). This was intentional, right? Acked-by: Ingo Molnar <mingo@xxxxxxx> Ingo - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html