[OpenBIOS] [PATCH] RFC: Change ofmem_common.c to set memory translation properties by reference

Andreas Färber andreas.faerber at web.de
Sat Oct 16 17:18:56 CEST 2010


Mark,

Am 14.10.2010 um 22:32 schrieb Mark Cave-Ayland:

> The attached patch changes OFMEM so that instead of allocating new  
> space within the Forth dictionary every time the /memory and / 
> virtual-memory available/translations nodes are updated, we simply  
> change the property to point directly to a static buffer. This has  
> the effect of saving substantial amounts of memory during  
> OpenSolaris 10 boot (in fact the final dictionary size after boot is  
> now < 256K once again).
>
> Blue/Andreas: please could you take a look at this patch and make  
> sure it doesn't break anything in your SPARC64/PPC tests?

It looks as if none of us actually tested or ack'ed it yet, all  
answers were about sparc progress and devices only...
Me for one couldn't save your inline patch and my mailer is known to  
cause whitespace damage, and git-am wasn't able to handle the mbox  
file since your SVN diff is missing the a/ directory level expected by  
Git.

If you're asking for comments from us, please be more patient. I'm  
still playing with Blue's slightly older libgcc patch, and some  
trivial ones of mine (e.g., CONFIG_RTAS or wrong return) are awaiting  
review/committing longer than yours. Please at least let us know in  
advance how long you're gonna wait, so that we have a chance to send  
in incomplete review comments even if still untested.

I like the general optimization idea, but I found one typo s/fix/fit/  
and I was wondering whether the property setting function really needs  
to live inside ofmem_common.c or whether we might want to make  
set_property_nocopy() out of it for general use.
I also had some style questions related to the inconsistent use of  
spacing inside braces. Some ppc code even has a wild mix of tabs and 8  
spaces, which is unfortunate when many other projects including QEMU  
use an indentation of 4. We're lacking a CODING_STYLE document imo, to  
look up which way it's supposed to be.

BR,
Andreas

P.S. I have local patches to prepare ofmem_release() inside  
ofmem_common.c.



More information about the OpenBIOS mailing list