From mboxrd@z Thu Jan  1 00:00:00 1970
From: "Martin J. Bligh" <mbligh@aracnet.com>
Date: Wed, 18 Jun 2003 16:54:40 +0000
Subject: Re: [Discontig-devel] [RFC/PATCH] discontig update for linux-ia64-2.5 bk tree
Message-Id: <marc-linux-ia64-105595612419340@msgid-missing>
List-Id: <linux-ia64.vger.kernel.org>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: linux-ia64@vger.kernel.org


You seem to have missed the comments here:

> -#ifndef CONFIG_DISCONTIGMEM          /* Don't use mapnrs, do it properly */
> extern unsigned long max_mapnr;
> -#endif
> ...
> -#ifndef CONFIG_DISCONTIGMEM
> /* The array of struct pages - for discontigmem use pgdat->lmem_map */
>  extern struct page *mem_map;
> -#endif 

Using global mapnrs for discontigmem is a horrible kludge, and very 
confusing. There was a damned good reason I removed those ...

It should be easy to just use local offsets into the node, right?

> diff -Nru a/mm/page_alloc.c b/mm/page_alloc.c
> --- a/mm/page_alloc.c	Tue Jun 17 14:45:16 2003
> +++ b/mm/page_alloc.c	Tue Jun 17 14:45:16 2003
> @@ -57,12 +57,15 @@
>   */
>  static int bad_range(struct zone *zone, struct page *page)
>  {
> -	if (page_to_pfn(page) >= zone->zone_start_pfn + zone->spanned_pages)
> +	if (page_to_pfn(page) >= zone->zone_start_pfn + zone->spanned_pages) {
>  		return 1;
> -	if (page_to_pfn(page) < zone->zone_start_pfn)
> +	}
> +	if (page_to_pfn(page) < zone->zone_start_pfn) {
>  		return 1;
> -	if (zone != page_zone(page))
> +	}
> +	if (zone != page_zone(page)) {
>  		return 1;
> +	}
>  	return 0;
>  }

Why? this just looks like removing standard Linux style? 
Is there a real change hidden in there?

M.