cpuidle sizes (Re: [PATCH 14/16] intel_idle: remove use and definition of MWAIT_MAX_NUM_CSTATES)

2013-02-17 Thread Len Brown
On 02/12/2013 05:43 PM, Daniel Lezcano wrote: > On 02/12/2013 12:46 AM, Len Brown wrote: >> On 02/11/2013 03:53 AM, Daniel Lezcano wrote: >>> On 02/09/2013 02:08 AM, Len Brown wrote: >> The reason to change is that intel_idle will soon be able to export more than the 8 "major" states

cpuidle sizes (Re: [PATCH 14/16] intel_idle: remove use and definition of MWAIT_MAX_NUM_CSTATES)

2013-02-17 Thread Len Brown
On 02/12/2013 05:43 PM, Daniel Lezcano wrote: On 02/12/2013 12:46 AM, Len Brown wrote: On 02/11/2013 03:53 AM, Daniel Lezcano wrote: On 02/09/2013 02:08 AM, Len Brown wrote: The reason to change is that intel_idle will soon be able to export more than the 8 major states supported by MWAIT.

Re: [PATCH 14/16] intel_idle: remove use and definition of MWAIT_MAX_NUM_CSTATES

2013-02-12 Thread Daniel Lezcano
On 02/12/2013 12:46 AM, Len Brown wrote: > On 02/11/2013 03:53 AM, Daniel Lezcano wrote: >> On 02/09/2013 02:08 AM, Len Brown wrote: > >>> The reason to change is that intel_idle will soon be able >>> to export more than the 8 "major" states supported by MWAIT. >>> When we hit that limit, it is

Re: [PATCH 14/16] intel_idle: remove use and definition of MWAIT_MAX_NUM_CSTATES

2013-02-12 Thread Daniel Lezcano
On 02/12/2013 12:46 AM, Len Brown wrote: On 02/11/2013 03:53 AM, Daniel Lezcano wrote: On 02/09/2013 02:08 AM, Len Brown wrote: The reason to change is that intel_idle will soon be able to export more than the 8 major states supported by MWAIT. When we hit that limit, it is important to

Re: [PATCH 14/16] intel_idle: remove use and definition of MWAIT_MAX_NUM_CSTATES

2013-02-11 Thread Len Brown
On 02/11/2013 03:53 AM, Daniel Lezcano wrote: > On 02/09/2013 02:08 AM, Len Brown wrote: >> The reason to change is that intel_idle will soon be able >> to export more than the 8 "major" states supported by MWAIT. >> When we hit that limit, it is important to know >> where the limit comes from. >

Re: [PATCH 14/16] intel_idle: remove use and definition of MWAIT_MAX_NUM_CSTATES

2013-02-11 Thread Daniel Lezcano
On 02/09/2013 02:08 AM, Len Brown wrote: > From: Len Brown > > Cosmetic only. > > Replace use of MWAIT_MAX_NUM_CSTATES with CPUIDLE_STATE_MAX. > They are both 8, so this patch has no functional change. > > The reason to change is that intel_idle will soon be able > to export more than the 8

Re: [PATCH 14/16] intel_idle: remove use and definition of MWAIT_MAX_NUM_CSTATES

2013-02-11 Thread Daniel Lezcano
On 02/09/2013 02:08 AM, Len Brown wrote: From: Len Brown len.br...@intel.com Cosmetic only. Replace use of MWAIT_MAX_NUM_CSTATES with CPUIDLE_STATE_MAX. They are both 8, so this patch has no functional change. The reason to change is that intel_idle will soon be able to export more

Re: [PATCH 14/16] intel_idle: remove use and definition of MWAIT_MAX_NUM_CSTATES

2013-02-11 Thread Len Brown
On 02/11/2013 03:53 AM, Daniel Lezcano wrote: On 02/09/2013 02:08 AM, Len Brown wrote: The reason to change is that intel_idle will soon be able to export more than the 8 major states supported by MWAIT. When we hit that limit, it is important to know where the limit comes from. Does it

[PATCH 14/16] intel_idle: remove use and definition of MWAIT_MAX_NUM_CSTATES

2013-02-08 Thread Len Brown
From: Len Brown Cosmetic only. Replace use of MWAIT_MAX_NUM_CSTATES with CPUIDLE_STATE_MAX. They are both 8, so this patch has no functional change. The reason to change is that intel_idle will soon be able to export more than the 8 "major" states supported by MWAIT. When we hit that limit, it

[PATCH 14/16] intel_idle: remove use and definition of MWAIT_MAX_NUM_CSTATES

2013-02-08 Thread Len Brown
From: Len Brown len.br...@intel.com Cosmetic only. Replace use of MWAIT_MAX_NUM_CSTATES with CPUIDLE_STATE_MAX. They are both 8, so this patch has no functional change. The reason to change is that intel_idle will soon be able to export more than the 8 major states supported by MWAIT. When we