On 2/22/2018 9:57 PM, Bjorn Helgaas wrote:>
Suggested text: pci_info(dev, "available bandwidth %dGB/s (capable of %dGB/s)\n", ... I'm not sure this merits a pci_warn(). In some cases a user might be able to move the device to a different slot or something, but I'm sure there are platforms where it is impossible for the user to do anything about this, and I don't think we need to warn about that.
We face a lot of customer "bugs" due to lower than expected PCIe bandwidth. I think an info print would be too light in this case. Even if the user has nothing to do it should be aware of the problem.