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

[Stefan Reinauer]

Uwe Hermann wrote:
> 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.
>
>   
The v2 build system is not exactly friendly in allowing stuff like this.
What do you suggest?


-- 
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
      Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: info at coresystems.de  • http://www.coresystems.de/
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 249 bytes
Desc: OpenPGP digital signature
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20081127/5dff5641/attachment.sig>