[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 09/11] target/nios2: Honour -semihosting-config userspace=on
From: |
Richard Henderson |
Subject: |
[PULL 09/11] target/nios2: Honour -semihosting-config userspace=on |
Date: |
Wed, 14 Sep 2022 14:23:06 +0100 |
From: Peter Maydell <peter.maydell@linaro.org>
Honour the commandline -semihosting-config userspace=on option,
instead of always permitting userspace semihosting calls in system
emulation mode, by passing the correct value to the is_userspace
argument of semihosting_enabled().
Note that this is a behaviour change: if the user wants to
do semihosting calls from userspace they must now specifically
enable them on the command line.
nios2 semihosting is not implemented for linux-user builds.
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20220822141230.3658237-6-peter.maydell@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
---
target/nios2/translate.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/target/nios2/translate.c b/target/nios2/translate.c
index ff631a42f6..8dc0a32c6c 100644
--- a/target/nios2/translate.c
+++ b/target/nios2/translate.c
@@ -817,8 +817,9 @@ static void gen_break(DisasContext *dc, uint32_t code,
uint32_t flags)
{
#ifndef CONFIG_USER_ONLY
/* The semihosting instruction is "break 1". */
+ bool is_user = FIELD_EX32(dc->tb_flags, TBFLAGS, U);
R_TYPE(instr, code);
- if (semihosting_enabled(false) && instr.imm5 == 1) {
+ if (semihosting_enabled(is_user) && instr.imm5 == 1) {
t_gen_helper_raise_exception(dc, EXCP_SEMIHOST);
return;
}
--
2.34.1
- [PULL 00/11] semihosting patch queue, Richard Henderson, 2022/09/14
- [PULL 01/11] target/nios2: Use semihosting/syscalls.h, Richard Henderson, 2022/09/14
- [PULL 02/11] target/nios2: Convert semihosting errno to gdb remote errno, Richard Henderson, 2022/09/14
- [PULL 03/11] target/m68k: Use semihosting/syscalls.h, Richard Henderson, 2022/09/14
- [PULL 05/11] semihosting: Allow optional use of semihosting from userspace, Richard Henderson, 2022/09/14
- [PULL 07/11] target/m68k: Honour -semihosting-config userspace=on, Richard Henderson, 2022/09/14
- [PULL 08/11] target/mips: Honour -semihosting-config userspace=on, Richard Henderson, 2022/09/14
- [PULL 06/11] target/arm: Honour -semihosting-config userspace=on, Richard Henderson, 2022/09/14
- [PULL 09/11] target/nios2: Honour -semihosting-config userspace=on,
Richard Henderson <=
- [PULL 04/11] target/m68k: Convert semihosting errno to gdb remote errno, Richard Henderson, 2022/09/14
- [PULL 11/11] target/riscv: Honour -semihosting-config userspace=on and enable=on, Richard Henderson, 2022/09/14
- [PULL 10/11] target/xtensa: Honour -semihosting-config userspace=on, Richard Henderson, 2022/09/14
- Re: [PULL 00/11] semihosting patch queue, Stefan Hajnoczi, 2022/09/17