PCI: Make pci_rescan_bus handle add_list
authorYinghai Lu <yinghai@kernel.org>
Sat, 21 Jan 2012 10:08:23 +0000 (02:08 -0800)
committerJesse Barnes <jbarnes@virtuousgeek.org>
Tue, 14 Feb 2012 16:44:53 +0000 (08:44 -0800)
This allows us to allocate resources to hotplug bridges during
remove/rescan.

We need to move the function to setup-bus.c so it can use
__pci_bus_size_bridges and __pci_bus_assign_resources directly to take
the add_list resource tracking list.

Signed-off-by: Yinghai Lu <yinghai@kernel.org>
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
drivers/pci/probe.c
drivers/pci/setup-bus.c

index 0e84e8c2a6d0357bf355f40fcedc1314281d8ca8..aad7d0ff6b082888c2361bb2899b5177693809bd 100644 (file)
@@ -1691,38 +1691,6 @@ unsigned int __ref pci_rescan_bus_bridge_resize(struct pci_dev *bridge)
        return max;
 }
 
-/**
- * pci_rescan_bus - scan a PCI bus for devices.
- * @bus: PCI bus to scan
- *
- * Scan a PCI bus and child buses for new devices, adds them,
- * and enables them.
- *
- * Returns the max number of subordinate bus discovered.
- */
-unsigned int __ref pci_rescan_bus(struct pci_bus *bus)
-{
-       unsigned int max;
-       struct pci_dev *dev;
-
-       max = pci_scan_child_bus(bus);
-
-       down_read(&pci_bus_sem);
-       list_for_each_entry(dev, &bus->devices, bus_list)
-               if (dev->hdr_type == PCI_HEADER_TYPE_BRIDGE ||
-                   dev->hdr_type == PCI_HEADER_TYPE_CARDBUS)
-                       if (dev->subordinate)
-                               pci_bus_size_bridges(dev->subordinate);
-       up_read(&pci_bus_sem);
-
-       pci_bus_assign_resources(bus);
-       pci_enable_bridges(bus);
-       pci_bus_add_devices(bus);
-
-       return max;
-}
-EXPORT_SYMBOL_GPL(pci_rescan_bus);
-
 EXPORT_SYMBOL(pci_add_new_bus);
 EXPORT_SYMBOL(pci_scan_slot);
 EXPORT_SYMBOL(pci_scan_bridge);
index 97c1eda96e649cb01ace401dd11562eb564a409b..c09c67ab5612cd500ebb078018ddbf07bea8abb7 100644 (file)
@@ -1357,3 +1357,42 @@ enable_all:
        pci_enable_bridges(parent);
 }
 EXPORT_SYMBOL_GPL(pci_assign_unassigned_bridge_resources);
+
+#ifdef CONFIG_HOTPLUG
+/**
+ * pci_rescan_bus - scan a PCI bus for devices.
+ * @bus: PCI bus to scan
+ *
+ * Scan a PCI bus and child buses for new devices, adds them,
+ * and enables them.
+ *
+ * Returns the max number of subordinate bus discovered.
+ */
+unsigned int __ref pci_rescan_bus(struct pci_bus *bus)
+{
+       unsigned int max;
+       struct pci_dev *dev;
+       struct resource_list_x add_list; /* list of resources that
+                                       want additional resources */
+
+       max = pci_scan_child_bus(bus);
+
+       add_list.next = NULL;
+       down_read(&pci_bus_sem);
+       list_for_each_entry(dev, &bus->devices, bus_list)
+               if (dev->hdr_type == PCI_HEADER_TYPE_BRIDGE ||
+                   dev->hdr_type == PCI_HEADER_TYPE_CARDBUS)
+                       if (dev->subordinate)
+                               __pci_bus_size_bridges(dev->subordinate,
+                                                        &add_list);
+       up_read(&pci_bus_sem);
+       __pci_bus_assign_resources(bus, &add_list, NULL);
+       BUG_ON(add_list.next);
+
+       pci_enable_bridges(bus);
+       pci_bus_add_devices(bus);
+
+       return max;
+}
+EXPORT_SYMBOL_GPL(pci_rescan_bus);
+#endif