From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ross Lagerwall Subject: Re: [PATCH 1/2] mwait_idle: Disable Baytrail Core and Module C6 auto-demotion Date: Mon, 8 Sep 2014 15:02:24 +0100 Message-ID: <540DB6F0.40204@citrix.com> References: <1410180127-4812-1-git-send-email-ross.lagerwall@citrix.com> <540DC9B20200007800031FDD@mail.emea.novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <540DC9B20200007800031FDD@mail.emea.novell.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Jan Beulich Cc: Keir Fraser , Xen-devel List-Id: xen-devel@lists.xenproject.org On 09/08/2014 02:22 PM, Jan Beulich wrote: >>>> On 08.09.14 at 14:42, wrote: >> From: Len Brown >> >> Power efficiency improves on Baytrail (Intel Atom Processor E3000) >> when C6 auto-demotion is disabled. >> >> Based on work by Srinidhi Kasagar . >> >> Signed-off-by: Len Brown >> Signed-off-by: Ross Lagerwall > > So we raced on pulling these over - I did so late last week too and > just didn't get to send them out yet. On this one I in fact doubt > (and sent a respective query to Len; no response yet) that the MSR > handling is. Hence my ported over patch is a little different: > According to Intel's manual, those two registers have package scope, i.e. all processor cores in the package share the same MSR, so presumably the original commit is OK. -- Ross Lagerwall