[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 09/12] target-mips: save cpu state if instructio
From: |
Aurelien Jarno |
Subject: |
Re: [Qemu-devel] [PATCH 09/12] target-mips: save cpu state if instruction can cause an exception |
Date: |
Fri, 20 Jun 2014 00:13:49 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Thu, Jun 19, 2014 at 03:45:40PM +0100, Leon Alrae wrote:
> Execution of these instructions can trigger exceptions which are supposed
> to update BadInstr/BadInstrP. Therefore saving cpu state in order capture
> the opcode.
As said in the previous patch, the performance impact would be quite
significant, now generating a few additional move per load store.
> Signed-off-by: Leon Alrae <address@hidden>
> ---
> target-mips/translate.c | 19 +++++++++++++++++++
> 1 files changed, 19 insertions(+), 0 deletions(-)
>
> diff --git a/target-mips/translate.c b/target-mips/translate.c
> index b27d22e..6835504 100644
> --- a/target-mips/translate.c
> +++ b/target-mips/translate.c
> @@ -1847,11 +1847,13 @@ static void gen_ld(DisasContext *ctx, uint32_t opc,
> switch (opc) {
> #if defined(TARGET_MIPS64)
> case OPC_LWU:
> + save_cpu_state(ctx, 0);
> tcg_gen_qemu_ld_tl(t0, t0, ctx->mem_idx, MO_TEUL);
> gen_store_gpr(t0, rt);
> opn = "lwu";
> break;
> case OPC_LD:
> + save_cpu_state(ctx, 0);
> tcg_gen_qemu_ld_tl(t0, t0, ctx->mem_idx, MO_TEQ);
> gen_store_gpr(t0, rt);
> opn = "ld";
> @@ -1864,6 +1866,7 @@ static void gen_ld(DisasContext *ctx, uint32_t opc,
> opn = "lld";
> break;
> case OPC_LDL:
> + save_cpu_state(ctx, 0);
> t1 = tcg_temp_new();
> tcg_gen_andi_tl(t1, t0, 7);
> #ifndef TARGET_WORDS_BIGENDIAN
> @@ -1885,6 +1888,7 @@ static void gen_ld(DisasContext *ctx, uint32_t opc,
> opn = "ldl";
> break;
> case OPC_LDR:
> + save_cpu_state(ctx, 0);
> t1 = tcg_temp_new();
> tcg_gen_andi_tl(t1, t0, 7);
> #ifdef TARGET_WORDS_BIGENDIAN
> @@ -1906,6 +1910,7 @@ static void gen_ld(DisasContext *ctx, uint32_t opc,
> opn = "ldr";
> break;
> case OPC_LDPC:
> + save_cpu_state(ctx, 0);
> t1 = tcg_const_tl(pc_relative_pc(ctx));
> gen_op_addr_add(ctx, t0, t0, t1);
> tcg_temp_free(t1);
> @@ -1915,6 +1920,7 @@ static void gen_ld(DisasContext *ctx, uint32_t opc,
> break;
> #endif
> case OPC_LWPC:
> + save_cpu_state(ctx, 0);
> t1 = tcg_const_tl(pc_relative_pc(ctx));
> gen_op_addr_add(ctx, t0, t0, t1);
> tcg_temp_free(t1);
> @@ -1923,31 +1929,37 @@ static void gen_ld(DisasContext *ctx, uint32_t opc,
> opn = "lwpc";
> break;
> case OPC_LW:
> + save_cpu_state(ctx, 0);
> tcg_gen_qemu_ld_tl(t0, t0, ctx->mem_idx, MO_TESL);
> gen_store_gpr(t0, rt);
> opn = "lw";
> break;
> case OPC_LH:
> + save_cpu_state(ctx, 0);
> tcg_gen_qemu_ld_tl(t0, t0, ctx->mem_idx, MO_TESW);
> gen_store_gpr(t0, rt);
> opn = "lh";
> break;
> case OPC_LHU:
> + save_cpu_state(ctx, 0);
> tcg_gen_qemu_ld_tl(t0, t0, ctx->mem_idx, MO_TEUW);
> gen_store_gpr(t0, rt);
> opn = "lhu";
> break;
> case OPC_LB:
> + save_cpu_state(ctx, 0);
> tcg_gen_qemu_ld_tl(t0, t0, ctx->mem_idx, MO_SB);
> gen_store_gpr(t0, rt);
> opn = "lb";
> break;
> case OPC_LBU:
> + save_cpu_state(ctx, 0);
> tcg_gen_qemu_ld_tl(t0, t0, ctx->mem_idx, MO_UB);
> gen_store_gpr(t0, rt);
> opn = "lbu";
> break;
> case OPC_LWL:
> + save_cpu_state(ctx, 0);
> t1 = tcg_temp_new();
> tcg_gen_andi_tl(t1, t0, 3);
> #ifndef TARGET_WORDS_BIGENDIAN
> @@ -1970,6 +1982,7 @@ static void gen_ld(DisasContext *ctx, uint32_t opc,
> opn = "lwl";
> break;
> case OPC_LWR:
> + save_cpu_state(ctx, 0);
> t1 = tcg_temp_new();
> tcg_gen_andi_tl(t1, t0, 3);
> #ifdef TARGET_WORDS_BIGENDIAN
> @@ -2017,6 +2030,7 @@ static void gen_st (DisasContext *ctx, uint32_t opc,
> int rt,
> switch (opc) {
> #if defined(TARGET_MIPS64)
> case OPC_SD:
> + save_cpu_state(ctx, 0);
> tcg_gen_qemu_st_tl(t1, t0, ctx->mem_idx, MO_TEQ);
> opn = "sd";
> break;
> @@ -2032,14 +2046,17 @@ static void gen_st (DisasContext *ctx, uint32_t opc,
> int rt,
> break;
> #endif
> case OPC_SW:
> + save_cpu_state(ctx, 0);
> tcg_gen_qemu_st_tl(t1, t0, ctx->mem_idx, MO_TEUL);
> opn = "sw";
> break;
> case OPC_SH:
> + save_cpu_state(ctx, 0);
> tcg_gen_qemu_st_tl(t1, t0, ctx->mem_idx, MO_TEUW);
> opn = "sh";
> break;
> case OPC_SB:
> + save_cpu_state(ctx, 0);
> tcg_gen_qemu_st_tl(t1, t0, ctx->mem_idx, MO_8);
> opn = "sb";
> break;
> @@ -8370,6 +8387,8 @@ static void gen_farith (DisasContext *ctx, enum fopcode
> op1,
> enum { BINOP, CMPOP, OTHEROP } optype = OTHEROP;
> uint32_t func = ctx->opcode & 0x3f;
>
> + save_cpu_state(ctx, 0);
> +
> switch (op1) {
> case OPC_ADD_S:
> {
> --
> 1.7.5.4
>
>
--
Aurelien Jarno GPG: 4096R/1DDD8C9B
address@hidden http://www.aurel32.net
- [Qemu-devel] [PATCH 02/12] target-mips: update cpu_save/cpu_load to support KScratch registers, (continued)
- [Qemu-devel] [PATCH 02/12] target-mips: update cpu_save/cpu_load to support KScratch registers, Leon Alrae, 2014/06/19
- [Qemu-devel] [PATCH 03/12] target-mips: distinguish between data load and instruction fetch, Leon Alrae, 2014/06/19
- [Qemu-devel] [PATCH 04/12] target-mips: add RI and XI fields to TLB entry, Leon Alrae, 2014/06/19
- [Qemu-devel] [PATCH 05/12] target-mips: update PageGrain and m{t, f}c0 EntryLo{0, 1}, Leon Alrae, 2014/06/19
- [Qemu-devel] [PATCH 08/12] target-mips: add BadInstr and BadInstrP support, Leon Alrae, 2014/06/19
- [Qemu-devel] [PATCH 07/12] target-mips: add TLBINV support, Leon Alrae, 2014/06/19
- [Qemu-devel] [PATCH 06/12] target-mips: add new Read-Inhibit and Execute-Inhibit exceptions, Leon Alrae, 2014/06/19
- [Qemu-devel] [PATCH 09/12] target-mips: save cpu state if instruction can cause an exception, Leon Alrae, 2014/06/19
- Re: [Qemu-devel] [PATCH 09/12] target-mips: save cpu state if instruction can cause an exception,
Aurelien Jarno <=
- [Qemu-devel] [PATCH 10/12] target-mips: update cpu_save/cpu_load to support BadInstr registers, Leon Alrae, 2014/06/19
- [Qemu-devel] [PATCH 11/12] target-mips: enable features in MIPS32R5-generic core, Leon Alrae, 2014/06/19
- [Qemu-devel] [PATCH 12/12] target-mips: enable features in MIPS64R6-generic core, Leon Alrae, 2014/06/19