[PATCH qemu v4 2/2] target/arm/gdbstub: Support reading M security extension registers from GDB

2023-02-07 Thread ~dreiss-meta
From: David Reiss Follows a fairly similar pattern to the existing special register debug support. Only reading is implemented, but it should be possible to implement writes. Signed-off-by: David Reiss --- target/arm/cpu.h | 15 +- target/arm/gdbstub.c | 116

[PATCH qemu v4 0/2] ARM: Add support for V8M special registers in GDB stub

2023-02-07 Thread ~dreiss-meta
New in v4: Moved arm_v7m_mrs_control out of the `#ifdef CONFIG_USER_ONLY` block, unbreaking the user-only build. The downside is that this function is now taking up space in the user-only binary, but it can (presumably?) never be used because there are no user modes for v8m cores. Let me know if

[PATCH qemu v4 1/2] target/arm/gdbstub: Support reading M system registers from GDB

2023-02-07 Thread ~dreiss-meta
From: David Reiss Follows a fairly similar pattern to the existing special register debug support. Only reading is implemented, but it should be possible to implement writes. `v7m_mrs_control` was renamed `arm_v7m_mrs_control` and made non-static so this logic could be shared between the MRS

[PATCH qemu v3 0/2] ARM: Add support for V8M special registers in GDB stub

2023-02-03 Thread ~dreiss-meta
Rebased and updated to use NULL as a sentinel. David Reiss (2): target/arm/gdbstub: Support reading M system registers from GDB target/arm/gdbstub: Support reading M security extension registers from GDB target/arm/cpu.h | 25 +++- target/arm/gdbstub.c | 241

[PATCH qemu v3 1/2] target/arm/gdbstub: Support reading M system registers from GDB

2023-02-03 Thread ~dreiss-meta
From: David Reiss Follows a fairly similar pattern to the existing special register debug support. Only reading is implemented, but it should be possible to implement writes. `v7m_mrs_control` was renamed `arm_v7m_mrs_control` and made non-static so this logic could be shared between the MRS

[PATCH qemu v3 2/2] target/arm/gdbstub: Support reading M security extension registers from GDB

2023-02-03 Thread ~dreiss-meta
From: David Reiss Follows a fairly similar pattern to the existing special register debug support. Only reading is implemented, but it should be possible to implement writes. Signed-off-by: David Reiss --- target/arm/cpu.h | 15 +- target/arm/gdbstub.c | 116

[PATCH qemu v2 1/3] target/arm: Unify checking for M Main Extension in MRS/MSR

2023-01-17 Thread ~dreiss-meta
From: David Reiss BASEPRI, FAULTMASK, and their _NS equivalents only exist on devices with the Main Extension. However, the MRS instruction did not check this, and the MSR instruction handled it inconsistently (warning BASEPRI, but silently ignoring writes to BASEPRI_NS). Unify this behavior

[PATCH qemu v2 2/3] target/arm/gdbstub: Support reading M system registers from GDB

2023-01-17 Thread ~dreiss-meta
From: David Reiss Follows a fairly similar pattern to the existing special register debug support. Only reading is implemented, but it should be possible to implement writes. `v7m_mrs_control` was renamed `arm_v7m_mrs_control` and made non-static so this logic could be shared between the MRS

[PATCH qemu v2 3/3] target/arm/gdbstub: Support reading M security extension registers from GDB

2023-01-17 Thread ~dreiss-meta
From: David Reiss Follows a fairly similar pattern to the existing special register debug support. Only reading is implemented, but it should be possible to implement writes. Signed-off-by: David Reiss --- target/arm/cpu.h | 15 +- target/arm/gdbstub.c | 116

[PATCH qemu v2 0/3] ARM: Add support for V8M special registers in GDB stub

2023-01-17 Thread ~dreiss-meta
Patch 1/3 was already accepted, but it seems is not in master yet. Comments addressed in patches 2 and 3. Let me know if you'd like me to split out a separate commit for renaming arm_v7m_get_sp_ptr. David Reiss (3): target/arm: Unify checking for M Main Extension in MRS/MSR

[PATCH qemu 3/3] target/arm/gdbstub: Support reading M security extension registers from GDB

2023-01-09 Thread ~dreiss-meta
From: David Reiss Follows a fairly similar pattern to the existing special register debug support. Only reading is implemented, but it should be possible to implement writes. Signed-off-by: David Reiss --- target/arm/cpu.h | 4 +- target/arm/gdbstub.c | 149

[PATCH qemu 2/3] target/arm/gdbstub: Support reading M system registers from GDB

2023-01-09 Thread ~dreiss-meta
From: David Reiss Follows a fairly similar pattern to the existing special register debug support. Only reading is implemented, but it should be possible to implement writes. `v7m_mrs_control` was renamed `arm_v7m_mrs_control` and made non-static so this logic could be shared between the MRS

[PATCH qemu 1/3] target/arm: Unify checking for M Main Extension in MRS/MSR

2023-01-09 Thread ~dreiss-meta
From: David Reiss BASEPRI, FAULTMASK, and their _NS equivalents only exist on devices with the Main Extension. However, the MRS instruction did not check this, and the MSR instruction handled it inconsistently (warning BASEPRI, but silently ignoring writes to BASEPRI_NS). Unify this behavior