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@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 linux-kernel" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > 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 kernel-testers" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
