[coreboot] [PATCH] add debug information to v2 builds
c-d.hailfinger.devel.2006 at gmx.net
Thu Nov 27 20:38:16 CET 2008
On 27.11.2008 20:24, 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
>> 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
> 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.
That's also one of the reasons why I didn't ack the complete patch. I do
coreboot development on my work machine which happens to be a laptop
with not enough hard disk space. I would have to delete some stuff to
run the new abuild.
AFAICS abuild images are not fully bootable anyway because they have no
payload. I may be wrong, though. However, if I'm right, the debug
symbols in abuild don't help anyone. Debug symbols in "real" builds
would make sense.
More information about the coreboot