On Tue, 7 Feb 2023 at 13:46, Peter Maydell <peter.mayd...@linaro.org> wrote: > > On Sat, 4 Feb 2023 at 00:34, ~dreiss-meta <dreiss-m...@git.sr.ht> wrote: > > > > 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 +++++++++++++++++++++++++++++++++++++++ > > target/arm/m_helper.c | 29 ++--- > > tests/lcitool/libvirt-ci | 2 +- > > You have an accidental submodule change in here. (It probably crept > in on a rebase -- git's terrible submodule handling makes that very > easy to do by accident.) > > I've removed that and applied these changes to target-arm.next; > thanks.
...the linux-user target fails to build, though: /usr/bin/ld: libqemu-arm-linux-user.fa.p/target_arm_gdbstub.c.o: in function `arm_gdb_get_m_secextreg': gdbstub.c:(.text+0x41cd): undefined reference to `arm_v7m_get_sp_ptr' Could you have a look at that, please? thanks -- PMM
