cpuidle changes and drivers/acpi/processor_idle.c

[Rob Lee <rob.lee@xxxxxxxxxx>]

In the various versions of acpi_idle_enter, I see the following code.

...
	cx->usage++;

	lapic_timer_state_broadcast(pr, cx, 0);
	cx->time += idle_time;
...

But this usage and idle_time and accounted for by the core cpuidle
driver, so is this a small bug and the usage and idle_time accounting
code should be removed?

Also, I see the following:

	/* Tell the scheduler how much we idled: */
	sched_clock_idle_wakeup_event(idle_time_ns);

The sched_clock_idle_wakeup_event does nothing with the idle_time_ns
parameter it is passed.  Perhaps it did at one time.  Is this call
still necessary?

Thanks,
Rob
--
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