Skip to content

Commit 90ca51e

Browse files
hkallweitdavem330
authored andcommitted
r8169: fix ASPM-related issues on a number of systems with NIC version from RTL8168h
This effectively reverts 4b5f82f. On a number of systems ASPM L1 causes tx timeouts with RTL8168h, see referenced bug report. Fixes: 4b5f82f ("r8169: enable ASPM L1/L1.1 from RTL8168h") Cc: [email protected] Closes: https://bugzilla.kernel.org/show_bug.cgi?id=217814 Signed-off-by: Heiner Kallweit <[email protected]> Signed-off-by: David S. Miller <[email protected]>
1 parent ec1b908 commit 90ca51e

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

drivers/net/ethernet/realtek/r8169_main.c

-4
Original file line numberDiff line numberDiff line change
@@ -5239,13 +5239,9 @@ static int rtl_init_one(struct pci_dev *pdev, const struct pci_device_id *ent)
52395239

52405240
/* Disable ASPM L1 as that cause random device stop working
52415241
* problems as well as full system hangs for some PCIe devices users.
5242-
* Chips from RTL8168h partially have issues with L1.2, but seem
5243-
* to work fine with L1 and L1.1.
52445242
*/
52455243
if (rtl_aspm_is_safe(tp))
52465244
rc = 0;
5247-
else if (tp->mac_version >= RTL_GIGA_MAC_VER_46)
5248-
rc = pci_disable_link_state(pdev, PCIE_LINK_STATE_L1_2);
52495245
else
52505246
rc = pci_disable_link_state(pdev, PCIE_LINK_STATE_L1);
52515247
tp->aspm_manageable = !rc;

0 commit comments

Comments
 (0)