[coreboot] v3 stage0

ron minnich rminnich at gmail.com
Mon Oct 6 22:46:11 CEST 2008


On Mon, Oct 6, 2008 at 1:15 PM, Carl-Daniel Hailfinger
<c-d.hailfinger.devel.2006 at gmx.net> wrote:
> Hi,
>
> currently we have 3 different files with stage0 asm code sharing lots of
> delicate code:
> - amd/stage0.S
> - geodelx/stage0.S
> - stage0_i586.S
> and all of them share at least the complete 16bit code which is near
> unreadable in itself.
>
> Can we extract the 16bit code into a common file and include it? I know
> that choice isn't going to be too popular because it makes the code more
> greppy. However, keeping all instances of that code in sync is a major
> headache and we even have remaining bugs in the common copied 32bit
> parts of that code.
> If that's not possible, I'd like to at least enforce a rule which keeps
> the common 16-bit code bit-for-bit identical including comments.
>

one file is fine. The v32 code is a total nightmare, so I would really
like to avoid turning it into that mess.

I almost have a kscope setup ready for v3 and it will make it far less
greppy! It's nice.

ron




More information about the coreboot mailing list