target/arm: Log M-profile vector table accesses
commit8cc2246caa6c3eb56b250293aea212cf91c38490
authorPeter Maydell <peter.maydell@linaro.org>
Tue, 15 Mar 2022 20:43:05 +0000 (15 20:43 +0000)
committerPeter Maydell <peter.maydell@linaro.org>
Fri, 18 Mar 2022 11:08:59 +0000 (18 11:08 +0000)
tree8d123889882c1c064dc55a08e17432c3ce68f2ae
parent299b4a3ecf1c7dcc57e55150d4cb234340749854
target/arm: Log M-profile vector table accesses

Currently the CPU_LOG_INT logging misses some useful information
about loads from the vector table.  Add logging where we load vector
table entries.  This is particularly helpful for cases where the user
has accidentally not put a vector table in their image at all, which
can result in confusing guest crashes at startup.

Here's an example of the new logging for a case where
the vector table contains garbage:

Loaded reset SP 0x0 PC 0x0 from vector table
Loaded reset SP 0xd008f8df PC 0xf000bf00 from vector table
Taking exception 3 [Prefetch Abort] on CPU 0
...with CFSR.IACCVIOL
...BusFault with BFSR.STKERR
...taking pending nonsecure exception 3
...loading from element 3 of non-secure vector table at 0xc
...loaded new PC 0x20000558
----------------
IN:
0x20000558:  08000079  stmdaeq  r0, {r0, r3, r4, r5, r6}

(The double reset logging is the result of our long-standing
"CPUs all get reset twice" weirdness; it looks a bit ugly
but it'll go away if we ever fix that :-))

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Message-id: 20220315204306.2797684-2-peter.maydell@linaro.org
target/arm/cpu.c
target/arm/m_helper.c