Re: 100% C0 with 2.6.25-rc

Previous thread: Re: [gentoo-amd64] Re: BT8x8 by Chris Brennan on Thursday, February 21, 2008 - 12:33 pm. (1 message)

Next thread: [PATCH] ptrace: fix 2.6.25 ptrace_bts_config structure declaration by Dave Anderson on Thursday, February 21, 2008 - 12:45 pm. (2 messages)
To: Jan Willies <jan@...>
Cc: <linux-acpi@...>, Ingo Molnar <mingo@...>, LKML <linux-kernel@...>, Thomas Gleixner <tglx@...>, Venki Pallipadi <venkatesh.pallipadi@...>
Date: Thursday, February 21, 2008 - 12:33 pm

Let's see what the experts say.

Thanks,
Rafael
--

To: Rafael J. Wysocki <rjw@...>
Cc: <linux-acpi@...>, Ingo Molnar <mingo@...>, LKML <linux-kernel@...>, Thomas Gleixner <tglx@...>, Venki Pallipadi <venkatesh.pallipadi@...>
Date: Friday, February 22, 2008 - 11:56 am

After a lot of bisecting I've found the bad commit:

9b12e18cdc1553de62d931e73443c806347cd974 is first bad commit
commit 9b12e18cdc1553de62d931e73443c806347cd974
Author: venkatesh.pallipadi@intel.com <venkatesh.pallipadi@intel.com>
Date: Thu Jan 31 17:35:05 2008 -0800

ACPI: cpuidle: Support C1 idle time accounting

Show C1 idle time in /sysfs cpuidle interface. C1 idle time may not
be entirely accurate in all cases. It includes the time spent
in the interrupt handler after wakeup with "hlt" based C1. But, it will
be accurate with "mwait" based C1.

Reverting the commit brings my laptop back to C2.

HTH

- Jan
--

To: Jan Willies <jan@...>, Rafael J. Wysocki <rjw@...>
Cc: <linux-acpi@...>, Ingo Molnar <mingo@...>, LKML <linux-kernel@...>, Thomas Gleixner <tglx@...>
Date: Friday, February 22, 2008 - 1:33 pm

One question. Do you have CONFIG_CPU_IDLE enabled in your confg?

Thanks,
Venki
--

To: Pallipadi, Venkatesh <venkatesh.pallipadi@...>
Cc: Rafael J. Wysocki <rjw@...>, <linux-acpi@...>, Ingo Molnar <mingo@...>, LKML <linux-kernel@...>, Thomas Gleixner <tglx@...>
Date: Friday, February 22, 2008 - 1:50 pm

Yes, I have. Attached is my .config file

- Jan

To: Jan Willies <jan@...>
Cc: Rafael J. Wysocki <rjw@...>, <linux-acpi@...>, Ingo Molnar <mingo@...>, LKML <linux-kernel@...>, Thomas Gleixner <tglx@...>
Date: Friday, February 22, 2008 - 1:55 pm

Can you try with it disabled and check whether you still see the
problem?

Thanks,
Venki
--

To: Pallipadi, Venkatesh <venkatesh.pallipadi@...>
Cc: Rafael J. Wysocki <rjw@...>, <linux-acpi@...>, Ingo Molnar <mingo@...>, LKML <linux-kernel@...>, Thomas Gleixner <tglx@...>
Date: Friday, February 22, 2008 - 2:16 pm

Yup you are right, that fixes the problem as well.

- Jan
--

To: Jan Willies <jan@...>, Rafael J. Wysocki <rjw@...>
Cc: <linux-acpi@...>, Ingo Molnar <mingo@...>, LKML <linux-kernel@...>, Thomas Gleixner <tglx@...>
Date: Friday, February 22, 2008 - 1:03 pm

Thanks for the bisect info. I will look at the bad side effects that patch may be having and I should have a patch for you to test later today....

Thanks,
Venki
--

Previous thread: Re: [gentoo-amd64] Re: BT8x8 by Chris Brennan on Thursday, February 21, 2008 - 12:33 pm. (1 message)

Next thread: [PATCH] ptrace: fix 2.6.25 ptrace_bts_config structure declaration by Dave Anderson on Thursday, February 21, 2008 - 12:45 pm. (2 messages)