Prev: Make Intel 8-way Xeons boot again
Next: [tip:x86/uv] x86, uv: uv_global_gru_mmr_address() macro fix
From: Arjan van de Ven on 9 Jan 2010 16:50
On Fri, 8 Jan 2010 10:15:13 -0700
Alex Chiang <achiang(a)hp.com> wrote:

> Since you bisected this down to a commit from Arjan, it might
> help to cc him.


please send me "powertop -d" output, as well as the output of
"dmidecode"..... quite possible that that's enough for me to fix this..


--
Arjan van de Ven Intel Open Source Technology Centre
For development, discussion and tips for power savings,
visit http://www.lesswatts.org
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Dimitrios Apostolou on 9 Jan 2010 19:00
On Sat, 9 Jan 2010, Arjan van de Ven wrote:
> please send me "powertop -d" output, as well as the output of
> "dmidecode"..... quite possible that that's enough for me to fix this..

Hi, please see the attached powertop-before.out, right before modprobing
the processor module, and powertop-after.out right after that, and also
dmidecode.out, all on a minimally booted system.


Dimitris
From: Arjan van de Ven on 9 Jan 2010 19:10
On Sun, 10 Jan 2010 01:55:42 +0200 (EET)
Dimitrios Apostolou <jimis(a)gmx.net> wrote:

> L8400B series Notebook PC

can you try this patch?

diff --git a/drivers/acpi/processor_idle.c b/drivers/acpi/processor_idle.c
index d1676b1..6c3145b 100644
--- a/drivers/acpi/processor_idle.c
+++ b/drivers/acpi/processor_idle.c
@@ -110,6 +110,14 @@ static struct dmi_system_id __cpuinitdata processor_power_dmi_table[] = {
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},
{},
};



--
Arjan van de Ven Intel Open Source Technology Centre
For development, discussion and tips for power savings,
visit http://www.lesswatts.org
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Dimitrios Apostolou on 9 Jan 2010 19:40
On Sat, 9 Jan 2010, Arjan van de Ven wrote:

> On Sun, 10 Jan 2010 01:55:42 +0200 (EET)
> Dimitrios Apostolou <jimis(a)gmx.net> wrote:
>
>> L8400B series Notebook PC
>
> can you try this patch?
>

OK I'm compiling a new kernel right now but that will take a while, I only
have access to old hardware at the moment... What exactly this patch does?

BTW, how can I remove that irritating -O2 flag? I 'm used to compiling
with -O0 my debug builds in userland, because compilation is *many times*
faster. I should be really useful for bisections.


Thanks,
Dimitris

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Arjan van de Ven on 9 Jan 2010 19:50
On Sun, 10 Jan 2010 02:32:11 +0200 (EET)
Dimitrios Apostolou <jimis(a)gmx.net> wrote:

> On Sat, 9 Jan 2010, Arjan van de Ven wrote:
>
> > On Sun, 10 Jan 2010 01:55:42 +0200 (EET)
> > Dimitrios Apostolou <jimis(a)gmx.net> wrote:
> >
> >> L8400B series Notebook PC
> >
> > can you try this patch?
> >
>
> OK I'm compiling a new kernel right now but that will take a while, I
> only have access to old hardware at the moment... What exactly this
> patch does?
>

basically it appears that your machine, when the kernel asks for C2,
exits C2 immediately again.

The old algorithm somehow caught this and stopped asking for C2 most of
the time; the new algorithm doesn't see any activity and asks for C2
again.

What the patch does is tell the kernel to just not use C2 at all...



--
Arjan van de Ven Intel Open Source Technology Centre
For development, discussion and tips for power savings,
visit http://www.lesswatts.org
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
 |  Next  |  Last
Pages: 1 2 3 4
Prev: Make Intel 8-way Xeons boot again
Next: [tip:x86/uv] x86, uv: uv_global_gru_mmr_address() macro fix