Skip to content

Commit

Permalink
memblock: Add arch function to control coalescing of memblock memory …
Browse files Browse the repository at this point in the history
…regions

Some archs such as ARM want to avoid coalescing accross things such
as the lowmem/highmem boundary or similar. This provides the option
to control it via an arch callback for which a weak default is provided
which always allows coalescing.

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
  • Loading branch information
Benjamin Herrenschmidt committed Aug 5, 2010
1 parent 142b45a commit d2cd563
Show file tree
Hide file tree
Showing 2 changed files with 20 additions and 1 deletion.
2 changes: 2 additions & 0 deletions include/linux/memblock.h
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,8 @@ extern void memblock_dump_all(void);

/* Provided by the architecture */
extern phys_addr_t memblock_nid_range(phys_addr_t start, phys_addr_t end, int *nid);
extern int memblock_memory_can_coalesce(phys_addr_t addr1, phys_addr_t size1,
phys_addr_t addr2, phys_addr_t size2);

/**
* memblock_set_current_limit - Set the current allocation limit to allow
Expand Down
19 changes: 18 additions & 1 deletion mm/memblock.c
Original file line number Diff line number Diff line change
Expand Up @@ -241,6 +241,12 @@ static int memblock_double_array(struct memblock_type *type)
return 0;
}

extern int __weak memblock_memory_can_coalesce(phys_addr_t addr1, phys_addr_t size1,
phys_addr_t addr2, phys_addr_t size2)
{
return 1;
}

static long memblock_add_region(struct memblock_type *type, phys_addr_t base, phys_addr_t size)
{
unsigned long coalesced = 0;
Expand All @@ -262,6 +268,10 @@ static long memblock_add_region(struct memblock_type *type, phys_addr_t base, ph
return 0;

adjacent = memblock_addrs_adjacent(base, size, rgnbase, rgnsize);
/* Check if arch allows coalescing */
if (adjacent != 0 && type == &memblock.memory &&
!memblock_memory_can_coalesce(base, size, rgnbase, rgnsize))
break;
if (adjacent > 0) {
type->regions[i].base -= size;
type->regions[i].size += size;
Expand All @@ -274,7 +284,14 @@ static long memblock_add_region(struct memblock_type *type, phys_addr_t base, ph
}
}

if ((i < type->cnt - 1) && memblock_regions_adjacent(type, i, i+1)) {
/* If we plugged a hole, we may want to also coalesce with the
* next region
*/
if ((i < type->cnt - 1) && memblock_regions_adjacent(type, i, i+1) &&
((type != &memblock.memory || memblock_memory_can_coalesce(type->regions[i].base,
type->regions[i].size,
type->regions[i+1].base,
type->regions[i+1].size)))) {
memblock_coalesce_regions(type, i, i+1);
coalesced++;
}
Expand Down

0 comments on commit d2cd563

Please sign in to comment.