Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 13 Nov 2020 17:38:21 +0100
From:      martin@vx.sk
To:        Alexander Motin <mav@freebsd.org>
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-12@freebsd.org
Subject:   Re: svn commit: r367359 - stable/12/sys/dev/pci
Message-ID:  <20201113173821.Horde.jq1A9wp72tHUvM10uhYxhOq@mail.vx.sk>
In-Reply-To: <202011050257.0A52veoR086637@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
  I am encountering a problem with hw.pci.enable_aspm:

System: ASUS RS500A-E10-RS12U
CPU: AMD EPYC 7502 32-Core.
NVMe drive model: Micron 9300 PRO
Number of NVMe drives: 10

  Using X2APIC

When hw.pci.enable_aspm is enabled, PCIe hot plug of Micron NVMe =20
drives does not work (including detection on boot) and the system =20
reports the following errors for the vast majority of the drives:

  Nov 13 15:11:30 xxx kernel: pcib9: Timed out waiting for Data Link =20
Layer Active
Nov 13 15:12:17 xxx kernel: pcib30: Timed out waiting for Data Link =20
Layer Active
Nov 13 15:12:44 xxx kernel: pcib29: Timed out waiting for Data Link =20
Layer Active
Nov 13 15:13:04 xxx kernel: pcib11: Timed out waiting for Data Link =20
Layer Active
...

Flipping the tunable back to 0 workarounds the problem.

Reproduced both on latest head and stable/12.

Best regards,
mm

Quoting Alexander Motin <mav@freebsd.org>:

> Author: mav
> Date: Thu Nov=C2=A0 5 02:57:40 2020
> New Revision: 367359
> URL: https://svnweb.freebsd.org/changeset/base/367359
>
> Log:
> MFC r364038: Enable hw.pci.enable_aspm tunable by default.
>
> While effects on power saving is only a guess, effects on hot-plug are
> clearly visible.=C2=A0 Lets try to enable it and see what happen.
>
> Modified:
> stable/12/sys/dev/pci/pci.c
> Directory Properties:
> stable/12/=C2=A0 =C2=A0(props changed)
>
> Modified: stable/12/sys/dev/pci/pci.c
> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D
> --- stable/12/sys/dev/pci/pci.c=C2=A0 =C2=A0 =C2=A0 =C2=A0 Thu Nov=C2=A0=
 5 02:12:33 2020=C2=A0 =C2=A0 =C2=A0 =20
> =C2=A0 (r367358)
> +++ stable/12/sys/dev/pci/pci.c=C2=A0 =C2=A0 =C2=A0 =C2=A0 Thu Nov=C2=A0=
 5 02:57:40 2020=C2=A0 =C2=A0 =C2=A0 =20
> =C2=A0 (r367359)
> @@ -411,7 +411,7 @@ static int pci_enable_ari =3D 1;
> SYSCTL_INT(_hw_pci, OID_AUTO, enable_ari, CTLFLAG_RDTUN, &pci_enable_ar=
i,
> =C2=A0 =C2=A0 0, "Enable support for PCIe Alternative RID Interpretatio=
n");
>
> -int pci_enable_aspm;
> +int pci_enable_aspm =3D 1;
> SYSCTL_INT(_hw_pci, OID_AUTO, enable_aspm, CTLFLAG_RDTUN, =20
> &pci_enable_aspm,=C2=A0 =C2=A0 0, "Enable support for PCIe Active State=
 Power =20
> Management");



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20201113173821.Horde.jq1A9wp72tHUvM10uhYxhOq>