Re: [PATCH] [POWERPC] Clean up some linker and symbol usage

[Michael Ellerman <michael@ellerman.id.au>]

[-- Attachment #1: Type: text/plain, Size: 1081 bytes --]


On Fri, 2008-03-28 at 10:14 +1100, Paul Mackerras wrote:
> Kumar Gala writes:
> 
> > @@ -1124,7 +1124,7 @@ void __init early_init_devtree(void *params)
> >  	parse_early_param();
> > 
> >  	/* Reserve LMB regions used by kernel, initrd, dt, etc... */
> > -	lmb_reserve(PHYSICAL_START, __pa(klimit) - PHYSICAL_START);
> > +	lmb_reserve(__pa(_stext), _end - _stext);
> 
> In the kdump case where the kernel starts at the 32MB point
> (physical), wouldn't we want to reserve the first 32MB of RAM so that
> it can't get allocated and used for random things?  Doesn't that first
> 32MB contain the old kernel that we want to write out?

For kdump there is nMB of memory reserved at 32MB for the 2nd kernel,
and that's all it gets, all of the other memory is the old kernel text
or the old kernel's data.

cheers

-- 
Michael Ellerman
OzLabs, IBM Australia Development Lab

wwweb: http://michael.ellerman.id.au
phone: +61 2 6212 1183 (tie line 70 21183)

We do not inherit the earth from our ancestors,
we borrow it from our children. - S.M.A.R.T Person

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 189 bytes --]