Skip to content

Commit 1232a83

Browse files
agattidpgeorge
authored andcommitted
qemu/mcu/arm: Dump exception cause and registers on machine error.
When a CPU exception is raised when emulating a Thumb-capable processor, the default exception handler would simply enter in an endless loop without providing any further information. This commit adds a more complete exception handler that dumps to STDOUT the exception cause and the status of the registers at the moment of the exception. Signed-off-by: Alessandro Gatti <[email protected]>
1 parent 0950f65 commit 1232a83

File tree

3 files changed

+199
-10
lines changed

3 files changed

+199
-10
lines changed

ports/qemu/Makefile

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@ LDFLAGS += -nostdlib
4444
LIBS = $(shell $(CC) $(CFLAGS) -print-libgcc-file-name)
4545

4646
SRC_C += \
47+
mcu/arm/errorhandler.c \
4748
mcu/arm/startup.c \
4849
shared/runtime/semihosting_arm.c \
4950

ports/qemu/mcu/arm/errorhandler.c

Lines changed: 176 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,176 @@
1+
/*
2+
* This file is part of the MicroPython project, https://micropython.org/
3+
*
4+
* The MIT License (MIT)
5+
*
6+
* Copyright (c) 2024 Alessandro Gatti
7+
*
8+
* Permission is hereby granted, free of charge, to any person obtaining a copy
9+
* of this software and associated documentation files (the "Software"), to deal
10+
* in the Software without restriction, including without limitation the rights
11+
* to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
12+
* copies of the Software, and to permit persons to whom the Software is
13+
* furnished to do so, subject to the following conditions:
14+
*
15+
* The above copyright notice and this permission notice shall be included in
16+
* all copies or substantial portions of the Software.
17+
*
18+
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
19+
* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
20+
* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
21+
* AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
22+
* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
23+
* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
24+
* THE SOFTWARE.
25+
*/
26+
27+
#include <stdint.h>
28+
#include <stdio.h>
29+
#include <stdlib.h>
30+
31+
#include "py/runtime.h"
32+
33+
#if defined(__ARM_ARCH_ISA_THUMB)
34+
35+
typedef enum _exception_kind_t {
36+
RESET = 1,
37+
NMI = 2,
38+
HARD_FAULT = 3,
39+
MEM_MANAGE = 4,
40+
BUS_FAULT = 5,
41+
USAGE_FAULT = 6,
42+
SV_CALL = 11,
43+
DEBUG_MONITOR = 12,
44+
PENDING_SV = 13,
45+
SYSTEM_TICK = 14,
46+
} exception_kind_t;
47+
48+
static const char *EXCEPTION_NAMES_TABLE[] = {
49+
"Reserved",
50+
"Reset",
51+
"NMI",
52+
"HardFault",
53+
"MemManage",
54+
"BusFault",
55+
"UsageFault",
56+
"SVCall",
57+
"DebugMonitor",
58+
"PendSV",
59+
"SysTick",
60+
"External interrupt"
61+
};
62+
63+
// R0-R15, PSR, Kind
64+
uintptr_t registers_copy[18] = { 0 };
65+
66+
__attribute__((naked)) NORETURN void exception_handler(uintptr_t kind) {
67+
// Save registers
68+
__asm volatile (
69+
"ldr r1, =registers_copy \n"
70+
"str r0, [r1, #68] \n" // Kind
71+
"ldr r0, [sp, #0] \n" // R0
72+
"str r0, [r1, #0] \n"
73+
"ldr r0, [sp, #4] \n" // R1
74+
"str r0, [r1, #4] \n"
75+
"ldr r0, [sp, #8] \n" // R2
76+
"str r0, [r1, #8] \n"
77+
"ldr r0, [sp, #12] \n" // R3
78+
"str r0, [r1, #12] \n"
79+
"str r4, [r1, #16] \n"
80+
"str r5, [r1, #20] \n"
81+
"str r6, [r1, #24] \n"
82+
"str r7, [r1, #28] \n"
83+
"mov r0, r8 \n"
84+
"str r0, [r1, #32] \n"
85+
"mov r0, r9 \n"
86+
"str r0, [r1, #36] \n"
87+
"mov r0, r10 \n"
88+
"str r0, [r1, #40] \n"
89+
"mov r0, r11 \n"
90+
"str r0, [r1, #44] \n"
91+
"ldr r0, [sp, #16] \n" // R12
92+
"str r0, [r1, #48] \n"
93+
"mov r0, sp \n"
94+
"sub r0, #32 \n"
95+
"str r0, [r1, #52] \n"
96+
"ldr r0, [sp, #20] \n" // R14
97+
"str r0, [r1, #56] \n"
98+
"ldr r0, [sp, #24] \n" // R15
99+
"str r0, [r1, #60] \n"
100+
"ldr r0, [sp, #28] \n" // xPSR
101+
"str r0, [r1, #64] \n"
102+
:
103+
:
104+
: "memory"
105+
);
106+
107+
uintptr_t saved_kind = registers_copy[17];
108+
109+
switch (saved_kind) {
110+
case RESET:
111+
case NMI:
112+
case HARD_FAULT:
113+
case MEM_MANAGE:
114+
case BUS_FAULT:
115+
case USAGE_FAULT:
116+
case SV_CALL:
117+
case DEBUG_MONITOR:
118+
case PENDING_SV:
119+
case SYSTEM_TICK:
120+
printf(EXCEPTION_NAMES_TABLE[saved_kind]);
121+
break;
122+
default:
123+
if (saved_kind >= 16) {
124+
printf("%s %d", EXCEPTION_NAMES_TABLE[11], saved_kind - 16);
125+
} else {
126+
printf(EXCEPTION_NAMES_TABLE[0]);
127+
}
128+
break;
129+
}
130+
printf(" exception caught:\n");
131+
printf("R0: %08X R1: %08X R2: %08X R3: %08X\n", registers_copy[0], registers_copy[1], registers_copy[2], registers_copy[3]);
132+
printf("R4: %08X R5: %08X R6: %08X R7: %08X\n", registers_copy[4], registers_copy[5], registers_copy[6], registers_copy[7]);
133+
printf("R8: %08X R9: %08X R10: %08X R11: %08X\n", registers_copy[8], registers_copy[9], registers_copy[10], registers_copy[11]);
134+
printf("R12: %08X R13: %08X R14: %08X R15: %08X\n", registers_copy[12], registers_copy[13], registers_copy[14], registers_copy[15]);
135+
printf("xPSR: %08X\n", registers_copy[16]);
136+
137+
for (;;) {}
138+
}
139+
140+
__attribute__((naked)) NORETURN void NMI_Handler(void) {
141+
exception_handler(NMI);
142+
}
143+
144+
__attribute__((naked)) NORETURN void HardFault_Handler(void) {
145+
exception_handler(HARD_FAULT);
146+
}
147+
148+
__attribute__((naked)) NORETURN void MemManage_Handler(void) {
149+
exception_handler(MEM_MANAGE);
150+
}
151+
152+
__attribute__((naked)) NORETURN void BusFault_Handler(void) {
153+
exception_handler(BUS_FAULT);
154+
}
155+
156+
__attribute__((naked)) NORETURN void UsageFault_Handler(void) {
157+
exception_handler(USAGE_FAULT);
158+
}
159+
160+
__attribute__((naked)) NORETURN void SVC_Handler(void) {
161+
exception_handler(SV_CALL);
162+
}
163+
164+
__attribute__((naked)) NORETURN void DebugMon_Handler(void) {
165+
exception_handler(DEBUG_MONITOR);
166+
}
167+
168+
__attribute__((naked)) NORETURN void PendSV_Handler(void) {
169+
exception_handler(PENDING_SV);
170+
}
171+
172+
__attribute__((naked)) NORETURN void SysTick_Handler(void) {
173+
exception_handler(SYSTEM_TICK);
174+
}
175+
176+
#endif

