Thread (12 messages) 12 messages, 3 authors, 2018-02-12

Re: [PATCH] Revert "cpuidle: Make drivers initialize polling state"

From: Ville Syrjälä <hidden>
Date: 2018-02-05 14:04:59
Also in: lkml

On Sun, Feb 04, 2018 at 10:18:07AM +0100, Rafael J. Wysocki wrote:
On Sun, Feb 4, 2018 at 10:12 AM, Rafael J. Wysocki [off-list ref] wrote:
quoted
On Mon, Jan 22, 2018 at 5:27 PM, Ville Syrjala
[off-list ref] wrote:
quoted
From: Ville Syrjälä <redacted>

This reverts commit 1b39e3f813b4685c7a30ae964d5529a1b0e3a286.

Makes my P3 machine oops somewhere in cpuidle. I suspect
CONFIG_ACPI=n may have something to do with this.
And if you don't do CONFIG_ACPI=n, does it still oops?
Don't think I actually tried that. I can give it a whirl tonight.

I think this machine should actually have ACPI, but it inherited
the .config from a P2 machine that did not. Apparently I was too
lazy to change .config when I swapped in the "new" machine.
quoted
Anyway, there are later changes depending on this one, so reverting it
won't work in general.

Let me look deeper at this.
What's there in /sys/devices/system/cpu/cpuidle/current_driver on your
system with the problematic commit reverted?
# cat /sys/devices/system/cpu/cpuidle/current_driver
apm_idle

-- 
Ville Syrjälä
Intel OTC
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help