Commit 4d01e55b authored by Heiner Kallweit's avatar Heiner Kallweit Committed by Jakub Kicinski
Browse files

r8169: log that system vendor flags ASPM as safe 



ASPM isn't disabled if system vendor flags it as safe. Log this,
in order to know whom to blame if a user complains about ASPM
issues on such a system.

Signed-off-by: default avatarHeiner Kallweit <hkallweit1@gmail.com>
Link: https://patch.msgid.link/a532b46b-ef68-4d68-a129-35ff0ee35150@gmail.com


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent dd749a3c

drivers/net/ethernet/realtek/r8169_main.c

+4 −2
Original line number Diff line number Diff line
@@ -5441,10 +5441,12 @@ static int rtl_init_one(struct pci_dev *pdev, const struct pci_device_id *ent) 
	/* Disable ASPM L1 as that cause random device stop working

	 * problems as well as full system hangs for some PCIe devices users.

	 */

	if (rtl_aspm_is_safe(tp))

	if (rtl_aspm_is_safe(tp)) {

		dev_info(&pdev->dev, "System vendor flags ASPM as safe\n");

		rc = 0;

	else

	} else {

		rc = pci_disable_link_state(pdev, PCIE_LINK_STATE_L1);

	}

	tp->aspm_manageable = !rc;



	tp->dash_type = rtl_get_dash_type(tp);