This patchset fixes a couple of bugs reported by Liviu Ionescu related to semihosting syscall handling.
Firstly, if the user sets up the gdbstub but doesn't connect a gdb (ie passes '-s' but not '-S'), and we try to use gdb for semihosting syscalls (either because the user explicitly set target=gdb or else via autodetection) then we currently segfault. This patch adjusts the behaviour so that "gdbstub initialized but no gdb attached to a vCPU" is treated the same way we already treat "gdbstub not initialized". Secondly, we weren't handling commandlines where --semihosting-config appears multiple times: these ought to be merged together but instead we were effectively ignoring all but the last one. thanks -- PMM Peter Maydell (2): gdbstub: Don't use GDB syscalls if no GDB is attached semihosting/config: Merge --semihosting-config option groups gdbstub.c | 14 +++++++++++--- semihosting/config.c | 1 + 2 files changed, 12 insertions(+), 3 deletions(-) -- 2.25.1
