[PATCH v4 00/53] semihosting cleanup

Richard Henderson posted 53 patches 1 year, 10 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20220607204557.658541-1-richard.henderson@linaro.org
Maintainers: Richard Henderson <richard.henderson@linaro.org>, Paolo Bonzini <pbonzini@redhat.com>, Laurent Vivier <laurent@vivier.eu>, "Alex Bennée" <alex.bennee@linaro.org>, "Philippe Mathieu-Daudé" <f4bug@amsat.org>, Peter Maydell <peter.maydell@linaro.org>, Aurelien Jarno <aurelien@aurel32.net>, Jiaxun Yang <jiaxun.yang@flygoat.com>, Aleksandar Rikalo <aleksandar.rikalo@syrmia.com>, Chris Wulff <crwulff@gmail.com>, Marek Vasut <marex@denx.de>, Palmer Dabbelt <palmer@dabbelt.com>, Alistair Francis <alistair.francis@wdc.com>, Bin Meng <bin.meng@windriver.com>
configs/targets/aarch64-linux-user.mak        |    1 +
configs/targets/aarch64_be-linux-user.mak     |    1 +
configs/targets/arm-linux-user.mak            |    1 +
configs/targets/armeb-linux-user.mak          |    1 +
configs/targets/riscv32-linux-user.mak        |    1 +
configs/targets/riscv64-linux-user.mak        |    1 +
include/exec/gdbstub.h                        |   64 +-
include/exec/softmmu-semi.h                   |  101 --
.../semihosting}/common-semi.h                |    2 +-
include/semihosting/console.h                 |   71 +-
include/semihosting/guestfd.h                 |   91 ++
include/semihosting/semihost.h                |   14 +-
include/semihosting/softmmu-uaccess.h         |   59 +
include/semihosting/syscalls.h                |   75 ++
target/arm/common-semi-target.h               |   62 +
target/riscv/common-semi-target.h             |   50 +
accel/stubs/tcg-stub.c                        |    7 +
gdbstub.c                                     |   38 +-
linux-user/aarch64/cpu_loop.c                 |    2 +-
linux-user/arm/cpu_loop.c                     |    2 +-
linux-user/main.c                             |    9 +
linux-user/riscv/cpu_loop.c                   |    2 +-
linux-user/semihost.c                         |   48 +-
semihosting/arm-compat-semi.c                 | 1010 ++++-------------
semihosting/config.c                          |   17 +-
semihosting/console.c                         |  153 +--
semihosting/guestfd.c                         |  160 +++
semihosting/syscalls.c                        |  978 ++++++++++++++++
semihosting/uaccess.c                         |   91 ++
softmmu/vl.c                                  |    3 +-
stubs/semihost.c                              |    6 +-
target/arm/helper.c                           |    4 +-
target/arm/m_helper.c                         |    2 +-
target/m68k/m68k-semi.c                       |   52 +-
target/mips/tcg/sysemu/mips-semi.c            |    2 +-
target/nios2/nios2-semi.c                     |   48 +-
target/riscv/cpu_helper.c                     |    2 +-
semihosting/meson.build                       |    6 +
38 files changed, 2081 insertions(+), 1156 deletions(-)
delete mode 100644 include/exec/softmmu-semi.h
rename {semihosting => include/semihosting}/common-semi.h (96%)
create mode 100644 include/semihosting/guestfd.h
create mode 100644 include/semihosting/softmmu-uaccess.h
create mode 100644 include/semihosting/syscalls.h
create mode 100644 target/arm/common-semi-target.h
create mode 100644 target/riscv/common-semi-target.h
create mode 100644 semihosting/guestfd.c
create mode 100644 semihosting/syscalls.c
create mode 100644 semihosting/uaccess.c
[PATCH v4 00/53] semihosting cleanup
Posted by Richard Henderson 1 year, 10 months ago
Changes for v4:
  * Standarize on host errno in callbacks.  So, from gdbstub, convert
    back to host errors, mapping GDB_EUNKNOWN to EINVAL.
  * Add tcg stub for probe_access_flags, to match existing stubs.
  * Misc tweaks per review.

Richard Henderson (53):
  semihosting: Move exec/softmmu-semi.h to semihosting/softmmu-uaccess.h
  semihosting: Return failure from softmmu-uaccess.h functions
  semihosting: Improve condition for config.c and console.c
  semihosting: Move softmmu-uaccess.h functions out of line
  accel/stubs: Add tcg stub for probe_access_flags
  semihosting: Add target_strlen for softmmu-uaccess.h
  semihosting: Simplify softmmu_lock_user_string
  semihosting: Split out guestfd.c
  semihosting: Inline set_swi_errno into common_semi_cb
  semihosting: Adjust error checking in common_semi_cb
  semihosting: Clean up common_semi_flen_cb
  semihosting: Clean up common_semi_open_cb
  semihosting: Return void from do_common_semihosting
  semihosting: Move common-semi.h to include/semihosting/
  semihosting: Remove GDB_O_BINARY
  include/exec: Move gdb open flags to gdbstub.h
  include/exec: Move gdb_stat and gdb_timeval to gdbstub.h
  include/exec: Define errno values in gdbstub.h
  gdbstub: Convert GDB error numbers to host error numbers
  semihosting: Use struct gdb_stat in common_semi_flen_cb
  semihosting: Split is_64bit_semihosting per target
  semihosting: Split common_semi_flen_buf per target
  semihosting: Split out common_semi_has_synccache
  semihosting: Split out common-semi-target.h
  semihosting: Use env more often in do_common_semihosting
  semihosting: Move GET_ARG/SET_ARG earlier in the file
  semihosting: Split out semihost_sys_open
  semihosting: Split out semihost_sys_close
  semihosting: Split out semihost_sys_read
  semihosting: Split out semihost_sys_write
  semihosting: Bound length for semihost_sys_{read,write}
  semihosting: Split out semihost_sys_lseek
  semihosting: Split out semihost_sys_isatty
  semihosting: Split out semihost_sys_flen
  semihosting: Split out semihost_sys_remove
  semihosting: Split out semihost_sys_rename
  semihosting: Split out semihost_sys_system
  semihosting: Create semihost_sys_{stat,fstat}
  semihosting: Create semihost_sys_gettimeofday
  gdbstub: Adjust gdb_syscall_complete_cb declaration
  semihosting: Fix docs comment for qemu_semihosting_console_inc
  semihosting: Pass CPUState to qemu_semihosting_console_inc
  semihosting: Expand qemu_semihosting_console_inc to read
  semihosting: Cleanup chardev init
  semihosting: Create qemu_semihosting_console_write
  semihosting: Add GuestFDConsole
  semihosting: Create qemu_semihosting_guestfd_init
  semihosting: Use console_in_gf for SYS_READC
  semihosting: Use console_out_gf for SYS_WRITEC
  semihosting: Remove qemu_semihosting_console_outc
  semihosting: Use console_out_gf for SYS_WRITE0
  semihosting: Remove qemu_semihosting_console_outs
  semihosting: Create semihost_sys_poll_one

 configs/targets/aarch64-linux-user.mak        |    1 +
 configs/targets/aarch64_be-linux-user.mak     |    1 +
 configs/targets/arm-linux-user.mak            |    1 +
 configs/targets/armeb-linux-user.mak          |    1 +
 configs/targets/riscv32-linux-user.mak        |    1 +
 configs/targets/riscv64-linux-user.mak        |    1 +
 include/exec/gdbstub.h                        |   64 +-
 include/exec/softmmu-semi.h                   |  101 --
 .../semihosting}/common-semi.h                |    2 +-
 include/semihosting/console.h                 |   71 +-
 include/semihosting/guestfd.h                 |   91 ++
 include/semihosting/semihost.h                |   14 +-
 include/semihosting/softmmu-uaccess.h         |   59 +
 include/semihosting/syscalls.h                |   75 ++
 target/arm/common-semi-target.h               |   62 +
 target/riscv/common-semi-target.h             |   50 +
 accel/stubs/tcg-stub.c                        |    7 +
 gdbstub.c                                     |   38 +-
 linux-user/aarch64/cpu_loop.c                 |    2 +-
 linux-user/arm/cpu_loop.c                     |    2 +-
 linux-user/main.c                             |    9 +
 linux-user/riscv/cpu_loop.c                   |    2 +-
 linux-user/semihost.c                         |   48 +-
 semihosting/arm-compat-semi.c                 | 1010 ++++-------------
 semihosting/config.c                          |   17 +-
 semihosting/console.c                         |  153 +--
 semihosting/guestfd.c                         |  160 +++
 semihosting/syscalls.c                        |  978 ++++++++++++++++
 semihosting/uaccess.c                         |   91 ++
 softmmu/vl.c                                  |    3 +-
 stubs/semihost.c                              |    6 +-
 target/arm/helper.c                           |    4 +-
 target/arm/m_helper.c                         |    2 +-
 target/m68k/m68k-semi.c                       |   52 +-
 target/mips/tcg/sysemu/mips-semi.c            |    2 +-
 target/nios2/nios2-semi.c                     |   48 +-
 target/riscv/cpu_helper.c                     |    2 +-
 semihosting/meson.build                       |    6 +
 38 files changed, 2081 insertions(+), 1156 deletions(-)
 delete mode 100644 include/exec/softmmu-semi.h
 rename {semihosting => include/semihosting}/common-semi.h (96%)
 create mode 100644 include/semihosting/guestfd.h
 create mode 100644 include/semihosting/softmmu-uaccess.h
 create mode 100644 include/semihosting/syscalls.h
 create mode 100644 target/arm/common-semi-target.h
 create mode 100644 target/riscv/common-semi-target.h
 create mode 100644 semihosting/guestfd.c
 create mode 100644 semihosting/syscalls.c
 create mode 100644 semihosting/uaccess.c

-- 
2.34.1