Re: [Bug 55211] pci_disable_link_state PCIE_LINK_STATE_L0S no longerdisables ASPM for ath5k

From: Yinghai Lu
Date: Sun Mar 17 2013 - 13:19:54 EST

On Sun, Mar 17, 2013 at 8:50 AM, <bugzilla-daemon@xxxxxxxxxxxxxxxxxxx> wrote:

> --- Comment #9 from Roman Yepishev <roman.yepishev@xxxxxxxxx> 2013-03-17 15:50:23 ---
> Re-tested on two laptops - AOA150, ath5k device got ASPM disabled and on a
> Lenovo E420 I got ASPM disabled for iwlwifi-driven Intel Corporation Centrino
> Wireless-N 1000 [8086:0084].
> Also I found where a
> question was raised why pci_disable_link_state stopped doing anything for
> iwlwifi devices too.


Did you test first patch or second patch?

Please test second patch only on affected platforms.

old logic is quite strange, on boot path and hotplug path it have
different aspm_disabled setting.
as it could set aspm_disabled after pci root bus scanning.

Second patch will not restore to old logic, and just remove not needed
aspm_disabled checking
for disabling path.

So second patch is right fix, but it need more test.



Attachment: pci_acpi_osc_aspm_fix.patch
Description: Binary data

Attachment: disable_aspm_remove_not_needed_check.patch
Description: Binary data