diff options
| author | Julius Werner <jwerner@chromium.org> | 2017-07-27 14:59:34 -0700 |
|---|---|---|
| committer | Julius Werner <jwerner@chromium.org> | 2017-08-21 13:50:54 -0700 |
| commit | 4d91838b8d01a9e99c44fd0ed72f5d81b4595f17 (patch) | |
| tree | 710929f9ed09fc109d0e998f65839c713dc671a2 /common/aarch32/debug.S | |
| parent | 5ae4dab2b86351579d85d5f5a8f85252336381b8 (diff) | |
| download | arm-trusted-firmware-4d91838b8d01a9e99c44fd0ed72f5d81b4595f17.tar.gz | |
Fix x30 reporting for unhandled exceptions
Some error paths that lead to a crash dump will overwrite the value in
the x30 register by calling functions with the no_ret macro, which
resolves to a BL instruction. This is not very useful and not what the
reader would expect, since a crash dump should usually show all
registers in the state they were in when the exception happened. This
patch replaces the offending function calls with a B instruction to
preserve the value in x30.
Change-Id: I2a3636f2943f79bab0cd911f89d070012e697c2a
Signed-off-by: Julius Werner <jwerner@chromium.org>
Diffstat (limited to 'common/aarch32/debug.S')
| -rw-r--r-- | common/aarch32/debug.S | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/common/aarch32/debug.S b/common/aarch32/debug.S index 2e60bd52b..583ee4a52 100644 --- a/common/aarch32/debug.S +++ b/common/aarch32/debug.S @@ -51,7 +51,7 @@ func do_panic 1: mov lr, r6 - no_ret plat_panic_handler + b plat_panic_handler endfunc do_panic /*********************************************************** |