[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20080506154922.GB5034@csn.ul.ie>
Date: Tue, 6 May 2008 16:49:23 +0100
From: Mel Gorman <mel@....ul.ie>
To: Kyle McMartin <kyle@...artin.ca>
Cc: Adrian Bunk <bunk@...nel.org>,
Christoph Lameter <clameter@....com>,
Lee Schermerhorn <lee.schermerhorn@...com>,
kyle@...isc-linux.org, matthew@....cx, grundler@...isc-linux.org,
linux-parisc@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: parisc DISCONTIGMEM compile breakage
On (06/05/08 11:24), Kyle McMartin didst pronounce:
> On Tue, May 06, 2008 at 04:46:48PM +0300, Adrian Bunk wrote:
> > Commit 54a6eb5c4765aa573a030ceeba2c14e3d2ea5706
> > (mm: use two zonelist that are filtered by GFP mask)
> > causes the following build error with CONFIG_DISCONTIGMEM=y
> > on parisc:
> >
> > <-- snip -->
> >
> > ...
> > CC arch/parisc/mm/init.o
> > /home/bunk/linux/kernel-2.6/git/linux-2.6/arch/parisc/mm/init.c: In function 'show_mem':
> > /home/bunk/linux/kernel-2.6/git/linux-2.6/arch/parisc/mm/init.c:609: error: too few arguments to function 'node_zonelist'
> > make[2]: *** [arch/parisc/mm/init.o] Error 1
> >
>
> yes. the obvious fix works and boots, but I haven't quite determined why
> they decided to add this yet, so no patch committed plz.
>
I'm not sure what you are referring to as "this" here. If you mean the
zonelist changes, they were done to reduce the number of zonelists that
exist, to clear up an anomoly with MPOL_BIND and to remove a hack called
alloc_should_filter_zonelist() in the page allocator. The pa-risc code that
walks zonelists (presumably for debugging) has been around for a while and
was updated at the same time as the zonelist patches were done, just not as
well as it should have been.
--
Mel Gorman
Part-time Phd Student Linux Technology Center
University of Limerick IBM Dublin Software Lab
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists