diff options
| author | Douglas Raillard <douglas.raillard@arm.com> | 2016-11-21 14:12:32 +0000 |
|---|---|---|
| committer | Douglas Raillard <douglas.raillard@arm.com> | 2016-11-21 16:59:26 +0000 |
| commit | b91d935feebdf2c3edef19949023645b6cb34b20 (patch) | |
| tree | 796e4d7a42e91df546bd5c926efde40b25fe54f2 | |
| parent | 686019d206949086f0c193c7b21a0cbbc3fa6dfd (diff) | |
Add CFI debug frame information for ASM functions
This allows the debugger to print the callstack when there is an
assembly function in the callstack.
It will work as long as the CFA pointer (frame pointer) location is not
modified (i.e. x29 is not touched in AArch64 state). It is the case in
almost all assembly functions, so this patch improves the average
debugging experience. Call stacks from the debugger should still be
interpreted with care. In more complex functions, one could use .cfi*
directives to inform the debugger about the new location of the CFA
pointer.
Change-Id: I9dabfbc033b45e8528e67f4823c17de7bf02fa24
Signed-off-by: Douglas Raillard <douglas.raillard@arm.com>
| -rw-r--r-- | include/common/asm_macros_common.S | 21 |
1 files changed, 20 insertions, 1 deletions
diff --git a/include/common/asm_macros_common.S b/include/common/asm_macros_common.S index 023124b3..4e9f8bb9 100644 --- a/include/common/asm_macros_common.S +++ b/include/common/asm_macros_common.S @@ -33,12 +33,30 @@ /* * This macro is used to create a function label and place the * code into a separate text section based on the function name - * to enable elimination of unused code during linking + * to enable elimination of unused code during linking. It also adds + * basic debug information to enable call stack printing most of the + * time. */ .macro func _name + /* + * Add Call Frame Information entry in the .debug_frame section for + * debugger consumption. This enables callstack printing in debuggers. + * This does not use any space in the final loaded binary, only in the + * ELF file. + * Note that a function manipulating the CFA pointer location (i.e. the + * x29 frame pointer on AArch64) should declare it using the + * appropriate .cfi* directives, or be prepared to have a degraded + * debugging experience. + */ + .cfi_sections .debug_frame .section .text.\_name, "ax" .type \_name, %function .func \_name + /* + * .cfi_startproc and .cfi_endproc are needed to output entries in + * .debug_frame + */ + .cfi_startproc \_name: .endm @@ -47,6 +65,7 @@ */ .macro endfunc _name .endfunc + .cfi_endproc .size \_name, . - \_name .endm |