Skip to content

Commit

Permalink
[PATCH] sparsemem base: teach discontig about sparse ranges
Browse files Browse the repository at this point in the history
discontig.c has some assumptions that mem_map[]s inside of a node are
contiguous.  Teach it to make sure that each region that it's bringing online
is actually made up of valid ranges of ram.

Written-by: Andy Whitcroft <apw@shadowen.org>
Signed-off-by: Dave Hansen <haveblue@us.ibm.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
  • Loading branch information
Dave Hansen authored and Linus Torvalds committed Jun 23, 2005
1 parent 348f8b6 commit 5b505b9
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 1 deletion.
14 changes: 14 additions & 0 deletions arch/i386/mm/discontig.c
Original file line number Diff line number Diff line change
Expand Up @@ -216,6 +216,7 @@ static unsigned long calculate_numa_remap_pages(void)
{
int nid;
unsigned long size, reserve_pages = 0;
unsigned long pfn;

for_each_online_node(nid) {
/*
Expand All @@ -234,6 +235,19 @@ static unsigned long calculate_numa_remap_pages(void)
size = (size + LARGE_PAGE_BYTES - 1) / LARGE_PAGE_BYTES;
/* now the roundup is correct, convert to PAGE_SIZE pages */
size = size * PTRS_PER_PTE;

/*
* Validate the region we are allocating only contains valid
* pages.
*/
for (pfn = node_end_pfn[nid] - size;
pfn < node_end_pfn[nid]; pfn++)
if (!page_is_ram(pfn))
break;

if (pfn != node_end_pfn[nid])
size = 0;

printk("Reserving %ld pages of KVA for lmem_map of node %d\n",
size, nid);
node_remap_size[nid] = size;
Expand Down
2 changes: 1 addition & 1 deletion arch/i386/mm/init.c
Original file line number Diff line number Diff line change
Expand Up @@ -191,7 +191,7 @@ static inline int page_kills_ppro(unsigned long pagenr)

extern int is_available_memory(efi_memory_desc_t *);

static inline int page_is_ram(unsigned long pagenr)
int page_is_ram(unsigned long pagenr)
{
int i;
unsigned long addr, end;
Expand Down
2 changes: 2 additions & 0 deletions include/asm-i386/page.h
Original file line number Diff line number Diff line change
Expand Up @@ -120,6 +120,8 @@ static __inline__ int get_order(unsigned long size)

extern int sysctl_legacy_va_layout;

extern int page_is_ram(unsigned long pagenr);

#endif /* __ASSEMBLY__ */

#ifdef __ASSEMBLY__
Expand Down

0 comments on commit 5b505b9

Please sign in to comment.