From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Li, Aubrey" Subject: Re: [PATCH v2] intel_idle: add idle values for Cherrytrail/Braswell Date: Wed, 27 Aug 2014 08:03:40 +0800 Message-ID: <53FD205C.8070302@linux.intel.com> References: <1408701508-13636-1-git-send-email-mika.westerberg@linux.intel.com> <1408707540-12393-1-git-send-email-mika.westerberg@linux.intel.com> <53F74E5D.2040108@linux.intel.com> <20140825101200.GH1660@lahna.fi.intel.com> <53FBF328.1080205@linux.intel.com> <20140826090032.GN1660@lahna.fi.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Return-path: Received: from mga09.intel.com ([134.134.136.24]:17404 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755695AbaH0AEY (ORCPT ); Tue, 26 Aug 2014 20:04:24 -0400 In-Reply-To: <20140826090032.GN1660@lahna.fi.intel.com> Sender: linux-pm-owner@vger.kernel.org List-Id: linux-pm@vger.kernel.org To: Mika Westerberg Cc: Len Brown , Kumar P Mahesh , Alan Cox , linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org 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 >>>>> >>>>> Cherrytrail/Braswell is a successor of Intel Baytrail but has slighly >>>>> different CPU idle values and latencies. >>>>> >>>>> Signed-off-by: Kumar P Mahesh >>>>> Signed-off-by: Alan Cox >>>>> Signed-off-by: Mika Westerberg >>>>> --- >>>>> 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