Re: [PATCH] PCI: Check for PCIe downtraining conditions

From: Alex G.
Date: Thu May 31 2018 - 12:01:54 EST




On 05/31/2018 10:54 AM, Sinan Kaya wrote:
> On 5/31/2018 11:46 AM, Alex G. wrote:
>>> https://lkml.org/lkml/2018/3/30/553
>> Oh, pcie_get_speed_cap()/pcie_get_width_cap() seems to handle the
>> capability. Not seeing one for status and speed name.
>>
>>> are you working on linux-next?
>> v4.17-rc7
>>
>
> I think everything you need is in the series. I don't know which linux
> tree this landed or if it landed. Probably, it is shipping for 4.18.
> Need some help from Bjorn where to locate these.
>
> Fri, 30 Mar 2018 16:04:40 -0500
>
> Bjorn Helgaas (6):
> bnx2x: Report PCIe link properties with pcie_print_link_status()
> bnxt_en: Report PCIe link properties with pcie_print_link_status()
> cxgb4: Report PCIe link properties with pcie_print_link_status()
> fm10k: Report PCIe link properties with pcie_print_link_status()
> ixgbe: Report PCIe link properties with pcie_print_link_status()
> PCI: Remove unused pcie_get_minimum_link()

I remember seeing some of these in my tree.

> Tal Gilboa (8):
> PCI: Add pcie_get_speed_cap() to find max supported link speed
> PCI: Add pcie_get_width_cap() to find max supported link width
> PCI: Add pcie_bandwidth_capable() to compute max supported link bandwidth
> PCI: Add pcie_bandwidth_available() to compute bandwidth available to device

I definitely have these.

> PCI: Add pcie_print_link_status() to log link speed and whether it's limited

This one, I have, but it's not what I need. This looks at the available
bandwidth from root port to endpoint, whereas I'm only interested in
downtraining between endpoint and upstream port.

Alex