Re: [PATCH 1/4][V2] powerpc : add support for linux, usable-memory properties for drconf memory

[Chandru <chandru@in.ibm.com>]

On Tuesday 22 July 2008 14:46:20 Paul Mackerras wrote:
> Chandru writes:
> 
> > Scan for linux,usable-memory properties in case of dynamic reconfiguration 
> > memory . Support for kexec/kdump.
> > 
> > Signed-off-by: Chandru Siddalingappa <chandru@in.ibm.com>
> 
> Could we *please* have a more comprehensive patch description that
> that?  Something which will help people coming along in two (or five
> or ten) years time to understand what problem exists in the code, how
> this patch solves it, and why this approach was chosen over any
> alternative approaches?
> 
> Thanks,
> Paul.
> 

Another alternate approach could be to create one 'linux,usable-drconf-memory' property and add all the usable memory regions into it, in a similar fashion to ibm,dynamic-memory property. For a given lmb in ibm,dynamic-memory , a corresponding usable-memory entry could be created which will contain 1 or more of (base,size) duple.  For each entry in this new 'linux,usable-drconf-memory' property, a counter within it will tell us how many (base,size) duple are available in it.  Some part of the current code may get duplicated. Let me go back and check if I can work on a patch for this approach .  

thx,
Chandru