From mboxrd@z Thu Jan 1 00:00:00 1970 From: Iain Subject: Re: [PATCH] ACPI: make acpi_idle Nehalem-aware Date: Fri, 23 Jul 2010 13:40:50 +0100 Message-ID: <4C498DD2.1040002@gmail.com> References: <20100427124703.GA16706@jgarrett.org> <20100430174447.GA14889@srcf.ucam.org> <20100525124325.GC7876@srcf.ucam.org> <20100525185507.GA15997@srcf.ucam.org> <20100722074750.GA22520@one.firstfloor.org> <4C48B730.30708@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from mail-ww0-f44.google.com ([74.125.82.44]:62820 "EHLO mail-ww0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753055Ab0GWMkz (ORCPT ); Fri, 23 Jul 2010 08:40:55 -0400 In-Reply-To: Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: Len Brown Cc: Andi Kleen , Matthew Garrett , "Yu, Luming" , Philip Langdale , Jeff Garrett , Linux Kernel Mailing List , "linux-acpi@vger.kernel.org" , "venki@google.com" Len Brown wrote: > You read it correctly, your BIOS does not request BM_STS, mjg59's does. Right, and on a DL360 G6 with the 07-24-2009 bios version I saw the same. > I expect that is to enable PCC, which would change P-states, > but unlikely would have an effect on C-states. I found another option in the bios to limit or disable the C-states today, so plenty of opportunity to configure the system into an odd state. > If you can try it both ways that might be good to know. > (include powertop display once again) > Of course, the default setting is what 99% of customers use... I'll upload an archive to the bugzilla entry with the details. What seems to happen is that when you set the default Balanced Power and Performance mode the CST code vanishes completely and the processor manages to get to c6 some of the time. Enable OS Control mode and the bad CST code appears. > This is BIOS writer "value add". > Unclear how it migh be an improvement over what Linux has been shipping > for years. Well yes, having Linux and the bios fighting for control probably isn't going to help. > Please upload the output from dmidecode to the bug report. > I am hopeful that you have a current BIOS and that > Matthew may have an pre-production BIOS. I've uploaded an archive with dmidecode, turbostat and powertop dumps. There are dumps with the bios set to the default, and to OS Control mode. The original bios on my DL360G6 was 07-24-2009 and has the same issue as Matthew. I upgraded the machine to the latest 2010.05.15 and repeated the tests. Good news is that the new bios has fixed the CST code so that the Access length values are all 0x01 when they're present and the dumps show the processor getting into c6 much more. So you were correct, bios fix was needed. Iain