[coreboot] [v3] r1129 makes bootblock too big

Ronald Hoogenboom ronald at zonnet.nl
Thu Feb 12 23:23:52 CET 2009


On Thu, 2009-02-12 at 23:16 +0100, Ronald Hoogenboom wrote:
> On Thu, 2009-02-12 at 14:44 -0700, Myles Watson wrote:
> > On Thu, Feb 12, 2009 at 2:29 PM, ron minnich <rminnich at gmail.com> wrote:
> > > On Thu, Feb 12, 2009 at 1:28 PM, Myles Watson <mylesgw at gmail.com> wrote:
> > >> On Thu, Feb 12, 2009 at 2:19 PM, Ronald Hoogenboom <ronald at zonnet.nl> wrote:
> > >>> Hi,
> > >>>
> > >>> After updating to r1129, adds call to allow_all_aps_stop to initram.c,
> > >>> bootblock is too big (16 bytes) and build fails. (gigabyte m57sli)
> > >>
> > >> Did we ever decide on how big was too big for the bootblock?  I
> > >> thought it was clear that we needed it to be larger for k8.  Should we
> > >> make it a parameter?  I realize it's hardcoded in lar right now.
> > >>
> > >
> > > it's really hardcoded in a few places.
> > 
> > So he should just take out a few printk statements to make it fit again?
> > 
> > Thanks,
> > Myles
> Sorry, its not the addition to initram.c, but the extra stuff in
> arch/x86/amd/stage0.S that makes it too big. No printk in there...
> 
> Ronald.
r1128:
build/arch/x86/amd/stage0.o:     file format elf32-i386

Sections:
Idx Name          Size      VMA       LMA       File off  Algn
  0 .text         000001e8  00000000  00000000  00000034  2**2
                  CONTENTS, ALLOC, LOAD, RELOC, READONLY, CODE

r1129:
build/arch/x86/amd/stage0.o:     file format elf32-i386

Sections:
Idx Name          Size      VMA       LMA       File off  Algn
  0 .text         000001fc  00000000  00000000  00000034  2**2
                  CONTENTS, ALLOC, LOAD, RELOC, READONLY, CODE

---- There was not much room left ----

Ronald.





More information about the coreboot mailing list