On Tue, 26 Jan 2010 13:43:15 +0100 "Rafael J. Wysocki" <rjw@xxxxxxx> wrote: > On Tuesday 26 January 2010, Arjan van de Ven wrote: > > On Mon, 25 Jan 2010 21:52:32 +0100 > > "Rafael J. Wysocki" <rjw@xxxxxxx> 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@xxxxxxxxxxxxxxx> 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@xxxxxxxxxxxxxxx> Reported-by: <akwatts@xxxxxxxxx> Cc: Dimitrios Apostolou <jimis@xxxxxxx> Cc: Alex Chiang <achiang@xxxxxx> Cc: Len Brown <lenb@xxxxxxxxxx> Cc: Bjorn Helgaas <bjorn.helgaas@xxxxxx> Cc: Yinghai Lu <yinghai@xxxxxxxxxx> Cc: Venkatesh Pallipadi <venkatesh.pallipadi@xxxxxxxxx> Cc: Lin Ming <ming.m.lin@xxxxxxxxx> Cc: "Rafael J. Wysocki" <rjw@xxxxxxx> Cc: <stable@xxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- 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 kernel-testers" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
