mirror of
https://github.com/Motorhead1991/qemu.git
synced 2025-08-04 08:13:54 -06:00
target/riscv: Remove the hyp load and store functions
Remove the special Virtulisation load and store functions and just use the standard tcg tcg_gen_qemu_ld_tl() and tcg_gen_qemu_st_tl() functions instead. As part of this change we ensure we still run an access check to make sure we can perform the operations. Signed-off-by: Alistair Francis <alistair.francis@wdc.com> Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Message-id: 189ac3e53ef2854824d18aad7074c6649f17de2c.1604464950.git.alistair.francis@wdc.com
This commit is contained in:
parent
1c1c060aa8
commit
743077b35b
5 changed files with 59 additions and 166 deletions
|
@ -16,26 +16,34 @@
|
|||
* this program. If not, see <http://www.gnu.org/licenses/>.
|
||||
*/
|
||||
|
||||
#ifndef CONFIG_USER_ONLY
|
||||
static void check_access(DisasContext *ctx) {
|
||||
if (!ctx->hlsx) {
|
||||
if (ctx->virt_enabled) {
|
||||
generate_exception(ctx, RISCV_EXCP_VIRT_INSTRUCTION_FAULT);
|
||||
} else {
|
||||
generate_exception(ctx, RISCV_EXCP_ILLEGAL_INST);
|
||||
}
|
||||
}
|
||||
}
|
||||
#endif
|
||||
|
||||
static bool trans_hlv_b(DisasContext *ctx, arg_hlv_b *a)
|
||||
{
|
||||
REQUIRE_EXT(ctx, RVH);
|
||||
#ifndef CONFIG_USER_ONLY
|
||||
TCGv t0 = tcg_temp_new();
|
||||
TCGv t1 = tcg_temp_new();
|
||||
TCGv mem_idx = tcg_temp_new();
|
||||
TCGv memop = tcg_temp_new();
|
||||
|
||||
check_access(ctx);
|
||||
|
||||
gen_get_gpr(t0, a->rs1);
|
||||
tcg_gen_movi_tl(mem_idx, ctx->mem_idx);
|
||||
tcg_gen_movi_tl(memop, MO_SB);
|
||||
|
||||
gen_helper_hyp_load(t1, cpu_env, t0, mem_idx, memop);
|
||||
tcg_gen_qemu_ld_tl(t1, t0, ctx->mem_idx | TB_FLAGS_PRIV_HYP_ACCESS_MASK, MO_SB);
|
||||
gen_set_gpr(a->rd, t1);
|
||||
|
||||
tcg_temp_free(t0);
|
||||
tcg_temp_free(t1);
|
||||
tcg_temp_free(mem_idx);
|
||||
tcg_temp_free(memop);
|
||||
return true;
|
||||
#else
|
||||
return false;
|
||||
|
@ -48,20 +56,16 @@ static bool trans_hlv_h(DisasContext *ctx, arg_hlv_h *a)
|
|||
#ifndef CONFIG_USER_ONLY
|
||||
TCGv t0 = tcg_temp_new();
|
||||
TCGv t1 = tcg_temp_new();
|
||||
TCGv mem_idx = tcg_temp_new();
|
||||
TCGv memop = tcg_temp_new();
|
||||
|
||||
check_access(ctx);
|
||||
|
||||
gen_get_gpr(t0, a->rs1);
|
||||
tcg_gen_movi_tl(mem_idx, ctx->mem_idx);
|
||||
tcg_gen_movi_tl(memop, MO_TESW);
|
||||
|
||||
gen_helper_hyp_load(t1, cpu_env, t0, mem_idx, memop);
|
||||
tcg_gen_qemu_ld_tl(t1, t0, ctx->mem_idx | TB_FLAGS_PRIV_HYP_ACCESS_MASK, MO_TESW);
|
||||
gen_set_gpr(a->rd, t1);
|
||||
|
||||
tcg_temp_free(t0);
|
||||
tcg_temp_free(t1);
|
||||
tcg_temp_free(mem_idx);
|
||||
tcg_temp_free(memop);
|
||||
return true;
|
||||
#else
|
||||
return false;
|
||||
|
@ -74,20 +78,16 @@ static bool trans_hlv_w(DisasContext *ctx, arg_hlv_w *a)
|
|||
#ifndef CONFIG_USER_ONLY
|
||||
TCGv t0 = tcg_temp_new();
|
||||
TCGv t1 = tcg_temp_new();
|
||||
TCGv mem_idx = tcg_temp_new();
|
||||
TCGv memop = tcg_temp_new();
|
||||
|
||||
check_access(ctx);
|
||||
|
||||
gen_get_gpr(t0, a->rs1);
|
||||
tcg_gen_movi_tl(mem_idx, ctx->mem_idx);
|
||||
tcg_gen_movi_tl(memop, MO_TESL);
|
||||
|
||||
gen_helper_hyp_load(t1, cpu_env, t0, mem_idx, memop);
|
||||
tcg_gen_qemu_ld_tl(t1, t0, ctx->mem_idx | TB_FLAGS_PRIV_HYP_ACCESS_MASK, MO_TESL);
|
||||
gen_set_gpr(a->rd, t1);
|
||||
|
||||
tcg_temp_free(t0);
|
||||
tcg_temp_free(t1);
|
||||
tcg_temp_free(mem_idx);
|
||||
tcg_temp_free(memop);
|
||||
return true;
|
||||
#else
|
||||
return false;
|
||||
|
@ -100,20 +100,16 @@ static bool trans_hlv_bu(DisasContext *ctx, arg_hlv_bu *a)
|
|||
#ifndef CONFIG_USER_ONLY
|
||||
TCGv t0 = tcg_temp_new();
|
||||
TCGv t1 = tcg_temp_new();
|
||||
TCGv mem_idx = tcg_temp_new();
|
||||
TCGv memop = tcg_temp_new();
|
||||
|
||||
check_access(ctx);
|
||||
|
||||
gen_get_gpr(t0, a->rs1);
|
||||
tcg_gen_movi_tl(mem_idx, ctx->mem_idx);
|
||||
tcg_gen_movi_tl(memop, MO_UB);
|
||||
|
||||
gen_helper_hyp_load(t1, cpu_env, t0, mem_idx, memop);
|
||||
tcg_gen_qemu_ld_tl(t1, t0, ctx->mem_idx | TB_FLAGS_PRIV_HYP_ACCESS_MASK, MO_UB);
|
||||
gen_set_gpr(a->rd, t1);
|
||||
|
||||
tcg_temp_free(t0);
|
||||
tcg_temp_free(t1);
|
||||
tcg_temp_free(mem_idx);
|
||||
tcg_temp_free(memop);
|
||||
return true;
|
||||
#else
|
||||
return false;
|
||||
|
@ -126,20 +122,15 @@ static bool trans_hlv_hu(DisasContext *ctx, arg_hlv_hu *a)
|
|||
#ifndef CONFIG_USER_ONLY
|
||||
TCGv t0 = tcg_temp_new();
|
||||
TCGv t1 = tcg_temp_new();
|
||||
TCGv mem_idx = tcg_temp_new();
|
||||
TCGv memop = tcg_temp_new();
|
||||
|
||||
check_access(ctx);
|
||||
|
||||
gen_get_gpr(t0, a->rs1);
|
||||
tcg_gen_movi_tl(mem_idx, ctx->mem_idx);
|
||||
tcg_gen_movi_tl(memop, MO_TEUW);
|
||||
|
||||
gen_helper_hyp_load(t1, cpu_env, t0, mem_idx, memop);
|
||||
tcg_gen_qemu_ld_tl(t1, t0, ctx->mem_idx | TB_FLAGS_PRIV_HYP_ACCESS_MASK, MO_TEUW);
|
||||
gen_set_gpr(a->rd, t1);
|
||||
|
||||
tcg_temp_free(t0);
|
||||
tcg_temp_free(t1);
|
||||
tcg_temp_free(mem_idx);
|
||||
tcg_temp_free(memop);
|
||||
return true;
|
||||
#else
|
||||
return false;
|
||||
|
@ -152,20 +143,16 @@ static bool trans_hsv_b(DisasContext *ctx, arg_hsv_b *a)
|
|||
#ifndef CONFIG_USER_ONLY
|
||||
TCGv t0 = tcg_temp_new();
|
||||
TCGv dat = tcg_temp_new();
|
||||
TCGv mem_idx = tcg_temp_new();
|
||||
TCGv memop = tcg_temp_new();
|
||||
|
||||
check_access(ctx);
|
||||
|
||||
gen_get_gpr(t0, a->rs1);
|
||||
gen_get_gpr(dat, a->rs2);
|
||||
tcg_gen_movi_tl(mem_idx, ctx->mem_idx);
|
||||
tcg_gen_movi_tl(memop, MO_SB);
|
||||
|
||||
gen_helper_hyp_store(cpu_env, t0, dat, mem_idx, memop);
|
||||
tcg_gen_qemu_st_tl(dat, t0, ctx->mem_idx | TB_FLAGS_PRIV_HYP_ACCESS_MASK, MO_SB);
|
||||
|
||||
tcg_temp_free(t0);
|
||||
tcg_temp_free(dat);
|
||||
tcg_temp_free(mem_idx);
|
||||
tcg_temp_free(memop);
|
||||
return true;
|
||||
#else
|
||||
return false;
|
||||
|
@ -178,20 +165,16 @@ static bool trans_hsv_h(DisasContext *ctx, arg_hsv_h *a)
|
|||
#ifndef CONFIG_USER_ONLY
|
||||
TCGv t0 = tcg_temp_new();
|
||||
TCGv dat = tcg_temp_new();
|
||||
TCGv mem_idx = tcg_temp_new();
|
||||
TCGv memop = tcg_temp_new();
|
||||
|
||||
check_access(ctx);
|
||||
|
||||
gen_get_gpr(t0, a->rs1);
|
||||
gen_get_gpr(dat, a->rs2);
|
||||
tcg_gen_movi_tl(mem_idx, ctx->mem_idx);
|
||||
tcg_gen_movi_tl(memop, MO_TESW);
|
||||
|
||||
gen_helper_hyp_store(cpu_env, t0, dat, mem_idx, memop);
|
||||
tcg_gen_qemu_st_tl(dat, t0, ctx->mem_idx | TB_FLAGS_PRIV_HYP_ACCESS_MASK, MO_TESW);
|
||||
|
||||
tcg_temp_free(t0);
|
||||
tcg_temp_free(dat);
|
||||
tcg_temp_free(mem_idx);
|
||||
tcg_temp_free(memop);
|
||||
return true;
|
||||
#else
|
||||
return false;
|
||||
|
@ -204,20 +187,16 @@ static bool trans_hsv_w(DisasContext *ctx, arg_hsv_w *a)
|
|||
#ifndef CONFIG_USER_ONLY
|
||||
TCGv t0 = tcg_temp_new();
|
||||
TCGv dat = tcg_temp_new();
|
||||
TCGv mem_idx = tcg_temp_new();
|
||||
TCGv memop = tcg_temp_new();
|
||||
|
||||
check_access(ctx);
|
||||
|
||||
gen_get_gpr(t0, a->rs1);
|
||||
gen_get_gpr(dat, a->rs2);
|
||||
tcg_gen_movi_tl(mem_idx, ctx->mem_idx);
|
||||
tcg_gen_movi_tl(memop, MO_TESL);
|
||||
|
||||
gen_helper_hyp_store(cpu_env, t0, dat, mem_idx, memop);
|
||||
tcg_gen_qemu_st_tl(dat, t0, ctx->mem_idx | TB_FLAGS_PRIV_HYP_ACCESS_MASK, MO_TESL);
|
||||
|
||||
tcg_temp_free(t0);
|
||||
tcg_temp_free(dat);
|
||||
tcg_temp_free(mem_idx);
|
||||
tcg_temp_free(memop);
|
||||
return true;
|
||||
#else
|
||||
return false;
|
||||
|
@ -231,20 +210,16 @@ static bool trans_hlv_wu(DisasContext *ctx, arg_hlv_wu *a)
|
|||
#ifndef CONFIG_USER_ONLY
|
||||
TCGv t0 = tcg_temp_new();
|
||||
TCGv t1 = tcg_temp_new();
|
||||
TCGv mem_idx = tcg_temp_new();
|
||||
TCGv memop = tcg_temp_new();
|
||||
|
||||
check_access(ctx);
|
||||
|
||||
gen_get_gpr(t0, a->rs1);
|
||||
tcg_gen_movi_tl(mem_idx, ctx->mem_idx);
|
||||
tcg_gen_movi_tl(memop, MO_TEUL);
|
||||
|
||||
gen_helper_hyp_load(t1, cpu_env, t0, mem_idx, memop);
|
||||
tcg_gen_qemu_ld_tl(t1, t0, ctx->mem_idx | TB_FLAGS_PRIV_HYP_ACCESS_MASK, MO_TEUL);
|
||||
gen_set_gpr(a->rd, t1);
|
||||
|
||||
tcg_temp_free(t0);
|
||||
tcg_temp_free(t1);
|
||||
tcg_temp_free(mem_idx);
|
||||
tcg_temp_free(memop);
|
||||
return true;
|
||||
#else
|
||||
return false;
|
||||
|
@ -257,20 +232,16 @@ static bool trans_hlv_d(DisasContext *ctx, arg_hlv_d *a)
|
|||
#ifndef CONFIG_USER_ONLY
|
||||
TCGv t0 = tcg_temp_new();
|
||||
TCGv t1 = tcg_temp_new();
|
||||
TCGv mem_idx = tcg_temp_new();
|
||||
TCGv memop = tcg_temp_new();
|
||||
|
||||
check_access(ctx);
|
||||
|
||||
gen_get_gpr(t0, a->rs1);
|
||||
tcg_gen_movi_tl(mem_idx, ctx->mem_idx);
|
||||
tcg_gen_movi_tl(memop, MO_TEQ);
|
||||
|
||||
gen_helper_hyp_load(t1, cpu_env, t0, mem_idx, memop);
|
||||
tcg_gen_qemu_ld_tl(t1, t0, ctx->mem_idx | TB_FLAGS_PRIV_HYP_ACCESS_MASK, MO_TEQ);
|
||||
gen_set_gpr(a->rd, t1);
|
||||
|
||||
tcg_temp_free(t0);
|
||||
tcg_temp_free(t1);
|
||||
tcg_temp_free(mem_idx);
|
||||
tcg_temp_free(memop);
|
||||
return true;
|
||||
#else
|
||||
return false;
|
||||
|
@ -283,20 +254,16 @@ static bool trans_hsv_d(DisasContext *ctx, arg_hsv_d *a)
|
|||
#ifndef CONFIG_USER_ONLY
|
||||
TCGv t0 = tcg_temp_new();
|
||||
TCGv dat = tcg_temp_new();
|
||||
TCGv mem_idx = tcg_temp_new();
|
||||
TCGv memop = tcg_temp_new();
|
||||
|
||||
check_access(ctx);
|
||||
|
||||
gen_get_gpr(t0, a->rs1);
|
||||
gen_get_gpr(dat, a->rs2);
|
||||
tcg_gen_movi_tl(mem_idx, ctx->mem_idx);
|
||||
tcg_gen_movi_tl(memop, MO_TEQ);
|
||||
|
||||
gen_helper_hyp_store(cpu_env, t0, dat, mem_idx, memop);
|
||||
tcg_gen_qemu_st_tl(dat, t0, ctx->mem_idx | TB_FLAGS_PRIV_HYP_ACCESS_MASK, MO_TEQ);
|
||||
|
||||
tcg_temp_free(t0);
|
||||
tcg_temp_free(dat);
|
||||
tcg_temp_free(mem_idx);
|
||||
tcg_temp_free(memop);
|
||||
return true;
|
||||
#else
|
||||
return false;
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue