APPLIED/cmnt [OEM-5.14] Re: [PATCH v3 0/1][SRU][OEM-5.13/OEM-5.14/I/U] add PCI bridge IDs for ADL-M/P/S

Timo Aaltonen tjaalton at ubuntu.com
Fri Sep 17 13:53:24 UTC 2021


On 16.9.2021 10.22, You-Sheng Yang wrote:
> BugLink: https://bugs.launchpad.net/bugs/1942160
> 
> [Impact]
> 
> VMD PCI-E bridges on ADL-M/P/S use new device IDs and therefore not
> covered by previous LTR reset fix in bug 1896598.
> 
> [Fix]
> 
> Additional entires to vmd_bridge_tbl in drivers/pci/quirks.c is
> required:
> * ADL-M: [8086:464d]
> * ADL-P: [8086:51b0]
> * ADL-S: [8086:7ab4]
> 
> However, from the document there is no complete/limited set of potential
> IDs. Instead, every assigned value for VMD capable platforms can be the
> device ID for PCI bridges behind VMD controller, because these PCI bridges
> are moved from the system root bus into VMD domain.
> 
> This Ubuntu sauced patch removes matching table and check if a bridge is
> attached to a vmd module driven device.
> 
> [Test Case]
> 
> $ sudo lspci -vvn | grep -E '^(1|\s+LnkCtl:)'
> ...
> 10000:e0:1d.4 0604: 8086:7ab4 (rev 10) (prog-if 00 [Normal decode])
>                  LnkCtl: ASPM L1 Enabled; RCB 64 bytes Disabled- CommClk+
> 10000:e1:00.0 0108: 8086:0975 (rev 03) (prog-if 02 [NVM Express])
>                  LnkCtl: ASPM L1 Enabled; RCB 64 bytes Disabled- CommClk+
> 
> [Where problems could occur]
> 
> This will reset zeroed PCI_LTR_MAX_SNOOP_LAT and PCI_LTR_MAX_NOSNOOP_LAT
> values to a fixed 3145728ns that is used on Windows. Devices might want
> to set itself to a smaller value, but while this is not yet correctly
> supported by VMD controller driver, so the device power consumption
> might be little worse than ideal.
> 
> [Other Info]
> 
> VMD ASPM was first supported for Tiger Lake cpus, so all kernel versions
> newer than 5.6 should have already the supporting function. This patch
> targets Alder Lake cpus that are supported since 5.13-oem and on.
> 
> v2: missed oem-5.14 in the nomination list
> v3: remove matching table and match by topology instead.
> 
> You-Sheng Yang (1):
>    UBUNTU: SAUCE: vmd: fixup bridge ASPM by driver name instead
> 
>   drivers/pci/quirks.c | 32 +++++++++++++++++++++++---------
>   1 file changed, 23 insertions(+), 9 deletions(-)
> 

applied to oem-5.14, thanks

I'm not sure about impish though, it will not fully support ADL-S anyway 
(and certainly not -M/P)


-- 
t



More information about the kernel-team mailing list