From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:50431) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bzoA1-0001En-RE for qemu-devel@nongnu.org; Thu, 27 Oct 2016 13:08:15 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bzoA0-00010V-Hf for qemu-devel@nongnu.org; Thu, 27 Oct 2016 13:08:13 -0400 Received: from mail-qt0-x243.google.com ([2607:f8b0:400d:c0d::243]:33680) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1bzoA0-00010D-Da for qemu-devel@nongnu.org; Thu, 27 Oct 2016 13:08:12 -0400 Received: by mail-qt0-x243.google.com with SMTP id 12so1498023qtm.0 for ; Thu, 27 Oct 2016 10:08:12 -0700 (PDT) Sender: Richard Henderson From: Richard Henderson Date: Thu, 27 Oct 2016 10:07:24 -0700 Message-Id: <1477588052-10152-8-git-send-email-rth@twiddle.net> In-Reply-To: <1477588052-10152-1-git-send-email-rth@twiddle.net> References: <1477588052-10152-1-git-send-email-rth@twiddle.net> Subject: [Qemu-devel] [PATCH v3 07/15] target-sparc: Implement ldstub_asi inline List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: mark.cave-ayland@ilande.co.uk Tested-by: Mark Cave-Ayland Signed-off-by: Richard Henderson --- target-sparc/translate.c | 52 +++++++++++++++++++----------------------------- 1 file changed, 21 insertions(+), 31 deletions(-) diff --git a/target-sparc/translate.c b/target-sparc/translate.c index 8cd8bb6..8d879a9 100644 --- a/target-sparc/translate.c +++ b/target-sparc/translate.c @@ -2013,6 +2013,20 @@ static void gen_swap(DisasContext *dc, TCGv dst, TCGv src, tcg_temp_free(t0); } +static void gen_ldstub(DisasContext *dc, TCGv dst, TCGv addr, int mmu_idx) +{ + /* ??? Should be atomic. */ + TCGv_i32 t0 = tcg_temp_new_i32(); + TCGv_i32 t1 = tcg_const_i32(0xff); + + gen_address_mask(dc, addr); + tcg_gen_qemu_ld_i32(t0, addr, mmu_idx, MO_UB); + tcg_gen_qemu_st_i32(t1, addr, mmu_idx, MO_UB); + tcg_gen_extu_i32_tl(dst, t0); + tcg_temp_free_i32(t0); + tcg_temp_free_i32(t1); +} + /* asi moves */ #if !defined(CONFIG_USER_ONLY) || defined(TARGET_SPARC64) typedef enum { @@ -2351,25 +2365,12 @@ static void gen_ldstub_asi(DisasContext *dc, TCGv dst, TCGv addr, int insn) switch (da.type) { case GET_ASI_EXCP: break; + case GET_ASI_DIRECT: + gen_ldstub(dc, dst, addr, da.mem_idx); + break; default: - { - TCGv_i32 r_asi = tcg_const_i32(da.asi); - TCGv_i32 r_mop = tcg_const_i32(MO_UB); - TCGv_i64 s64, t64; - - save_state(dc); - t64 = tcg_temp_new_i64(); - gen_helper_ld_asi(t64, cpu_env, addr, r_asi, r_mop); - - s64 = tcg_const_i64(0xff); - gen_helper_st_asi(cpu_env, addr, s64, r_asi, r_mop); - tcg_temp_free_i64(s64); - tcg_temp_free_i32(r_mop); - tcg_temp_free_i32(r_asi); - - tcg_gen_trunc_i64_tl(dst, t64); - tcg_temp_free_i64(t64); - } + /* ??? Should be DAE_invalid_asi. */ + gen_exception(dc, TT_DATA_ACCESS); break; } } @@ -5189,19 +5190,8 @@ static void disas_sparc_insn(DisasContext * dc, unsigned int insn) gen_address_mask(dc, cpu_addr); tcg_gen_qemu_ld16s(cpu_val, cpu_addr, dc->mem_idx); break; - case 0xd: /* ldstub -- XXX: should be atomically */ - { - TCGv r_const; - TCGv tmp = tcg_temp_new(); - - gen_address_mask(dc, cpu_addr); - tcg_gen_qemu_ld8u(tmp, cpu_addr, dc->mem_idx); - r_const = tcg_const_tl(0xff); - tcg_gen_qemu_st8(r_const, cpu_addr, dc->mem_idx); - tcg_gen_mov_tl(cpu_val, tmp); - tcg_temp_free(r_const); - tcg_temp_free(tmp); - } + case 0xd: /* ldstub */ + gen_ldstub(dc, cpu_val, cpu_addr, dc->mem_idx); break; case 0x0f: /* swap, swap register with memory. Also atomically */ -- 2.7.4