]> git.baikalelectronics.ru Git - kernel.git/commit
ACPI: processor: idle: Avoid falling back to C3 type C-states
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Thu, 21 Apr 2022 13:36:34 +0000 (16:36 +0300)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Thu, 21 Apr 2022 17:55:41 +0000 (19:55 +0200)
commitdea78c2408fc8ff850d7ea4bcd87ef350f996a81
treee0437ec315350966bf2bf6ae3c90c256b6664067
parent8dd3e26bbbc758feb75d55e38f0ba1ef6979f6d1
ACPI: processor: idle: Avoid falling back to C3 type C-states

The "safe state" index is used by acpi_idle_enter_bm() to avoid
entering a C-state that may require bus mastering to be disabled
on entry in the cases when this is not going to happen.  For this
reason, it should not be set to point to C3 type of C-states, because
they may require bus mastering to be disabled on entry in principle.

This was broken by commit ea47b5b79918 ("ACPI: processor idle: Allow
playing dead in C3 state") which inadvertently allowed the "safe
state" index to point to C3 type of C-states.

This results in a machine that won't boot past the point when it first
enters C3. Restore the correct behaviour (either demote to C1/C2, or
use C3 but also set ARB_DIS=1).

I hit this on a Fujitsu Siemens Lifebook S6010 (P3) machine.

Fixes: ea47b5b79918 ("ACPI: processor idle: Allow playing dead in C3 state")
Cc: 5.16+ <stable@vger.kernel.org> # 5.16+
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Tested-by: Woody Suwalski <wsuwalski@gmail.com>
[ rjw: Subject and changelog adjustments ]
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/acpi/processor_idle.c