[Qemu-devel] [PATCH v5 0/5] semihosting cleanups (plus minor tests/tcg tweak)

Alex Bennée posted 5 patches 1 week ago
Test docker-clang@ubuntu passed
Test FreeBSD passed
Test checkpatch passed
Test docker-quick@centos7 passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190911164959.11003-1-alex.bennee@linaro.org
linux-user/arm/cpu_loop.c         |  3 -
linux-user/arm/target_syscall.h   |  3 -
target/arm/helper.c               | 96 +++++++------------------------
target/arm/m_helper.c             | 18 ++----
target/arm/translate.c            | 27 +++++++--
tests/tcg/Makefile.target         |  7 ++-
tests/tcg/aarch64/Makefile.target |  8 ++-
tests/tcg/arm/Makefile.target     | 20 ++++---
tests/tcg/arm/semihosting.c       | 45 +++++++++++++++
9 files changed, 120 insertions(+), 107 deletions(-)
create mode 100644 tests/tcg/arm/semihosting.c

[Qemu-devel] [PATCH v5 0/5] semihosting cleanups (plus minor tests/tcg tweak)

Posted by Alex Bennée 1 week ago
Hi Peter,

I've restored the #ifndef CONFIG_USER_ONLY to ensure A-profile
linux-user still works. I've cleaned up the now unused linux-user code
and added a simple smoke test to make sure we don't break it my
accident. There is a tests/tcg patch at the beginning of the series
which was just fixing some obvious errors in the tests/tcg comments
which I couldn't leave untouched while I was adding the smoke test. I
can put it in my next testing PR but I'm happy enough for you to take
it through your tree if you want.

Alex Bennée (5):
  tests/tcg: clean-up some comments after the de-tangling
  target/arm: handle M-profile semihosting at translate time
  target/arm: handle A-profile semihosting at translate time
  target/arm: remove run time semihosting checks
  target/arm: remove run-time semihosting checks for linux-user

 linux-user/arm/cpu_loop.c         |  3 -
 linux-user/arm/target_syscall.h   |  3 -
 target/arm/helper.c               | 96 +++++++------------------------
 target/arm/m_helper.c             | 18 ++----
 target/arm/translate.c            | 27 +++++++--
 tests/tcg/Makefile.target         |  7 ++-
 tests/tcg/aarch64/Makefile.target |  8 ++-
 tests/tcg/arm/Makefile.target     | 20 ++++---
 tests/tcg/arm/semihosting.c       | 45 +++++++++++++++
 9 files changed, 120 insertions(+), 107 deletions(-)
 create mode 100644 tests/tcg/arm/semihosting.c

-- 
2.20.1