Re: [Bug #15040] High cpu temperature with 2.6.32

Len Brown
2010-02-16T09:12:04Z

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-KKrjLPT3xs0@public.gmane.org> wrote:
>
> > On Tuesday 26 January 2010, Arjan van de Ven wrote:
> > > On Mon, 25 Jan 2010 21:52:32 +0100
> > > "Rafael J. Wysocki" <rjw-KKrjLPT3xs0@public.gmane.org> 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-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
>
> 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-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
> Reported-by: <akwatts-DaQTI0RpDDMAvxtiuMwx3w@public.gmane.org>
> Cc: Dimitrios Apostolou <jimis-hi6Y0CQ0nG0@public.gmane.org>
> Cc: Alex Chiang <achiang-VXdhtT5mjnY@public.gmane.org>
> Cc: Len Brown <lenb-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
> Cc: Bjorn Helgaas <bjorn.helgaas-VXdhtT5mjnY@public.gmane.org>
> Cc: Yinghai Lu <yinghai-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
> Cc: Venkatesh Pallipadi <venkatesh.pallipadi-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
> Cc: Lin Ming <ming.m.lin-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
> Cc: "Rafael J. Wysocki" <rjw-KKrjLPT3xs0@public.gmane.org>
> Cc: <stable-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
> Signed-off-by: Andrew Morton <akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.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-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>