Subject: Re: [v7 PATCH 0/7]: cpuidle/x86/POWER: Cleanup idle power management code in x86, cleanup drivers/cpuidle/cpuidle.c and introduce cpuidle to POWER.

From: Arun R Bharadwaj <[email-masked]>
Date: Tue, 6 Oct 2009 22:05:21 +0530

* Arun R Bharadwaj <[email-masked]> [2009-10-06 20:54:21]:

> Hi
> 
> Please consider this for inclusion into the testing tree.
> 
> This patchset introduces cpuidle infrastructure to POWER, prototyping
> for pSeries, and also does a major refactoring of current x86 idle
> power management and a cleanup of cpuidle infrastructure.
> 
> Earlier discussions on the same can be found at:
> 
> v6 --> http://lkml.org/lkml/2009/9/22/180
> v5 --> http://lkml.org/lkml/2009/9/22/26
> v4 --> http://lkml.org/lkml/2009/9/1/133
> v3 --> http://lkml.org/lkml/2009/8/27/124
> v2 --> http://lkml.org/lkml/2009/8/26/233
> v1 --> http://lkml.org/lkml/2009/8/19/150
> 
> 
> Changes in this version:
> -----------------------------------------
> 
> * Earlier implementation maintained a list of cpuidle devices and
>   the list was traversed to disable and enable the cpuidle devices
>   when switching governors. This list is not necessary and is removed.
>   Now, we get the per cpu structure cpuidle_devices and disable and
>   enable it for each online cpu while switching governors.
> 
> * Earlier iterations were buggy w.r.t the sysfs interface of the cpuidle.
>   This iteration removes the bug in PATCH 1/7.
> 
> * In PATCH 7/7, smt_snooze_delay was not being updated whenever it was
>   changed through the sysfs interface. Provide a helper routine to
>   update it.
> 
> * In PATCH 7/7, the idle loops are renamed to denote the idle states
>   to avoid any confusion.
> 

Also, the per-cpu nature of registration/unregistration of cpuidle
has been maintained as ACPI needs this.

> thanks,
> arun