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-parisc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
