summaryrefslogtreecommitdiffstats
path: root/arch/x86_64/pci/k8-bus.c
diff options
context:
space:
mode:
authorAndi Kleen <ak@suse.de>2005-08-23 03:14:27 +0200
committerLinus Torvalds <torvalds@g5.osdl.org>2005-08-23 19:59:39 -0700
commitd3813fcf105814d06b47fa586f6b61f3cff1cefc (patch)
tree88cc854eac01d12063cfc6722c46e73e91a77efb /arch/x86_64/pci/k8-bus.c
parent2bbfb16bf345acd81ab1e6e3d4b35964650517ac (diff)
downloadop-kernel-dev-d3813fcf105814d06b47fa586f6b61f3cff1cefc.zip
op-kernel-dev-d3813fcf105814d06b47fa586f6b61f3cff1cefc.tar.gz
[PATCH] x86_64: Don't oops at boot when empty Opteron node has IO
The code to detect IO links on Opteron would not check if the node had actually memory. This could lead to pci_bus_to_node returning an invalid node, which might cause crashes later when dma_alloc_coherent passes it to page_alloc_node(). The bug has been there forever but for some reason it is causing now crashes. Signed-off-by: Andi Kleen <ak@suse.de> Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'arch/x86_64/pci/k8-bus.c')
-rw-r--r--arch/x86_64/pci/k8-bus.c13
1 files changed, 11 insertions, 2 deletions
diff --git a/arch/x86_64/pci/k8-bus.c b/arch/x86_64/pci/k8-bus.c
index c2c38b5..d80c323 100644
--- a/arch/x86_64/pci/k8-bus.c
+++ b/arch/x86_64/pci/k8-bus.c
@@ -47,13 +47,22 @@ fill_mp_bus_to_cpumask(void)
* if there are no busses hanging off of the current
* ldt link then both the secondary and subordinate
* bus number fields are set to 0.
+ *
+ * RED-PEN
+ * This is slightly broken because it assumes
+ * HT node IDs == Linux node ids, which is not always
+ * true. However it is probably mostly true.
*/
if (!(SECONDARY_LDT_BUS_NUMBER(ldtbus) == 0
&& SUBORDINATE_LDT_BUS_NUMBER(ldtbus) == 0)) {
for (j = SECONDARY_LDT_BUS_NUMBER(ldtbus);
j <= SUBORDINATE_LDT_BUS_NUMBER(ldtbus);
- j++)
- pci_bus_to_node[j] = NODE_ID(nid);
+ j++) {
+ int node = NODE_ID(nid);
+ if (!node_online(node))
+ node = 0;
+ pci_bus_to_node[j] = node;
+ }
}
}
}
OpenPOWER on IntegriCloud