From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: From: Mika Westerberg To: Bjorn Helgaas , "Rafael J. Wysocki" Cc: Len Brown , Mario.Limonciello@dell.com, Michael Jamet , Yehezkel Bernat , Mika Westerberg , Andy Shevchenko , linux-pci@vger.kernel.org, linux-acpi@vger.kernel.org Subject: [PATCH v3 0/5] PCI: Fixes for native PCIe and ACPI hotplug Date: Mon, 26 Feb 2018 16:21:07 +0300 Message-Id: <20180226132112.81447-1-mika.westerberg@linux.intel.com> Sender: linux-acpi-owner@vger.kernel.org List-ID: Hi, When Thunderbolt controller is configured to be in native enumeration mode it actually includes two slightly different modes. First there is non-RTD3 mode where the Thunderbolt host controller is only present when there is a device connected. The second one is RTD3 mode where the controller is always present. In non-RTD3 mode the Thunderbolt host controller (NHI) and USB host (xHCI) controller are not hotplugged using native PCIe hotplug but instead they will be hotplugged via BIOS triggered ACPI Notify() to the root port. This is done to preserve resources since the NHI and xHCI only need 1 MB of MMIO space and no additional buses. Currently Linux does not support this very well and ends up failing the hotplug in one way or another. More detailed explanation is in changelog of patch [4/5]. This series fixes this issue and in addition includes fixes for few other issues found during testing on a system that has Thunderbolt controller in non-RTD3 native PCIe enumeration mode. However, the fixes here are not in any way Thunderbolt specific and should be applicable to other systems as well. The previous versions of the patch series can be found here: v2: https://www.spinics.net/lists/linux-pci/msg69186.html v1: https://www.spinics.net/lists/linux-acpi/msg80607.html Changes from v2: - Added Rafael's tag to patch [1/5]. - Updated changelog of patch [1/5] to include more details about how the problem can be observed from dmesg and lspci output. Changes from v1: - Drop 'cmax - max ?: 1' and use similar construct than we use in second pass loop in patch [1/5]. - Drop unnecessary parentheses in patch [1/5]. - Added Rafael's tag to patches [2-5/5]. Mika Westerberg (5): PCI: Make sure all bridges reserve at least one bus number PCI: Take bridge window alignment into account when distributing resources PCI: pciehp: Clear Presence Detect and Data Link Layer Status Changed on resume ACPI / hotplug / PCI: Do not scan all bridges when native PCIe hotplug is used ACPI / hotplug / PCI: Mark stale PCI devices disconnected drivers/pci/hotplug/acpiphp.h | 1 + drivers/pci/hotplug/acpiphp_glue.c | 78 ++++++++++++++++++++++++++++++-------- drivers/pci/hotplug/pciehp.h | 2 +- drivers/pci/hotplug/pciehp_core.c | 2 +- drivers/pci/hotplug/pciehp_hpc.c | 13 ++++++- drivers/pci/probe.c | 11 ++++-- drivers/pci/setup-bus.c | 41 +++++++++++++++++++- 7 files changed, 126 insertions(+), 22 deletions(-) -- 2.16.1