On Wed, Jun 11, 2014 at 4:29 AM, Bjorn Helgaas <bhelgaas@xxxxxxxxxx> wrote: > On Tue, Jun 10, 2014 at 2:51 PM, Rafael J. Wysocki <rjw@xxxxxxxxxxxxx> wrote: >> From: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx> >> >> After relatively recent changes in the ACPI-based PCI hotplug >> (ACPIPHP) code, the acpiphp_check_host_bridge() executed for PCI >> host bridges via acpi_pci_root_scan_dependent() doesn't do anything >> useful, because those bridges do not have hotplug contexts. That >> happens by mistake, so fix it by making acpiphp_enumerate_slots() >> add hotplug contexts to PCI host bridges too and modify >> acpiphp_remove_slots() to drop those contexts for host bridges >> as appropriate. >> >> Link: https://bugzilla.kernel.org/show_bug.cgi?id=76901 >> Fixes: 2d8b1d566a5f (ACPI / hotplug / PCI: Get rid of check_sub_bridges()) >> Reported-and-tested-by: Gavin Guo <gavin.guo@xxxxxxxxxxxxx> >> Cc: 3.15+ <stable@xxxxxxxxxxxxxxx> # 3.15+ >> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx> > > Acked-by: Bjorn Helgaas <bhelgaas@xxxxxxxxxx> Thanks! > Rafael, do you want to merge this via your tree, since you merged the > original acpiphp rework? Yes, I'll take care of this. > I do have a small cleanup of acpiphp_glue.c in my queue, but it won't > conflict with this. > > Thanks a lot for fixing this! You're very welcome. :-) Rafael -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html