]> git.baikalelectronics.ru Git - kernel.git/commit
ACPI / PM: Leave Bus Master Arbitration enabled for suspend/resume
authorJonathan Nieder <jrnieder@gmail.com>
Sun, 8 Jul 2012 19:55:14 +0000 (21:55 +0200)
committerRafael J. Wysocki <rjw@sisk.pl>
Tue, 10 Jul 2012 19:50:59 +0000 (21:50 +0200)
commite19ba669ba53fcd25722fad580427e3edbc74bcd
tree641771422c54874c1b33d5b475ecd6f377f0e56d
parentb77fb3774c0aa30f00a1fd9893c11a4017b88b63
ACPI / PM: Leave Bus Master Arbitration enabled for suspend/resume

This is an old suspend/resume lockup fix:

commit 2780cc4660e1
Author: Len Brown <len.brown@intel.com>
Date:   Thu Dec 23 13:43:30 2004 -0500

    [ACPI] Fix suspend/resume lockup issue
    by leaving Bus Master Arbitration enabled.
    The ACPI spec mandates it be disabled only for C3.

    http://bugzilla.kernel.org/show_bug.cgi?id=3599

Signed-off-by: David Shaohua Li <shaohua.li@intel.com>
Signed-off-by: Len Brown <len.brown@intel.com>
The bug snuck back in in commit 7e07c95938cc (ACPICA: ACPI 5: Support
for new FADT SleepStatus, SleepControl registers, 2012-02-14),
presumably by copy/pasting a copy of the code without that fix for the
legacy case.

On affected machines, after that commit, the machine locks up hard on
resume from suspend.  The same fix as seven years ago still works.

Addresses <https://bugzilla.kernel.org/show_bug.cgi?id=43641>.

Reported-bisected-and-tested-by: Octavio Alvarez <alvarezp@alvarezp.com>
Reported-by: Adrian Knoth <adi@drcomp.erfurt.thur.de>
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
Cc: stable@vger.kernel.org
Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl>
drivers/acpi/acpica/hwsleep.c