[v4,0/5] Risc-V/gdb: replace exit calls with proper shutdown

Message

Clément Chigot Oct. 3, 2023, 7:14 a.m. UTC

This series replaces some of the call to exit in hardware used by
Risc-V boards. Otherwise, the gdb connection can be abruptly
disconnected resulting in the last gdb packet "Wxx" being not sent.

For the gdbstub modification, gdb_exit calls ensure that the "Wxx"
packet is sent before exiting. However, some features (see
net/vhost-vdpa.c: vhost_vdpa_cleanup for example) are expecting
that a cleanup is being made before exiting. This, it's probably
safer to follow the same logic here as well.

Difference with v3:
 - Rebase on riscv-to-apply

Clément Chigot (5):
  softmmu: add means to pass an exit code when requesting a shutdown
  softmmu: pass the main loop status to gdb "Wxx" packet
  hw/misc/sifive_test.c: replace exit calls with proper shutdown
  hw/char: riscv_htif: replace exit calls with proper shutdown
  gdbstub: replace exit calls with proper shutdown for softmmu

 gdbstub/gdbstub.c          |  5 +++--
 gdbstub/softmmu.c          |  6 ++++++
 gdbstub/user.c             |  6 ++++++
 hw/char/riscv_htif.c       |  5 ++++-
 hw/misc/sifive_test.c      |  9 +++++++--
 include/gdbstub/syscalls.h |  9 +++++++++
 include/sysemu/runstate.h  |  2 ++
 include/sysemu/sysemu.h    |  2 +-
 softmmu/main.c             |  2 +-
 softmmu/runstate.c         | 16 +++++++++++++---
 10 files changed, 52 insertions(+), 10 deletions(-)

Comments

Alistair Francis Oct. 5, 2023, 1:45 a.m. UTC | #1

On Tue, Oct 3, 2023 at 5:14 PM Clément Chigot <[e-mail]> wrote:
>
> This series replaces some of the call to exit in hardware used by
> Risc-V boards. Otherwise, the gdb connection can be abruptly
> disconnected resulting in the last gdb packet "Wxx" being not sent.
>
> For the gdbstub modification, gdb_exit calls ensure that the "Wxx"
> packet is sent before exiting. However, some features (see
> net/vhost-vdpa.c: vhost_vdpa_cleanup for example) are expecting
> that a cleanup is being made before exiting. This, it's probably
> safer to follow the same logic here as well.
>
> Difference with v3:
>  - Rebase on riscv-to-apply
>
> Clément Chigot (5):
>   softmmu: add means to pass an exit code when requesting a shutdown
>   softmmu: pass the main loop status to gdb "Wxx" packet
>   hw/misc/sifive_test.c: replace exit calls with proper shutdown
>   hw/char: riscv_htif: replace exit calls with proper shutdown
>   gdbstub: replace exit calls with proper shutdown for softmmu

Thanks!

Applied to riscv-to-apply.next

Alistair

>
>  gdbstub/gdbstub.c          |  5 +++--
>  gdbstub/softmmu.c          |  6 ++++++
>  gdbstub/user.c             |  6 ++++++
>  hw/char/riscv_htif.c       |  5 ++++-
>  hw/misc/sifive_test.c      |  9 +++++++--
>  include/gdbstub/syscalls.h |  9 +++++++++
>  include/sysemu/runstate.h  |  2 ++
>  include/sysemu/sysemu.h    |  2 +-
>  softmmu/main.c             |  2 +-
>  softmmu/runstate.c         | 16 +++++++++++++---
>  10 files changed, 52 insertions(+), 10 deletions(-)
>
> --
> 2.25.1
>