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. Signed-off-by: Stefan Reinauer <stepan@coresystems.de> Please commit while this is being worked out. Acked-by: Peter Stuge <peter@stuge.se> git-svn-id: svn://svn.coreboot.org/coreboot/trunk@3778 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
This commit is contained in:
committed by
Patrick Georgi
parent
38bee3c8b6
commit
4ed326be5d
@@ -124,6 +124,7 @@ makerule coreboot
|
||||
depends "crt0.o $(INIT-OBJECTS) $(COREBOOT_APC) $(COREBOOT_RAM_ROM) ldscript.ld"
|
||||
action "$(CC) $(DISTRO_LFLAGS) -nostdlib -nostartfiles -static -o $@ -T ldscript.ld crt0.o $(INIT-OBJECTS)"
|
||||
action "$(CROSS_COMPILE)nm -n coreboot | sort > coreboot.map"
|
||||
action "$(CROSS_COMPILE)objdump -dS coreboot > coreboot.disasm"
|
||||
end
|
||||
|
||||
# the buildrom tool
|
||||
@@ -139,9 +140,10 @@ makerule crt0.s
|
||||
action "$(CPP) $(CPPFLAGS) -I. -I$(TOP)/src $< > $@.new && mv $@.new $@"
|
||||
end
|
||||
|
||||
# generate an assembly listing via -a switch.
|
||||
makerule crt0.o
|
||||
depends "crt0.s"
|
||||
action "@$(CC) -c $(CPU_OPT) -o $@ $<"
|
||||
action "$(CC) -Wa,-acdlns -c $(CPU_OPT) -o $@ $< >crt0.disasm"
|
||||
end
|
||||
|
||||
makerule etags
|
||||
|
Reference in New Issue
Block a user