[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-id: <alpine.LFD.2.00.1002160411480.21595@localhost.localdomain>
Date: Tue, 16 Feb 2010 04:12:04 -0500 (EST)
From: Len Brown <lenb@...nel.org>
To: Andrew Morton <akpm@...ux-foundation.org>
Cc: "Rafael J. Wysocki" <rjw@...k.pl>,
Arjan van de Ven <arjan@...radead.org>,
Dimitrios Apostolou <jimis@....net>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Kernel Testers List <kernel-testers@...r.kernel.org>,
Linus Torvalds <torvalds@...ux-foundation.org>
Subject: Re: [Bug #15040] High cpu temperature with 2.6.32
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" <rjw@...k.pl> wrote:
>
> > On Tuesday 26 January 2010, Arjan van de Ven wrote:
> > > On Mon, 25 Jan 2010 21:52:32 +0100
> > > "Rafael J. Wysocki" <rjw@...k.pl> 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 <arjan@...ux.intel.com>
>
> 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 <arjan@...ux.intel.com>
> Reported-by: <akwatts@...il.com>
> Cc: Dimitrios Apostolou <jimis@....net>
> Cc: Alex Chiang <achiang@...com>
> Cc: Len Brown <lenb@...nel.org>
> Cc: Bjorn Helgaas <bjorn.helgaas@...com>
> Cc: Yinghai Lu <yinghai@...nel.org>
> Cc: Venkatesh Pallipadi <venkatesh.pallipadi@...el.com>
> Cc: Lin Ming <ming.m.lin@...el.com>
> Cc: "Rafael J. Wysocki" <rjw@...k.pl>
> Cc: <stable@...nel.org>
> Signed-off-by: Andrew Morton <akpm@...ux-foundation.org>
> ---
>
> 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@...r.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@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists