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?