Re: [PATCH] x86_64: cleanup setup_node_zones called by paging_init v2

2008-01-14 Thread Christoph Lameter
On Sat, 12 Jan 2008, Yinghai Lu wrote: > please check the following patch. Exactly! Reviewed-by: Christoph Lameter <[EMAIL PROTECTED]> -- 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.kern

Re: [PATCH] x86_64: cleanup setup_node_zones called by paging_init v2

2008-01-12 Thread Yinghai Lu
On Wednesday 09 January 2008 11:19:01 am Christoph Lameter wrote: > On Wed, 9 Jan 2008, Yinghai Lu wrote: > Christoph's x86: 64-bit, make sparsemem vmemmap the only memory model is in x86.git mm so we could remove setup_node_zones because it is not needed by SPARSEMEM please check the followi

Re: [PATCH] x86_64: cleanup setup_node_zones called by paging_init v2

2008-01-09 Thread Christoph Lameter
On Wed, 9 Jan 2008, Yinghai Lu wrote: > +#ifdef CONFIG_FLAT_NODE_MEM_MAP > /* Initialize final allocator for a zone */ > -void __init setup_node_zones(int nodeid) > +static void __init setup_node_zones(int nodeid) > { > unsigned long start_pfn, end_pfn, memmapsize, limit; > > @@ -244,14

Re: [PATCH] x86_64: cleanup setup_node_zones called by paging_init v2

2008-01-09 Thread Dave Hansen
On Wed, 2008-01-09 at 10:30 -0800, Yinghai Lu wrote: > > [PATCH] x86_64: cleanup setup_node_zones called by paging_init v2 > > setup_node_zones calcuates some variable but only use them when > FLAT_NODE_MEM_MAP is set > > so change the MACRO postion to avoid calculating. > > also change it to s

Re: [PATCH] x86_64: cleanup setup_node_zones called by paging_init

2008-01-09 Thread Christoph Lameter
On Tue, 8 Jan 2008, Yinghai Lu wrote: > [PATCH] x86_64: cleanup setup_node_zones called by paging_init The whitespace cleanups make it difficult to review the patch. Could you send those separately? -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a mess