Commit 8424d759 authored by Yinghai Lu's avatar Yinghai Lu Committed by Jesse Barnes

PCI: Use add_list in pcie hotplug path.

We need add size for hot plug path when pluging in hotplug chassis
without cards.

-v2: change descriptions. make it applicable after "pci: Check bridge
     resources after resource allocation."
Signed-off-by: default avatarYinghai Lu <yinghai@kernel.org>
Signed-off-by: default avatarJesse Barnes <jbarnes@virtuousgeek.org>
parent 3e6e0d80
...@@ -330,13 +330,14 @@ static void __assign_resources_sorted(struct resource_list *head, ...@@ -330,13 +330,14 @@ static void __assign_resources_sorted(struct resource_list *head,
} }
static void pdev_assign_resources_sorted(struct pci_dev *dev, static void pdev_assign_resources_sorted(struct pci_dev *dev,
struct resource_list_x *add_head,
struct resource_list_x *fail_head) struct resource_list_x *fail_head)
{ {
struct resource_list head; struct resource_list head;
head.next = NULL; head.next = NULL;
__dev_sort_resources(dev, &head); __dev_sort_resources(dev, &head);
__assign_resources_sorted(&head, NULL, fail_head); __assign_resources_sorted(&head, add_head, fail_head);
} }
...@@ -1006,17 +1007,19 @@ void __ref pci_bus_assign_resources(const struct pci_bus *bus) ...@@ -1006,17 +1007,19 @@ void __ref pci_bus_assign_resources(const struct pci_bus *bus)
EXPORT_SYMBOL(pci_bus_assign_resources); EXPORT_SYMBOL(pci_bus_assign_resources);
static void __ref __pci_bridge_assign_resources(const struct pci_dev *bridge, static void __ref __pci_bridge_assign_resources(const struct pci_dev *bridge,
struct resource_list_x *add_head,
struct resource_list_x *fail_head) struct resource_list_x *fail_head)
{ {
struct pci_bus *b; struct pci_bus *b;
pdev_assign_resources_sorted((struct pci_dev *)bridge, fail_head); pdev_assign_resources_sorted((struct pci_dev *)bridge,
add_head, fail_head);
b = bridge->subordinate; b = bridge->subordinate;
if (!b) if (!b)
return; return;
__pci_bus_assign_resources(b, NULL, fail_head); __pci_bus_assign_resources(b, add_head, fail_head);
switch (bridge->class >> 8) { switch (bridge->class >> 8) {
case PCI_CLASS_BRIDGE_PCI: case PCI_CLASS_BRIDGE_PCI:
...@@ -1291,6 +1294,8 @@ pci_assign_unassigned_resources(void) ...@@ -1291,6 +1294,8 @@ pci_assign_unassigned_resources(void)
void pci_assign_unassigned_bridge_resources(struct pci_dev *bridge) void pci_assign_unassigned_bridge_resources(struct pci_dev *bridge)
{ {
struct pci_bus *parent = bridge->subordinate; struct pci_bus *parent = bridge->subordinate;
struct resource_list_x add_list; /* list of resources that
want additional resources */
int tried_times = 0; int tried_times = 0;
struct resource_list_x head, *list; struct resource_list_x head, *list;
int retval; int retval;
...@@ -1298,11 +1303,12 @@ void pci_assign_unassigned_bridge_resources(struct pci_dev *bridge) ...@@ -1298,11 +1303,12 @@ void pci_assign_unassigned_bridge_resources(struct pci_dev *bridge)
IORESOURCE_PREFETCH; IORESOURCE_PREFETCH;
head.next = NULL; head.next = NULL;
add_list.next = NULL;
again: again:
pci_bus_size_bridges(parent); __pci_bus_size_bridges(parent, &add_list);
__pci_bridge_assign_resources(bridge, &head); __pci_bridge_assign_resources(bridge, &add_list, &head);
BUG_ON(add_list.next);
tried_times++; tried_times++;
if (!head.next) if (!head.next)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment