On Sunday, July 08, 2012, Jonathan Nieder wrote: > Hi again, > > Jonathan Nieder wrote: > > > The bug snuck back in in commit 33620c5419e8 (ACPICA: Support for > > custom ACPICA build for ACPI 5 reduced hardware, 2012-02-14), > > presumably by copy/pasting from the wrong source for the legacy case. > > Um, wrong commit. Sorry, here's a fixed message. > > ACPI: Leave Bus Master Arbitration enabled for suspend/resume > > This is an old suspend/resume lockup fix: > > commit 2780cc4660e1 > Author: Len Brown <len.brown@xxxxxxxxx> > 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@xxxxxxxxx> > Signed-off-by: Len Brown <len.brown@xxxxxxxxx> > > The bug snuck back in in commit 2feec47d4c5f (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@xxxxxxxxxxxx> > Reported-by: Adrian Knoth <adi@xxxxxxxxxxxxxxxxxxxxx> > Signed-off-by: Jonathan Nieder <jrnieder@xxxxxxxxx> > Cc: <stable@xxxxxxxxxxxxxxx> # 3.4 > -- Can you please repost it with _both_ the changelog and the patch? Thanks, Rafael -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
