Commit f7b634c5 authored by Paul Mackerras's avatar Paul Mackerras Committed by Linus Torvalds

[PATCH] ppc64: fix missing RELOCs, add linux,phandle property

This fixes a couple of bugs in arch/ppc64/kernel/prom.c.  We were
missing a couple of places where we needed to use RELOC().  I added the
RELOC in one case, and in the other, moved the variable that we were
accessing onto the stack (and reduced its size).  (We use the variable
to get a property value, but we aren't interested in the value, just in
whether the property exists or not.  Since we pass the size of the
variable to the OF getprop call, it won't overflow.) The effect of
missing the RELOCs would be that random memory locations get used on IBM
pSeries systems (possibly causing random boot failures). 

The other thing that this does is add a linux,phandle property to each
node, containing the phandle for the node, which is the token that OF
uses to identify the node.  Some nodes reference other nodes by means
of their phandle.  Without the linux,phandle property, userspace code
looking at the OF device-tree image in /proc/device-tree has no way of
knowing which other node is being referenced.
Signed-off-by: default avatarPaul Mackerras <paulus@samba.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 9102d1e2
...@@ -654,8 +654,6 @@ static void __init prom_initialize_lmb(void) ...@@ -654,8 +654,6 @@ static void __init prom_initialize_lmb(void)
#endif /* DEBUG_PROM */ #endif /* DEBUG_PROM */
} }
static char hypertas_funcs[1024];
static void __init static void __init
prom_instantiate_rtas(void) prom_instantiate_rtas(void)
{ {
...@@ -665,6 +663,7 @@ prom_instantiate_rtas(void) ...@@ -665,6 +663,7 @@ prom_instantiate_rtas(void)
struct systemcfg *_systemcfg = RELOC(systemcfg); struct systemcfg *_systemcfg = RELOC(systemcfg);
ihandle prom_rtas; ihandle prom_rtas;
u32 getprop_rval; u32 getprop_rval;
char hypertas_funcs[4];
#ifdef DEBUG_PROM #ifdef DEBUG_PROM
prom_print(RELOC("prom_instantiate_rtas: start...\n")); prom_print(RELOC("prom_instantiate_rtas: start...\n"));
...@@ -1556,7 +1555,7 @@ static void __init *__make_room(unsigned long *mem_start, unsigned long *mem_end ...@@ -1556,7 +1555,7 @@ static void __init *__make_room(unsigned long *mem_start, unsigned long *mem_end
if (*mem_end != RELOC(initrd_start)) if (*mem_end != RELOC(initrd_start))
prom_panic(RELOC("No memory for copy_device_tree")); prom_panic(RELOC("No memory for copy_device_tree"));
prom_print("Huge device_tree: moving initrd\n"); prom_print(RELOC("Huge device_tree: moving initrd\n"));
/* Move by 4M. */ /* Move by 4M. */
initrd_len = RELOC(initrd_end) - RELOC(initrd_start); initrd_len = RELOC(initrd_end) - RELOC(initrd_start);
*mem_end = RELOC(initrd_start) + 4 * 1024 * 1024; *mem_end = RELOC(initrd_start) + 4 * 1024 * 1024;
...@@ -1590,6 +1589,7 @@ inspect_node(phandle node, struct device_node *dad, ...@@ -1590,6 +1589,7 @@ inspect_node(phandle node, struct device_node *dad,
char *prev_name, *namep; char *prev_name, *namep;
unsigned char *valp; unsigned char *valp;
unsigned long offset = reloc_offset(); unsigned long offset = reloc_offset();
phandle ibm_phandle;
np = make_room(mem_start, mem_end, struct device_node); np = make_room(mem_start, mem_end, struct device_node);
memset(np, 0, sizeof(*np)); memset(np, 0, sizeof(*np));
...@@ -1652,23 +1652,24 @@ inspect_node(phandle node, struct device_node *dad, ...@@ -1652,23 +1652,24 @@ inspect_node(phandle node, struct device_node *dad,
prev_propp = &pp->next; prev_propp = &pp->next;
} }
/* Add a "linux_phandle" value */ /* Add a "linux,phandle" property. */
if (np->node) { namep = make_room(mem_start, mem_end, char[16]);
u32 ibm_phandle = 0; strcpy(namep, RELOC("linux,phandle"));
int len; pp = make_room(mem_start, mem_end, struct property);
pp->name = PTRUNRELOC(namep);
pp->length = sizeof(phandle);
valp = make_room(mem_start, mem_end, phandle);
pp->value = PTRUNRELOC(valp);
*(phandle *)valp = node;
*prev_propp = PTRUNRELOC(pp);
pp->next = NULL;
/* First see if "ibm,phandle" exists and use its value */ /* Set np->linux_phandle to the value of the ibm,phandle property
len = (int) if it exists, otherwise to the phandle for this node. */
call_prom(RELOC("getprop"), 4, 1, node, RELOC("ibm,phandle"), np->linux_phandle = node;
&ibm_phandle, sizeof(ibm_phandle)); if ((int)call_prom(RELOC("getprop"), 4, 1, node, RELOC("ibm,phandle"),
if (len < 0) { &ibm_phandle, sizeof(ibm_phandle)) > 0)
np->linux_phandle = np->node;
} else {
np->linux_phandle = ibm_phandle; np->linux_phandle = ibm_phandle;
}
}
*prev_propp = 0;
/* get the node's full name */ /* get the node's full name */
namep = (char *)*mem_start; namep = (char *)*mem_start;
......
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