[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 07/20] target/openrisc: Form the spr index from
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH 07/20] target/openrisc: Form the spr index from tcg |
Date: |
Sun, 27 May 2018 22:31:57 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.8.0 |
On 05/27/2018 11:13 AM, Richard Henderson wrote:
> Rather than pass base+offset to the helper, pass the full index.
> In most cases the base is r0 and optimization yields a constant.
and while here you use generic TCGv instead of 32bit version.
>
> Signed-off-by: Richard Henderson <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> target/openrisc/helper.h | 4 ++--
> target/openrisc/sys_helper.c | 9 +++------
> target/openrisc/translate.c | 16 +++++++++-------
> 3 files changed, 14 insertions(+), 15 deletions(-)
>
> diff --git a/target/openrisc/helper.h b/target/openrisc/helper.h
> index e37dabc77a..9db9bf3963 100644
> --- a/target/openrisc/helper.h
> +++ b/target/openrisc/helper.h
> @@ -56,5 +56,5 @@ FOP_CMP(le)
> DEF_HELPER_FLAGS_1(rfe, 0, void, env)
>
> /* sys */
> -DEF_HELPER_FLAGS_4(mtspr, 0, void, env, tl, tl, tl)
> -DEF_HELPER_FLAGS_4(mfspr, TCG_CALL_NO_WG, tl, env, tl, tl, tl)
> +DEF_HELPER_FLAGS_3(mtspr, 0, void, env, tl, tl)
> +DEF_HELPER_FLAGS_3(mfspr, TCG_CALL_NO_WG, tl, env, tl, tl)
> diff --git a/target/openrisc/sys_helper.c b/target/openrisc/sys_helper.c
> index b284064381..a8d287d6ef 100644
> --- a/target/openrisc/sys_helper.c
> +++ b/target/openrisc/sys_helper.c
> @@ -27,13 +27,11 @@
>
> #define TO_SPR(group, number) (((group) << 11) + (number))
>
> -void HELPER(mtspr)(CPUOpenRISCState *env,
> - target_ulong ra, target_ulong rb, target_ulong offset)
> +void HELPER(mtspr)(CPUOpenRISCState *env, target_ulong spr, target_ulong rb)
> {
> #ifndef CONFIG_USER_ONLY
> OpenRISCCPU *cpu = openrisc_env_get_cpu(env);
> CPUState *cs = CPU(cpu);
> - int spr = (ra | offset);
> int idx;
>
> switch (spr) {
> @@ -201,13 +199,12 @@ void HELPER(mtspr)(CPUOpenRISCState *env,
> #endif
> }
>
> -target_ulong HELPER(mfspr)(CPUOpenRISCState *env,
> - target_ulong rd, target_ulong ra, uint32_t offset)
> +target_ulong HELPER(mfspr)(CPUOpenRISCState *env, target_ulong rd,
> + target_ulong spr)
> {
> #ifndef CONFIG_USER_ONLY
> OpenRISCCPU *cpu = openrisc_env_get_cpu(env);
> CPUState *cs = CPU(cpu);
> - int spr = (ra | offset);
> int idx;
>
> switch (spr) {
> diff --git a/target/openrisc/translate.c b/target/openrisc/translate.c
> index c7bfb395b0..b26c473870 100644
> --- a/target/openrisc/translate.c
> +++ b/target/openrisc/translate.c
> @@ -926,9 +926,10 @@ static bool trans_l_mfspr(DisasContext *dc, arg_l_mfspr
> *a, uint32_t insn)
> if (is_user(dc)) {
> gen_illegal_exception(dc);
> } else {
> - TCGv_i32 ti = tcg_const_i32(a->k);
> - gen_helper_mfspr(cpu_R[a->d], cpu_env, cpu_R[a->d], cpu_R[a->a], ti);
> - tcg_temp_free_i32(ti);
> + TCGv spr = tcg_temp_new();
> + tcg_gen_ori_tl(spr, cpu_R[a->a], a->k);
> + gen_helper_mfspr(cpu_R[a->d], cpu_env, cpu_R[a->d], spr);
> + tcg_temp_free(spr);
> }
> return true;
> }
> @@ -940,7 +941,7 @@ static bool trans_l_mtspr(DisasContext *dc, arg_l_mtspr
> *a, uint32_t insn)
> if (is_user(dc)) {
> gen_illegal_exception(dc);
> } else {
> - TCGv_i32 ti;
> + TCGv spr;
>
> /* For SR, we will need to exit the TB to recognize the new
> * exception state. For NPC, in theory this counts as a branch
> @@ -953,9 +954,10 @@ static bool trans_l_mtspr(DisasContext *dc, arg_l_mtspr
> *a, uint32_t insn)
> tcg_gen_movi_tl(cpu_ppc, dc->base.pc_next);
> tcg_gen_movi_tl(cpu_pc, dc->base.pc_next + 4);
>
> - ti = tcg_const_i32(a->k);
> - gen_helper_mtspr(cpu_env, cpu_R[a->a], cpu_R[a->b], ti);
> - tcg_temp_free_i32(ti);
> + spr = tcg_temp_new();
> + tcg_gen_ori_tl(spr, cpu_R[a->a], a->k);
> + gen_helper_mtspr(cpu_env, spr, cpu_R[a->b]);
> + tcg_temp_free(spr);
>
> /* For PPC, we want the value that was just written and not
> the generic update that we'd get from DISAS_EXIT. */
>
- Re: [Qemu-devel] [PATCH 01/20] target/openrisc: Remove DISAS_JUMP & DISAS_TB_JUMP, (continued)
- [Qemu-devel] [PATCH 03/20] target/openrisc: Fix singlestep_enabled, Richard Henderson, 2018/05/27
- [Qemu-devel] [PATCH 05/20] target/openrisc: Split out is_user, Richard Henderson, 2018/05/27
- [Qemu-devel] [PATCH 04/20] target/openrisc: Link more translation blocks, Richard Henderson, 2018/05/27
- [Qemu-devel] [PATCH 08/20] target/openrisc: Merge tlb allocation into CPUOpenRISCState, Richard Henderson, 2018/05/27
- [Qemu-devel] [PATCH 10/20] target/openrisc: Merge mmu_helper.c into mmu.c, Richard Henderson, 2018/05/27
- [Qemu-devel] [PATCH 06/20] target/openrisc: Exit the TB after l.mtspr, Richard Henderson, 2018/05/27
- [Qemu-devel] [PATCH 07/20] target/openrisc: Form the spr index from tcg, Richard Henderson, 2018/05/27
- Re: [Qemu-devel] [PATCH 07/20] target/openrisc: Form the spr index from tcg,
Philippe Mathieu-Daudé <=
- [Qemu-devel] [PATCH 09/20] target/openrisc: Remove indirect function calls for mmu, Richard Henderson, 2018/05/27
- [Qemu-devel] [PATCH 11/20] target/openrisc: Reduce tlb to a single dimension, Richard Henderson, 2018/05/27
- [Qemu-devel] [PATCH 12/20] target/openrisc: Fix tlb flushing in mtspr, Richard Henderson, 2018/05/27
- [Qemu-devel] [PATCH 13/20] target/openrisc: Fix cpu_mmu_index, Richard Henderson, 2018/05/27
- [Qemu-devel] [PATCH 14/20] target/openrisc: Use identical sizes for ITLB and DTLB, Richard Henderson, 2018/05/27
- [Qemu-devel] [PATCH 15/20] target/openrisc: Stub out handle_mmu_fault for softmmu, Richard Henderson, 2018/05/27
- [Qemu-devel] [PATCH 16/20] target/openrisc: Log interrupts, Richard Henderson, 2018/05/27