[coreboot] [PATCH] add debug information to v2 builds

[Uwe Hermann]

On Thu, Nov 27, 2008 at 02:40:43PM +0100, Stefan Reinauer wrote:
> This patch from Ralf Grosse Boerger makes debugging more comfortable. 
> With this patch it's possible to 
> 
> - determine the according source code line for each asm statement
>   (objdump -dS)
> - determine the source code file for each asm statement 
>   (objdump -ddl)
> 
> This isn't exactly trivial because cache_as_ram_auto.c gets compiled to
> assembly and converted by a perl script afterwards.
> 
> This patch solves the problem 
> - by extending cache_as_ram_auto.inc with debug information and line numbers
> - by correcting the perl calls (".text" --> "\.text")
> - by creating a disassembly with source code and line numbers. (ctr0.disasm and
>   coreboot.disasm)
> 
> There's one minor downside to the patch: A complete abuild run takes up around
> 1.6G instead of about 700MB now. But I'm sure this is quite reasonable for the
> benefits.

Can we make this optional, please? 700MB is already quite a lot, let alone 1.6GB.

The changes are fine with me, but there should be some option somewhere
to not generate the extra 900MB of debug output if you don't want it.


Thanks, Uwe.
-- 
http://www.hermann-uwe.de  | http://www.holsham-traders.de
http://www.crazy-hacks.org | http://www.unmaintained-free-software.org