Prev: lmb: Add lmb_find_area()
Next: lmb: Add lmb_find_area_node()
From: Benjamin Herrenschmidt on 13 Apr 2010 00:30
On Thu, 2010-04-08 at 23:03 -0700, Yinghai Lu wrote:
> Generic version __lmb_find_area() is going from high to low, and for 32bit
> active_region for 32bit does include high pages
>
> need to replace the limit with lmb.default_alloc_limit, aka get_max_mapped()
>
> with this patch, x86 32bit could use generic version of __lmb_find_area()

So you unconditionally add access to some lmb specific data structure to
generic code ? That isn't going to work very well on archs that don't
use lmb.

Also, those things should be local to lmb_* anyways.

Cheers,
Ben.

> Signed-off-by: Yinghai Lu <yinghai(a)kernel.org>
> ---
> mm/page_alloc.c | 3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/mm/page_alloc.c b/mm/page_alloc.c
> index 79bd44b..256aed0 100644
> --- a/mm/page_alloc.c
> +++ b/mm/page_alloc.c
> @@ -3445,6 +3445,9 @@ void * __init __alloc_memory_core_early(int nid, u64 size, u64 align,
>
> u64 addr;
>
> + if (limit > lmb.default_alloc_limit)
> + limit = lmb.default_alloc_limit;
> +
> addr = find_memory_core_early(nid, size, align, goal, limit);
>
> if (addr == -1ULL)


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
 | 
Pages: 1
Prev: lmb: Add lmb_find_area()
Next: lmb: Add lmb_find_area_node()