[U-Boot] [PATCH v2 01/13] sandbox: Improve debugging in initcall_run_list()

sjg at google.com sjg at google.com
Mon Apr 22 02:38:45 UTC 2019


At present if one of the initcalls fails on sandbox the address printing
is not help, e.g.:

  initcall sequence 0000557678967c80 failed at call 00005576709dfe1f (err=-96)

This is because U-Boot gets relocated high into memory and the relocation
offset (gd->reloc_off) does not work correctly for sandbox.

Add support for finding the base address of the text region (at least on
Linux) and use that to set the relocation offset. This makes the output
better:

  initcall sequence 0000560775957c80 failed at call 0000000000048134 (err=-96)

Then you use can use grep to see which init call failed, e.g.:

   $ grep 0000000000048134 u-boot.map
   stdio_add_devices

Of course another option is to run it with a debugger such as gdb:

   $ gdb u-boot
   ...
   (gdb) br initcall.h:41
   Breakpoint 1 at 0x4db9d: initcall.h:41. (2 locations)

Note that two locations are reported, since this function is used in both
board_init_f() and board_init_r().

   (gdb) r
   Starting program: /tmp/b/sandbox/u-boot
   [Thread debugging using libthread_db enabled]
   Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".

   U-Boot 2018.09-00264-ge0c2ba9814-dirty (Sep 22 2018 - 12:21:46 -0600)

   DRAM:  128 MiB
   MMC:

Breakpoint 1, initcall_run_list (init_sequence=0x5555559619e0 <init_sequence_f>)
    at /scratch/sglass/cosarm/src/third_party/u-boot/files/include/initcall.h:41
41				printf("initcall sequence %p failed at call %p (err=%d)\n",
   (gdb) print *init_fnc_ptr
   $1 = (const init_fnc_t) 0x55555559c114 <stdio_add_devices>
   (gdb)

Signed-off-by: Simon Glass <sjg at chromium.org>
---

Changes in v2:
- Adjust code to allow it to work before relocation also
- Update commit message to take account of initcall.h being in a header
- Add documentation to README.sandbox

 arch/sandbox/cpu/os.c                  | 37 ++++++++++++++++++++++
 arch/sandbox/cpu/start.c               | 12 +++++--
 arch/sandbox/include/asm/global_data.h |  1 +
 board/sandbox/README.sandbox           | 43 ++++++++++++++++++++++++++
 common/board_f.c                       |  2 +-
 include/initcall.h                     |  8 +++--
 include/os.h                           | 11 +++++++
 7 files changed, 109 insertions(+), 5 deletions(-)

Applied to u-boot-dm, thanks!


More information about the U-Boot mailing list