Commit 27a3f0e9 authored by Nicolas Pitre's avatar Nicolas Pitre Committed by Nicolas Pitre

ARM: sort the meminfo array earlier

The meminfo array has to be sorted before sanity_check_meminfo() in
arch/arm/mm/mmu.c is called for it to work properly.  This also allows
for a simpler find_limits() in arch/arm/mm/init.c.

The sort is moved to arch/arm/kernel/setup.c because that's where the
meminfo array is populated.  Eventually this should be improved upon
to make the memory bank parser a bit more robust against problems
such as overlapping memory ranges.
Signed-off-by: default avatarNicolas Pitre <nicolas.pitre@linaro.org>
parent 3e28c800
...@@ -31,6 +31,7 @@ ...@@ -31,6 +31,7 @@
#include <linux/memblock.h> #include <linux/memblock.h>
#include <linux/bug.h> #include <linux/bug.h>
#include <linux/compiler.h> #include <linux/compiler.h>
#include <linux/sort.h>
#include <asm/unified.h> #include <asm/unified.h>
#include <asm/cpu.h> #include <asm/cpu.h>
...@@ -888,6 +889,12 @@ static struct machine_desc * __init setup_machine_tags(unsigned int nr) ...@@ -888,6 +889,12 @@ static struct machine_desc * __init setup_machine_tags(unsigned int nr)
return mdesc; return mdesc;
} }
static int __init meminfo_cmp(const void *_a, const void *_b)
{
const struct membank *a = _a, *b = _b;
long cmp = bank_pfn_start(a) - bank_pfn_start(b);
return cmp < 0 ? -1 : cmp > 0 ? 1 : 0;
}
void __init setup_arch(char **cmdline_p) void __init setup_arch(char **cmdline_p)
{ {
...@@ -916,6 +923,7 @@ void __init setup_arch(char **cmdline_p) ...@@ -916,6 +923,7 @@ void __init setup_arch(char **cmdline_p)
parse_early_param(); parse_early_param();
sort(&meminfo.bank, meminfo.nr_banks, sizeof(meminfo.bank[0]), meminfo_cmp, NULL);
sanity_check_meminfo(); sanity_check_meminfo();
arm_memblock_init(&meminfo, mdesc); arm_memblock_init(&meminfo, mdesc);
......
...@@ -20,7 +20,6 @@ ...@@ -20,7 +20,6 @@
#include <linux/highmem.h> #include <linux/highmem.h>
#include <linux/gfp.h> #include <linux/gfp.h>
#include <linux/memblock.h> #include <linux/memblock.h>
#include <linux/sort.h>
#include <asm/mach-types.h> #include <asm/mach-types.h>
#include <asm/prom.h> #include <asm/prom.h>
...@@ -134,30 +133,18 @@ void show_mem(unsigned int filter) ...@@ -134,30 +133,18 @@ void show_mem(unsigned int filter)
} }
static void __init find_limits(unsigned long *min, unsigned long *max_low, static void __init find_limits(unsigned long *min, unsigned long *max_low,
unsigned long *max_high) unsigned long *max_high)
{ {
struct meminfo *mi = &meminfo; struct meminfo *mi = &meminfo;
int i; int i;
*min = -1UL; /* This assumes the meminfo array is properly sorted */
*max_low = *max_high = 0; *min = bank_pfn_start(&mi->bank[0]);
for_each_bank (i, mi)
for_each_bank (i, mi) { if (mi->bank[i].highmem)
struct membank *bank = &mi->bank[i]; break;
unsigned long start, end; *max_low = bank_pfn_end(&mi->bank[i - 1]);
*max_high = bank_pfn_end(&mi->bank[mi->nr_banks - 1]);
start = bank_pfn_start(bank);
end = bank_pfn_end(bank);
if (*min > start)
*min = start;
if (*max_high < end)
*max_high = end;
if (bank->highmem)
continue;
if (*max_low < end)
*max_low = end;
}
} }
static void __init arm_bootmem_init(unsigned long start_pfn, static void __init arm_bootmem_init(unsigned long start_pfn,
...@@ -319,19 +306,10 @@ static void arm_memory_present(void) ...@@ -319,19 +306,10 @@ static void arm_memory_present(void)
} }
#endif #endif
static int __init meminfo_cmp(const void *_a, const void *_b)
{
const struct membank *a = _a, *b = _b;
long cmp = bank_pfn_start(a) - bank_pfn_start(b);
return cmp < 0 ? -1 : cmp > 0 ? 1 : 0;
}
void __init arm_memblock_init(struct meminfo *mi, struct machine_desc *mdesc) void __init arm_memblock_init(struct meminfo *mi, struct machine_desc *mdesc)
{ {
int i; int i;
sort(&meminfo.bank, meminfo.nr_banks, sizeof(meminfo.bank[0]), meminfo_cmp, NULL);
memblock_init(); memblock_init();
for (i = 0; i < mi->nr_banks; i++) for (i = 0; i < mi->nr_banks; i++)
memblock_add(mi->bank[i].start, mi->bank[i].size); memblock_add(mi->bank[i].start, mi->bank[i].size);
......
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