mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-08-23 17:30:02 +00:00
PCI / ACPI / PM: Resume bridges w/o drivers on suspend-to-RAM
It is reported that commitc62ec4610c
(PM / core: Fix direct_complete handling for devices with no callbacks) introduced a system suspend regression on Samsung 305V4A by allowing a PCI bridge (not a PCIe port) to stay in D3 over suspend-to-RAM, which is a side effect of setting power.direct_complete for the children of that bridge that have no PM callbacks. On the majority of systems PCI bridges are not allowed to be runtime-suspended (the power/control sysfs attribute is set to "on" for them by default), but user space can change that setting and if it does so and a given bridge has no children with PM callbacks, the direct_complete optimization will be applied to it and it will stay in suspend over system suspend. Apparently, that confuses the platform firmware on the affected machine and that may very well happen elsewhere, so avoid the direct_complete optimization for PCI bridges with no drivers (if there is a driver, it should take care of the PM handling) on suspend-to-RAM altogether (that should not matter for suspend-to-idle as platform firmware is not involved in it). Fixes:c62ec4610c
(PM / core: Fix direct_complete handling for devices with no callbacks) Link: https://bugzilla.kernel.org/show_bug.cgi?id=199941 Reported-by: n0000b.n000b@gmail.com Tested-by: n0000b.n000b@gmail.com Reviewed-by: Mika Westerberg <mika.westerberg@linux.intel.com> Acked-by: Bjorn Helgaas <bhelgaas@google.com> Cc: 4.15+ <stable@vger.kernel.org> # 4.15+ Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
This commit is contained in:
parent
7daf201d7f
commit
26112ddc25
1 changed files with 12 additions and 0 deletions
|
@ -629,6 +629,18 @@ static bool acpi_pci_need_resume(struct pci_dev *dev)
|
|||
{
|
||||
struct acpi_device *adev = ACPI_COMPANION(&dev->dev);
|
||||
|
||||
/*
|
||||
* In some cases (eg. Samsung 305V4A) leaving a bridge in suspend over
|
||||
* system-wide suspend/resume confuses the platform firmware, so avoid
|
||||
* doing that, unless the bridge has a driver that should take care of
|
||||
* the PM handling. According to Section 16.1.6 of ACPI 6.2, endpoint
|
||||
* devices are expected to be in D3 before invoking the S3 entry path
|
||||
* from the firmware, so they should not be affected by this issue.
|
||||
*/
|
||||
if (pci_is_bridge(dev) && !dev->driver &&
|
||||
acpi_target_system_state() != ACPI_STATE_S0)
|
||||
return true;
|
||||
|
||||
if (!adev || !acpi_device_power_manageable(adev))
|
||||
return false;
|
||||
|
||||
|
|
Loading…
Reference in a new issue