On 2014/8/25 18:12, Mika Westerberg wrote: > On Fri, Aug 22, 2014 at 10:06:21PM +0800, Li, Aubrey wrote: >> On 2014/8/22 19:39, Mika Westerberg wrote: >>> From: Mahesh Kumar P <mahesh.kuma...@intel.com> >>> >>> Cherrytrail/Braswell is a successor of Intel Baytrail but has slighly >>> different CPU idle values and latencies. >>> >>> Signed-off-by: Kumar P Mahesh <mahesh.kuma...@intel.com> >>> Signed-off-by: Alan Cox <a...@linux.intel.com> >>> Signed-off-by: Mika Westerberg <mika.westerb...@linux.intel.com> >>> --- >>> I learned from Mahesh that C1e, C4 and S0i2 states are deprecated so those >>> are now removed. >> >> If the platform has ACPI supported, I believe Len wants to see what >> C-states number exported by ACPI. >> >> Please boot the machine with "intel_idle.max_cstate=0" to disable >> intel_idle, then post the output of the following commands > > OK, here goes:
ACPI exposes 0x58 as C6 mwait hint while the patch proposes 0x52 for C6. Can you please remove "intel_idle.max_cstate=0" and run "dmesg | grep idle" again to see how many C6 supports in CPUID enumeration? Thanks, -Aubrey > >> dmesg | grep idle > > [ 0.523624] cpuidle: using governor ladder > [ 0.528215] cpuidle: using governor menu > [ 16.649461] intel_idle: disabled > [ 16.736481] ACPI: acpi_idle registered with cpuidle > >> grep . /sys/devices/system/cpu/cpu0/cpuidle/*/* > > /sys/devices/system/cpu/cpu0/cpuidle/state0/desc:CPUIDLE CORE POLL IDLE > /sys/devices/system/cpu/cpu0/cpuidle/state0/disable:0 > /sys/devices/system/cpu/cpu0/cpuidle/state0/latency:0 > /sys/devices/system/cpu/cpu0/cpuidle/state0/name:POLL > /sys/devices/system/cpu/cpu0/cpuidle/state0/power:4294967295 > /sys/devices/system/cpu/cpu0/cpuidle/state0/residency:0 > /sys/devices/system/cpu/cpu0/cpuidle/state0/time:0 > /sys/devices/system/cpu/cpu0/cpuidle/state0/usage:0 > /sys/devices/system/cpu/cpu0/cpuidle/state1/desc:ACPI FFH INTEL MWAIT 0x0 > /sys/devices/system/cpu/cpu0/cpuidle/state1/disable:0 > /sys/devices/system/cpu/cpu0/cpuidle/state1/latency:1 > /sys/devices/system/cpu/cpu0/cpuidle/state1/name:C1 > /sys/devices/system/cpu/cpu0/cpuidle/state1/power:0 > /sys/devices/system/cpu/cpu0/cpuidle/state1/residency:2 > /sys/devices/system/cpu/cpu0/cpuidle/state1/time:1255076 > /sys/devices/system/cpu/cpu0/cpuidle/state1/usage:763 > /sys/devices/system/cpu/cpu0/cpuidle/state2/desc:ACPI FFH INTEL MWAIT 0x58 > /sys/devices/system/cpu/cpu0/cpuidle/state2/disable:0 > /sys/devices/system/cpu/cpu0/cpuidle/state2/latency:500 > /sys/devices/system/cpu/cpu0/cpuidle/state2/name:C2 > /sys/devices/system/cpu/cpu0/cpuidle/state2/power:0 > /sys/devices/system/cpu/cpu0/cpuidle/state2/residency:1000 > /sys/devices/system/cpu/cpu0/cpuidle/state2/time:1694866 > /sys/devices/system/cpu/cpu0/cpuidle/state2/usage:639 > /sys/devices/system/cpu/cpu0/cpuidle/state3/desc:ACPI FFH INTEL MWAIT 0x64 > /sys/devices/system/cpu/cpu0/cpuidle/state3/disable:0 > /sys/devices/system/cpu/cpu0/cpuidle/state3/latency:5000 > /sys/devices/system/cpu/cpu0/cpuidle/state3/name:C3 > /sys/devices/system/cpu/cpu0/cpuidle/state3/power:0 > /sys/devices/system/cpu/cpu0/cpuidle/state3/residency:10000 > /sys/devices/system/cpu/cpu0/cpuidle/state3/time:380547473 > /sys/devices/system/cpu/cpu0/cpuidle/state3/usage:744 > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/