On Tue, 18 Sep 2012 11:57:59 -0700 Yinghai Lu <yinghai@xxxxxxxxxx> wrote: > On Tue, Sep 18, 2012 at 11:41 AM, Yinghai Lu <yinghai@xxxxxxxxxx> wrote: > > On Mon, Sep 17, 2012 at 11:12 PM, Taku Izumi <izumi.taku@xxxxxxxxxxxxxx> wrote: > >> Hi all, > >> > >> This patchset is 3rd version of the following: > >> http://marc.info/?l=linux-pci&m=134457928913775&w=2 > >> http://marc.info/?l=linux-pci&m=134665909321684&w=2 > >> > >> v2 -> v3: > >> - change acpi_pci_drivers' add/remove interface according to Bjorn's suggestion > >> - abandon RCU for protecting acpi_pci_roots list according to Bjorn's suggestion > >> - change acpi_pci_find_root implementation to avoid potential deadlock > >> > >> * [PATCH v3 1/8] ACPI, PCI: Use normal list for struct acpi_pci_driver > >> * [PATCH v3 2/8] ACPI, PCI: Notify acpi_pci_drivers when hot-plugging PCI root bridges > >> * [PATCH v3 3/8] ACPI, PCI: add acpi_pci_drivers protection > >> * [PATCH v3 4/8] ACPI, PCI: change acpi_pci_drivers' add/remove interface > >> * [PATCH v3 5/8] ACPI, PCI: change acpi_pci_find_root implementation > >> * [PATCH v3 6/8] ACPI, PCI: add acpi_pci_roots protection > >> * [PATCH v3 7/8] ACPI, PCI: add hostbridge removal function > >> * [PATCH v3 8/8] ACPI, PCI: add resoruce-assign code for devices under hot-added hostbridge > > > > now it is more similar to my for-pci-root-bus-hotplug-part4 and part5 > > > > http://git.kernel.org/?p=linux/kernel/git/yinghai/linux-yinghai.git;a=shortlog;h=refs/heads/for-pci-root-bus-hotplug-part5 > > > > big difference: > > I separated code from acpiphp_glue.c to pci_root_hp.c for root bridge > > hotplug support > > > > I thought we all agreed on that acpi root bridge hotplug plug need to > > be separated from acpiphp at first. > > > > http://git.kernel.org/?p=linux/kernel/git/yinghai/linux-yinghai.git;a=commitdiff;h=ab9fd4ccbfb6120a4a4adf9833487d656b0501d3 > > > > and CONFIG_HOTPLUG will always be enabled, so root bridge hotplug will > > be always enabled. > > acpiphp will still be used as one module, and to keep it simple pciehp > > will be used for pcie etc. > > That make the system BIOS much simple to avoid acpiphp. > > also current acpiphp only have root bridge add notification. > and your code do not add any code for remove notification. > > So how to you test your code with removal path? You were saying how to hot-remove hostbridge ? I tested by ejecting ACPI container which includes hostbridge. acpiphp is not the only way to hotplug hostbridge, so I ommit acpiphp work from this patchset. This patchset is core part. Best regards, Taku Izumi <izumi.taku@xxxxxxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
