linux-kernel - Re: [PATCH v2] intel_idle: add idle values for Cherrytrail/Braswell

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <53FD205C.8070302@linux.intel.com>
Date:	Wed, 27 Aug 2014 08:03:40 +0800
From:	"Li, Aubrey" <aubrey.li@...ux.intel.com>
To:	Mika Westerberg <mika.westerberg@...ux.intel.com>
CC:	Len Brown <lenb@...nel.org>,
	Kumar P Mahesh <mahesh.kumar.p@...el.com>,
	Alan Cox <alan@...ux.intel.com>, linux-pm@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] intel_idle: add idle values for Cherrytrail/Braswell

On 2014/8/26 17:00, Mika Westerberg wrote:
> On Tue, Aug 26, 2014 at 10:38:32AM +0800, Li, Aubrey wrote:
>> 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.kumar.p@...el.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.kumar.p@...el.com>
>>>>> Signed-off-by: Alan Cox <alan@...ux.intel.com>
>>>>> Signed-off-by: Mika Westerberg <mika.westerberg@...ux.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?
> 
> [   16.685769] intel_idle: MWAIT substates: 0x33000020
> [   16.691236] intel_idle: v0.4 model 0x4C
> [   16.695539] intel_idle: lapic_timer_reliable_states 0xffffffff
> 

enough to turn over to Len to finalize, thanks Mika.

-Aubrey

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/