target/mips: Honour -semihosting-config userspace=on

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.

MIPS semihosting is not implemented for linux-user builds.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20220822141230.3658237-5-peter.maydell@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
This commit is contained in:
Peter Maydell 2022-08-22 15:12:27 +01:00 committed by Richard Henderson
parent a52417e1ee
commit b35d74015b
4 changed files with 11 additions and 10 deletions

View File

@ -825,7 +825,7 @@ static void gen_pool16c_insn(DisasContext *ctx)
generate_exception_break(ctx, extract32(ctx->opcode, 0, 4)); generate_exception_break(ctx, extract32(ctx->opcode, 0, 4));
break; break;
case SDBBP16: case SDBBP16:
if (is_uhi(extract32(ctx->opcode, 0, 4))) { if (is_uhi(ctx, extract32(ctx->opcode, 0, 4))) {
ctx->base.is_jmp = DISAS_SEMIHOST; ctx->base.is_jmp = DISAS_SEMIHOST;
} else { } else {
/* /*
@ -941,7 +941,7 @@ static void gen_pool16c_r6_insn(DisasContext *ctx)
break; break;
case R6_SDBBP16: case R6_SDBBP16:
/* SDBBP16 */ /* SDBBP16 */
if (is_uhi(extract32(ctx->opcode, 6, 4))) { if (is_uhi(ctx, extract32(ctx->opcode, 6, 4))) {
ctx->base.is_jmp = DISAS_SEMIHOST; ctx->base.is_jmp = DISAS_SEMIHOST;
} else { } else {
if (ctx->hflags & MIPS_HFLAG_SBRI) { if (ctx->hflags & MIPS_HFLAG_SBRI) {
@ -1310,7 +1310,7 @@ static void gen_pool32axf(CPUMIPSState *env, DisasContext *ctx, int rt, int rs)
generate_exception_end(ctx, EXCP_SYSCALL); generate_exception_end(ctx, EXCP_SYSCALL);
break; break;
case SDBBP: case SDBBP:
if (is_uhi(extract32(ctx->opcode, 16, 10))) { if (is_uhi(ctx, extract32(ctx->opcode, 16, 10))) {
ctx->base.is_jmp = DISAS_SEMIHOST; ctx->base.is_jmp = DISAS_SEMIHOST;
} else { } else {
check_insn(ctx, ISA_MIPS_R1); check_insn(ctx, ISA_MIPS_R1);

View File

@ -951,7 +951,7 @@ static int decode_ase_mips16e(CPUMIPSState *env, DisasContext *ctx)
} }
break; break;
case RR_SDBBP: case RR_SDBBP:
if (is_uhi(extract32(ctx->opcode, 5, 6))) { if (is_uhi(ctx, extract32(ctx->opcode, 5, 6))) {
ctx->base.is_jmp = DISAS_SEMIHOST; ctx->base.is_jmp = DISAS_SEMIHOST;
} else { } else {
/* /*

View File

@ -3694,7 +3694,7 @@ static int decode_nanomips_32_48_opc(CPUMIPSState *env, DisasContext *ctx)
generate_exception_end(ctx, EXCP_BREAK); generate_exception_end(ctx, EXCP_BREAK);
break; break;
case NM_SDBBP: case NM_SDBBP:
if (is_uhi(extract32(ctx->opcode, 0, 19))) { if (is_uhi(ctx, extract32(ctx->opcode, 0, 19))) {
ctx->base.is_jmp = DISAS_SEMIHOST; ctx->base.is_jmp = DISAS_SEMIHOST;
} else { } else {
if (ctx->hflags & MIPS_HFLAG_SBRI) { if (ctx->hflags & MIPS_HFLAG_SBRI) {
@ -4633,7 +4633,7 @@ static int decode_isa_nanomips(CPUMIPSState *env, DisasContext *ctx)
generate_exception_end(ctx, EXCP_BREAK); generate_exception_end(ctx, EXCP_BREAK);
break; break;
case NM_SDBBP16: case NM_SDBBP16:
if (is_uhi(extract32(ctx->opcode, 0, 3))) { if (is_uhi(ctx, extract32(ctx->opcode, 0, 3))) {
ctx->base.is_jmp = DISAS_SEMIHOST; ctx->base.is_jmp = DISAS_SEMIHOST;
} else { } else {
if (ctx->hflags & MIPS_HFLAG_SBRI) { if (ctx->hflags & MIPS_HFLAG_SBRI) {

View File

@ -12082,12 +12082,13 @@ static void gen_cache_operation(DisasContext *ctx, uint32_t op, int base,
tcg_temp_free_i32(t0); tcg_temp_free_i32(t0);
} }
static inline bool is_uhi(int sdbbp_code) static inline bool is_uhi(DisasContext *ctx, int sdbbp_code)
{ {
#ifdef CONFIG_USER_ONLY #ifdef CONFIG_USER_ONLY
return false; return false;
#else #else
return semihosting_enabled() && sdbbp_code == 1; bool is_user = (ctx->hflags & MIPS_HFLAG_KSU) == MIPS_HFLAG_UM;
return semihosting_enabled(is_user) && sdbbp_code == 1;
#endif #endif
} }
@ -13898,7 +13899,7 @@ static void decode_opc_special_r6(CPUMIPSState *env, DisasContext *ctx)
} }
break; break;
case R6_OPC_SDBBP: case R6_OPC_SDBBP:
if (is_uhi(extract32(ctx->opcode, 6, 20))) { if (is_uhi(ctx, extract32(ctx->opcode, 6, 20))) {
ctx->base.is_jmp = DISAS_SEMIHOST; ctx->base.is_jmp = DISAS_SEMIHOST;
} else { } else {
if (ctx->hflags & MIPS_HFLAG_SBRI) { if (ctx->hflags & MIPS_HFLAG_SBRI) {
@ -14310,7 +14311,7 @@ static void decode_opc_special2_legacy(CPUMIPSState *env, DisasContext *ctx)
gen_cl(ctx, op1, rd, rs); gen_cl(ctx, op1, rd, rs);
break; break;
case OPC_SDBBP: case OPC_SDBBP:
if (is_uhi(extract32(ctx->opcode, 6, 20))) { if (is_uhi(ctx, extract32(ctx->opcode, 6, 20))) {
ctx->base.is_jmp = DISAS_SEMIHOST; ctx->base.is_jmp = DISAS_SEMIHOST;
} else { } else {
/* /*