[coreboot] buildrom: cleanup package includes (updated)

[Jordan Crouse]

On 05/05/08 19:39 -0600, Myles Watson wrote:
> 
> 
> > -----Original Message-----
> > From: Jordan Crouse [mailto:jordan.crouse at amd.com]
> > Sent: Monday, May 05, 2008 5:58 PM
> > To: Myles Watson
> > Cc: coreboot at coreboot.org
> > Subject: Re: buildrom: cleanup package includes (updated)
> > 
> > On 05/05/08 17:29 -0600, Jordan Crouse wrote:
> > > On 05/05/08 17:02 -0600, Myles Watson wrote:
> > > >
> > > >
> > > > > -----Original Message-----
> > > > > From: coreboot-bounces at coreboot.org [mailto:coreboot-
> > bounces at coreboot.org]
> > > > > On Behalf Of Jordan Crouse
> > > > > Sent: Monday, May 05, 2008 3:55 PM
> > > > > To: coreboot at coreboot.org
> > > > > Subject: [coreboot] buildrom: cleanup package includes (updated)
> > > > >
> > > > > Attached is an updated version of the patch I sent before that
> > > > > streamined the .mk files that are included in buildrom.  This
> > version
> > > > > is updated with the latest and greatest from Myles and everybody.
> > > > >
> > > > > Tested with all payloads + coreinfo, and hd8mr + kernel (I'm going
> > to
> > > > > test the other platforms with kernel, but I didn't want to wait
> > around
> > > > > for them to finish building).
> > > >
> > > > Acked-by: Myles Watson <mylesgw at gmail.com>
> > >
> > > Thanks - I am going to wait on this until tomorrow.  Coreboot-v2 +
> > kernel
> > > failed on 13 of 17 targets (and only 3 failures are due to missing .mk
> > files).
> > > I don't know if this is becuase of my patch, my test harness, or
> > something
> > > else.  Either way, it is best that I hold off until I can debug it more
> > > fully.
> > 
> > I had a second to build one of the failing platforms at random (s2882),
> > and
> > the error ended up being that the kernel was too big for the ROM.  I'm
> > going
> > to go ahead and re-run the entire test with verboseness turned on over
> > night to get a clear picture of which platforms need love.
> > 
> 
> Several of the platforms have kernels that have never fit for me.  I always
> figured the problem was my broken toolchain.  I only test that changes don't
> break things that were working.

Its a good thing I waited - I had a misspelling in coreboot-v2.mk 
(PLAFORM_GEODE).  I fixed that, and the Geode platforms still failed,
but at least this time they failed because of size.

So, for the record, the following platforms have kernel configs that are
too big (or don't have an accomodating Config.lb):

dbe61
db800
ga-m57sli-s4 
serengeti-cheetah
norwich
msm800sev
alix1c
s2892
s2881
s2882

The following platforms have no kernel.mk:

a8v-e-se 
s-c-fam10
ga-2761gxdk

And the following platforms build a kernel successfully:

h8dmr
qemu
s2891
s2895

All platforms successfully built FILO and coreinfo.  I haven't yet added
any of the other payloads to the test.

Jordan

-- 
Jordan Crouse
Systems Software Development Engineer 
Advanced Micro Devices, Inc.