Hi Folks,
This series of patches is critical in supporting the NVMe SSD hot plug on the Flash JBOF platforms such Facebook/OCP Lightning. The PCIe hotplug improvements have been presented in multiple conferences (namely OCP Summit 17 in Santa Clara and Linux Vault 17 in Cambridge) in the past month. These changes have been validated by the Facebook internal testing as well as multiple ODM vendors testing teams.
Are there additional comments on this? I talked to Keith at the Vault last week. He mentioned there was no outstanding review feedback to be addressed. We would appreciate if we can upstream this patch ASAP!
Tested-by: Krishna Dhulipala <krishnad@xxxxxx>
Reviewed-by: Wei Zhang <wzhang@xxxxxx>
Thanks!
-Wei
--
wei zhang | software engineer | facebook
wzhang@xxxxxx | (408) 460-4803
On 2/7/17, 11:32 AM, "Keith Busch" <keith.busch@xxxxxxxxx> wrote:
The code is functionally equivalent to v5. I've fixed the style
points mentioned in review on line warpping, adjusted the change log
on the msi/msix skippage to not exclude mentioning msi, and added the
accumulated Reviewed-by's.
Keith Busch (5):
pci: Export pci device config accessors
pci: Add device disconnected state
pci: No config access for disconnected devices
pci/msix: Skip disabling disconnected devices
pci: Quick return for pci_device_is_present
drivers/pci/access.c | 56 ++++++++++++++++++++++++++++++++++++++++
drivers/pci/hotplug/pciehp_pci.c | 6 +++++
drivers/pci/msi.c | 7 ++++-
drivers/pci/pci.c | 2 ++
drivers/pci/pci.h | 14 ++++++++++
drivers/pci/pcie/pcie-dpc.c | 5 ++++
include/linux/pci.h | 34 ++++++------------------
7 files changed, 97 insertions(+), 27 deletions(-)
--
2.7.2
