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

[Len Brown <lenb-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>]

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/
>