Show
Ignore:
Timestamp:
11/28/08 13:09:17 (6 weeks ago)
Author:
oxygene
Message:

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@…>

Please commit while this is being worked out.
Acked-by: Peter Stuge <peter@…>

Files:
1 modified

Legend:

Unmodified
Added
Removed
  • trunk/coreboot-v2/src/mainboard/amd/db800/Config.lb

    r3329 r3778  
    5555        makerule ./cache_as_ram_auto.inc 
    5656                        depends "$(MAINBOARD)/cache_as_ram_auto.c option_table.h" 
    57                         action "$(CC) $(DISTRO_CFLAGS) -I$(TOP)/src -I. $(CPPFLAGS) $(MAINBOARD)/cache_as_ram_auto.c -Os -nostdinc -nostdlib -fno-builtin -Wall -c -S -o $@" 
    58                         action "perl -e 's/.rodata/.rom.data/g' -pi $@" 
    59                         action "perl -e 's/.text/.section .rom.text/g' -pi $@" 
     57                        action "$(CC) $(DISTRO_CFLAGS) -I$(TOP)/src -I. $(CPPFLAGS) $(MAINBOARD)/cache_as_ram_auto.c -Os -nostdinc -nostdlib -fno-builtin -Wall -g -dA -fverbose-asm -c -S -o $@" 
     58                        action "perl -e 's/\.rodata/.rom.data/g' -pi $@" 
     59                        action "perl -e 's/\.text/.section .rom.text/g' -pi $@" 
    6060        end 
    6161end