Eric W. Biederman
ebiederman at lnxi.com
Thu Nov 27 17:20:01 CET 2003
Stefan Reinauer <stepan at suse.de> writes:
> * YhLu <YhLu at tyan.com> [031125 03:37]:
> > Stefan,
> > I'm porting Tyan S4880 now. I found that the following building err, do you
> > meet it in AMD board?
> No, the AMD boards build fine for me.
> > Even I only enable the CPU0's memctrl in the auto.c
> It seems the struct itself in auto.c is pretty uncritical for size.
> Other parts like the log level need a lot more tuning if you want the
> image to be as verbose as possible.
I think that is more because changing the loglevel is something
we have real control over. I think most of the bloat is in
auto.c but I have not gone back and looked for a while.
Before the memory controller setup was called in a loop it
made a real difference if you setup 1 memory controller or
2 because both calls were inlined. With the loop the memory
controller code is only inlined once no matter how many memory
controllers you have.
> * Do you compile with romcc -O (not -O0 and not -O2)
Hmm. -O2 should work and better than -O. I know it was
broken for a while...
> guess the size issue should be mentioned in the docs. Everybody
> fiddling with LinuxBIOS on AMD64 _has_ seen it..
Yes. And this is why I keep working on making inlining optional
More information about the coreboot