Re: [3/3] sparsemem memory model for ppc64

From: mike kravetz
Date: Thu May 05 2005 - 12:54:43 EST


On Wed, May 04, 2005 at 09:30:57PM +0100, Andy Whitcroft wrote:
> + /*
> + * Note presence of first (logical/coalasced) LMB which will
> + * contain RMO region
> + */
> + start_pfn = lmb.memory.region[0].physbase >> PAGE_SHIFT;
> + end_pfn = start_pfn + (lmb.memory.region[0].size >> PAGE_SHIFT);
> + memory_present(0, start_pfn, end_pfn);

I need to take a close look at this again, but I think this special
handling for the RMO region in unnecessary. I added it in the 'early
days of SPARSE' when there were some 'bootstrap' issues and we needed
to initialize some memory before setting up the bootmem bitmap. I'm
pretty sure all those issues have gone away.

--
Mike
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/