* Len Brown <[EMAIL PROTECTED]> 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 <[EMAIL PROTECTED]>

        Ingo
-
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to