ports/qemu/mcu/arm/startup.c

Lines changed: 22 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,8 @@
2828
#include <stdio.h>
2929
#include <stdlib.h>
3030

31+
#include "py/runtime.h"
32+
3133
#include "shared/runtime/semihosting_arm.h"
3234
#include "uart.h"
3335

@@ -50,7 +52,7 @@ __attribute__((naked)) void Reset_Handler(void) {
5052
_start();
5153
}
5254

53-
void Default_Handler(void) {
55+
NORETURN void Default_Handler(void) {
5456
for (;;) {
5557
}
5658
}
@@ -74,25 +76,35 @@ __attribute__((naked, section(".isr_vector"))) void isr_vector(void) {
7476

7577
#elif defined(__ARM_ARCH_ISA_THUMB)
7678

79+
extern void NMI_Handler(void);
80+
extern void HardFault_Handler(void);
81+
extern void MemManage_Handler(void);
82+
extern void BusFault_Handler(void);
83+
extern void UsageFault_Handler(void);
84+
extern void SVC_Handler(void);
85+
extern void DebugMon_Handler(void);
86+
extern void PendSV_Handler(void);
87+
extern void SysTick_Handler(void);
88+
7789
// ARM architecture with Thumb-only ISA.
7890

7991
const uint32_t isr_vector[] __attribute__((section(".isr_vector"))) = {
8092
(uint32_t)&_estack,
8193
(uint32_t)&Reset_Handler,
82-
(uint32_t)&Default_Handler, // NMI_Handler
83-
(uint32_t)&Default_Handler, // HardFault_Handler
84-
(uint32_t)&Default_Handler, // MemManage_Handler
85-
(uint32_t)&Default_Handler, // BusFault_Handler
86-
(uint32_t)&Default_Handler, // UsageFault_Handler
94+
(uint32_t)&NMI_Handler,
95+
(uint32_t)&HardFault_Handler,
96+
(uint32_t)&MemManage_Handler,
97+
(uint32_t)&BusFault_Handler,
98+
(uint32_t)&UsageFault_Handler,
8799
0,
88100
0,
89101
0,
90102
0,
91-
(uint32_t)&Default_Handler, // SVC_Handler
92-
(uint32_t)&Default_Handler, // DebugMon_Handler
103+
(uint32_t)&SVC_Handler,
104+
(uint32_t)&DebugMon_Handler,
93105
0,
94-
(uint32_t)&Default_Handler, // PendSV_Handler
95-
(uint32_t)&Default_Handler, // SysTick_Handler
106+
(uint32_t)&PendSV_Handler,
107+
(uint32_t)&SysTick_Handler,
96108
};
97109

98110
#endif

0 commit comments

Comments
 (0)