Re: [PATCH 0/4][V2] cpuidle : multiple drivers support

Peter De Schrijver
2012-10-25T13:49:47Z

On Fri, Oct 19, 2012 at 12:10:45PM +0200, Daniel Lezcano wrote:
> The discussion about having different cpus on the system with
> different latencies bring us to a first attemp by adding a
> pointer in the cpuidle_device to the states array.
>
> But as Rafael suggested, it would make more sense to create a
> driver per cpu [1].
>
> This patch adds support for multiple cpuidle drivers.
>
> It creates a per cpu cpuidle driver pointer.
>
> In order to not break the different drivers, the function cpuidle_register_driver
> assign for each cpu, the driver.
>
> The multiple driver support is optional and if it is not set, the cpuide driver
> core code remains the same (except some code reorganisation).
>
> I did the following tests compiled, booted, tested without/with CONFIG_CPU_IDLE,
> with/without CONFIG_CPU_IDLE_MULTIPLE_DRIVERS.
>
> Tested on Core2 Duo T9500 with acpi_idle [and intel_idle]
> Tested on ARM Dual Cortex-A9 U8500 (aka Snowball)
>
> V1 tested on Tegra3 and Vexpress TC2
>

V2 tested on Tegra3.

Cheers,

Peter.