On Tue, 17 May 2005, Andy Whitcroft wrote:
> Andrew, please consider this patch for -mm.
I agree. Forget about my patch and include this one.
> Originally __free_pages_bulk used the relative page number within
> a zone to define its buddies. This meant that to maintain the
> "maximally aligned" requirements (that an allocation of size N will
> be aligned at least to N physically) zones had to also be aligned to
> 1<<MAX_ORDER pages. When __free_pages_bulk was updated to use the
> relative page frame numbers of the free'd pages to pair buddies this
> released the alignment constraint on the 'left' edge of the zone.
> This allows _either_ edge of the zone to contain partial MAX_ORDER
> sized buddies. These simply never will have matching buddies and
> thus will never make it to the 'top' of the pyramid.
>
> The patch below removes a now redundant check ensuring that the
> mem_map was aligned to MAX_ORDER.
>
> Signed-off-by: Andy Whitcroft <[email protected]>
>
> diffstat free_area_init_core-remove-bogus-warning
> ---
> page_alloc.c | 4 ----
> 1 files changed, 4 deletions(-)
>
> diff -X /home/apw/brief/lib/vdiff.excl -rupN reference/mm/page_alloc.c current/mm/page_alloc.c
> --- reference/mm/page_alloc.c
> +++ current/mm/page_alloc.c
> @@ -1942,7 +1942,6 @@ static void __init free_area_init_core(s
> unsigned long *zones_size, unsigned long *zholes_size)
> {
> unsigned long i, j;
> - const unsigned long zone_required_alignment = 1UL << (MAX_ORDER-1);
> int cpu, nid = pgdat->node_id;
> unsigned long zone_start_pfn = pgdat->node_start_pfn;
>
> @@ -2033,9 +2032,6 @@ static void __init free_area_init_core(s
> zone->zone_mem_map = pfn_to_page(zone_start_pfn);
> zone->zone_start_pfn = zone_start_pfn;
>
> - if ((zone_start_pfn) & (zone_required_alignment-1))
> - printk(KERN_CRIT "BUG: wrong zone alignment, it will crash\n");
> -
> memmap_init(size, nid, j, zone_start_pfn);
>
> zonetable_add(zone, nid, j, zone_start_pfn, size);
>
>
>
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
