Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756807Ab0BPJMO (ORCPT ); Tue, 16 Feb 2010 04:12:14 -0500 Received: from vms173017pub.verizon.net ([206.46.173.17]:44565 "EHLO vms173017pub.verizon.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756720Ab0BPJMK (ORCPT ); Tue, 16 Feb 2010 04:12:10 -0500 Date: Tue, 16 Feb 2010 04:12:04 -0500 (EST) From: Len Brown X-X-Sender: lenb@localhost.localdomain To: Andrew Morton Cc: "Rafael J. Wysocki" , Arjan van de Ven , Dimitrios Apostolou , Linux Kernel Mailing List , Kernel Testers List , Linus Torvalds Subject: Re: [Bug #15040] High cpu temperature with 2.6.32 In-reply-to: <20100127152539.eb2c8231.akpm@linux-foundation.org> Message-id: References: <201001252152.33031.rjw@sisk.pl> <20100125214150.1f2e1410@infradead.org> <201001261343.15685.rjw@sisk.pl> <20100127152539.eb2c8231.akpm@linux-foundation.org> User-Agent: Alpine 2.00 (LFD 1167 2008-08-23) MIME-version: 1.0 Content-type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4173 Lines: 104 applied thanks, Len Brown, Intel Open Source Technology Center On Wed, 27 Jan 2010, Andrew Morton wrote: > On Tue, 26 Jan 2010 13:43:15 +0100 > "Rafael J. Wysocki" wrote: > > > On Tuesday 26 January 2010, Arjan van de Ven wrote: > > > On Mon, 25 Jan 2010 21:52:32 +0100 > > > "Rafael J. Wysocki" wrote: > > > > > > > On Monday 25 January 2010, Dimitrios Apostolou wrote: > > > > > I will let you know when this bug is fixed in a regular kernel > > > > > release. I just tested 2.6.32.5 and the bug persists. BTW, Arjan, I > > > > > think this bug will be biting other people too, see the following > > > > > thread: http://bugs.archlinux.org/task/17771 > > > > > > > > I don't think it's fixed, I haven't seen the Arjan's patch anywhere > > > > close to the mainline. > > > > > > > > Arjan, perhaps send it directly to Linus, please (unless it's already > > > > waiting somewhere for merging)? > > > > > > I sent it to Len and Andrew (Len as maintainer, Andrew as maintainer of > > > last resort)..... what more would I need? > > > > Well, Andrew, do you have the patch at http://patchwork.kernel.org/patch/71962/ > > in your queue? It's a regression fix. > > > > I had half of it, as > acpi-add-the-hp-pavilion-zv5000-to-the-power-dmi-table.patch. Updated. > > > > From: Arjan van de Ven > > Since the rewrite of the CPU idle governor in 2.6.32, two laptops have > surfaced where the BIOS advertises a C2 power state, but for some reason > this state is not functioning (as verified in both cases by powertop > before the patch in .32). > > The old governor had the accidental behavior that if a non-working state > was chosen too many times, it would end up falling back to C1. The new > governor works differently and this accidental behavior is no longer > there; the result is a high temperature on these two machines. > > This patch adds these 2 machines to the DMI table for C state anomalies; > by just not using C2 both these machines are better off (the TSC can be > used instead of the pm timer, giving a performance boost for example). > > Addresses http://bugzilla.kernel.org/show_bug.cgi?id=14742 > > Signed-off-by: Arjan van de Ven > Reported-by: > Cc: Dimitrios Apostolou > Cc: Alex Chiang > Cc: Len Brown > Cc: Bjorn Helgaas > Cc: Yinghai Lu > Cc: Venkatesh Pallipadi > Cc: Lin Ming > Cc: "Rafael J. Wysocki" > Cc: > Signed-off-by: Andrew Morton > --- > > drivers/acpi/processor_idle.c | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff -puN drivers/acpi/processor_idle.c~drivers-acpi-processor_idlec-add-two-laptops-to-the-c-state-dmi-table drivers/acpi/processor_idle.c > --- a/drivers/acpi/processor_idle.c~drivers-acpi-processor_idlec-add-two-laptops-to-the-c-state-dmi-table > +++ a/drivers/acpi/processor_idle.c > @@ -110,6 +110,14 @@ static struct dmi_system_id __cpuinitdat > DMI_MATCH(DMI_BIOS_VENDOR,"Phoenix Technologies LTD"), > DMI_MATCH(DMI_BIOS_VERSION,"SHE845M0.86C.0013.D.0302131307")}, > (void *)2}, > + { set_max_cstate, "Pavilion zv5000", { > + DMI_MATCH(DMI_SYS_VENDOR, "Hewlett-Packard"), > + DMI_MATCH(DMI_PRODUCT_NAME,"Pavilion zv5000 (DS502A#ABA)")}, > + (void *)1}, > + { set_max_cstate, "Asus L8400B", { > + DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK Computer Inc."), > + DMI_MATCH(DMI_PRODUCT_NAME,"L8400B series Notebook PC")}, > + (void *)1}, > {}, > }; > > _ > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@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 majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/