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