diff options
author | David S. Miller <davem@sunset.davemloft.net> | 2006-06-22 00:00:00 -0700 |
---|---|---|
committer | David S. Miller <davem@sunset.davemloft.net> | 2006-06-23 23:15:19 -0700 |
commit | 486ad10a7e1b2bb61ea9e48c4d9510f50cd74fc5 (patch) | |
tree | eac1b6fe80626767f7bea620b9e06aedc6600b95 /arch | |
parent | 07f8e5f358a0b7240f1dad6b3819f2fd1103f159 (diff) | |
download | op-kernel-dev-486ad10a7e1b2bb61ea9e48c4d9510f50cd74fc5.zip op-kernel-dev-486ad10a7e1b2bb61ea9e48c4d9510f50cd74fc5.tar.gz |
[SPARC64]: Minor bug fix to obp_read_memory().
If we end up zero'ing out the size of one of the entries,
pop it out of the array completely because some code that
examines these things cannot handle a zero length element
properly.
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'arch')
-rw-r--r-- | arch/sparc64/mm/init.c | 21 |
1 files changed, 19 insertions, 2 deletions
diff --git a/arch/sparc64/mm/init.c b/arch/sparc64/mm/init.c index 45a70d6..9d2b0da 100644 --- a/arch/sparc64/mm/init.c +++ b/arch/sparc64/mm/init.c @@ -102,8 +102,6 @@ static void __init read_obp_memory(const char *property, prom_halt(); } - *num_ents = ents; - /* Sanitize what we got from the firmware, by page aligning * everything. */ @@ -125,6 +123,25 @@ static void __init read_obp_memory(const char *property, regs[i].phys_addr = base; regs[i].reg_size = size; } + + for (i = 0; i < ents; i++) { + if (regs[i].reg_size == 0UL) { + int j; + + for (j = i; j < ents - 1; j++) { + regs[j].phys_addr = + regs[j+1].phys_addr; + regs[j].reg_size = + regs[j+1].reg_size; + } + + ents--; + i--; + } + } + + *num_ents = ents; + sort(regs, ents, sizeof(struct linux_prom64_registers), cmp_p64, NULL); } |