[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <5B3AAEAF6B46EA4D955DF7AD46C2C48516886FB954@AUSX7MCPS303.AMER.DELL.COM>
Date: Wed, 12 Nov 2014 10:18:25 -0600
From: <Jordan_Hargrave@...l.com>
To: <bhelgaas@...gle.com>, <David_Waggoner@...l.com>,
<Austin_Bolen@...l.com>
CC: <jdmason@...zu.us>, <linux-pci@...r.kernel.org>,
<linux-kernel@...r.kernel.org>
Subject: PCI Max payload setting
Jon,
We are still seeing problems with the current implementation of PCIE MaxPayload setting with hotplug devices (PCIE SSD).
The original implementation I proposed a few years ago used a bottom-up approach for configuring a PCI device when it was added to the system.
It would read the Parent MPS setting and configure the device to the speed of the parent (or fail to configure the device if parent speed was above device capability).
The implementation in the kernel uses a top-down approach, it is scanning all children devices and potentially setting the bridge speed based on the capability of child devices. If the default setting is ued (pcie_bus_config = PCIE_BUS_TUNE_OFF) this causes the device to fail as the default speed is 128 and bridge is configured to 256 or greater. The workaround for that is setting pcie_bus_perf. However I have seen issues with this on some systems if an I/O transaction is occurring (RAID card) while the bridge speed is being changed.
We have had to use my original code in both the mtip32xx and nvme drivers to support hotplug properly on our systems due to these shortcomings.
I am looking for a solution that will work properly on our systems out of the box. I think the bottom-up approach is still best. We are assuming the BIOS has already setup the payload of the bridges to optimal value.
--jordan hargrave
Dell Enterprise Linux Engineering
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists