From nobody Fri May 3 03:26:33 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) client-ip=208.118.235.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org Return-Path: Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) by mx.zohomail.com with SMTPS id 1504808498423850.1702001708154; Thu, 7 Sep 2017 11:21:38 -0700 (PDT) Received: from localhost ([::1]:41660 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dq1Qn-0004da-CV for importer@patchew.org; Thu, 07 Sep 2017 14:21:37 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41791) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dq1P2-0003IT-LV for qemu-devel@nongnu.org; Thu, 07 Sep 2017 14:19:54 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dq1Ox-0007ZJ-1G for qemu-devel@nongnu.org; Thu, 07 Sep 2017 14:19:48 -0400 Received: from mail-pg0-x229.google.com ([2607:f8b0:400e:c05::229]:37144) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dq1Ow-0007Yk-PA for qemu-devel@nongnu.org; Thu, 07 Sep 2017 14:19:42 -0400 Received: by mail-pg0-x229.google.com with SMTP id d8so854103pgt.4 for ; Thu, 07 Sep 2017 11:19:42 -0700 (PDT) Received: from pike.twiddle.net (97-126-108-236.tukw.qwest.net. [97.126.108.236]) by smtp.gmail.com with ESMTPSA id s68sm412051pfd.72.2017.09.07.11.19.40 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Thu, 07 Sep 2017 11:19:40 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=U8G7jLxyl2fvoViYLUQ2i9RVyDuMQWVbBH5sbJtd/IU=; b=M+2Y06aG+zPRTd2Ca7H8mLcR8oIi/Fx3oM/mCx/OPmkPLHPMSuImh+yEpiraPa/YEr 4uzPHJtOkOJoKVmKIjGeRQvLB7YEZbr0QbNz+UnozkYZ1dlO+WM/gMtREsjb0UXFVv7q S5j1SYCFGYB5wgb0NEChE+8XMZcW1HBdpZ1uU= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=U8G7jLxyl2fvoViYLUQ2i9RVyDuMQWVbBH5sbJtd/IU=; b=gOTLxCRoPBKolWl7qsM+LqjNSfTw+MiCDGH0LSEVvUxHJAm6yg7NaqiaanSXjNP3hV 3bV2oEe8ZgtB0DC7G6z9OD6ACpGxkiVw9+4U02Sa7ewzHnF3FoJwCQEKwWGc8Iwa1yZZ M+1j7+tu/lysJE11Ag9lqZsb6ZIqbCv7eVqOh3OnOzgYx+bwI73jvGiC2H60ptF0mHip +cXRWg3LCPt8ljFOOk2ucFe/X4NmTFIc6fXb+OAgwzBf6/J15zDU8wt10fn+BjZo01Hv t/p9pcK0umn/+O+p5k7GuPsRPrlr35j9Xgc6RUNjnrK5g3uX+PA7hGI5SJh9mwlmcveM FZ1w== X-Gm-Message-State: AHPjjUixJGR/RdL/+5CtwPh/j1FGmxNBrUcD7JbpTZf1xipXDpWRZsWj nKux+xaudtbw3T3d7JjZAw== X-Google-Smtp-Source: ADKCNb6FEF6AAWGG/eBy3uKjOFYA+RDuQsvHhc8vteQylc1i/kucUo9rC+AE1AhjU6xXOJzx3HMJ4g== X-Received: by 10.99.4.8 with SMTP id 8mr216134pge.123.1504808381470; Thu, 07 Sep 2017 11:19:41 -0700 (PDT) From: Richard Henderson To: qemu-devel@nongnu.org Date: Thu, 7 Sep 2017 11:19:35 -0700 Message-Id: <20170907181938.3948-2-richard.henderson@linaro.org> X-Mailer: git-send-email 2.13.5 In-Reply-To: <20170907181938.3948-1-richard.henderson@linaro.org> References: <20170907181938.3948-1-richard.henderson@linaro.org> X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 2607:f8b0:400e:c05::229 Subject: [Qemu-devel] [PULL 1/4] target/alpha: Convert to DisasJumpType X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: peter.maydell@linaro.org, Richard Henderson Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) X-ZohoMail: RDKM_2 RSF_0 Z_629925259 SPT_0 Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" From: Richard Henderson Signed-off-by: Richard Henderson --- target/alpha/translate.c | 132 ++++++++++++++++++++-----------------------= ---- 1 file changed, 55 insertions(+), 77 deletions(-) diff --git a/target/alpha/translate.c b/target/alpha/translate.c index f465752208..a75bf5dd90 100644 --- a/target/alpha/translate.c +++ b/target/alpha/translate.c @@ -25,11 +25,10 @@ #include "exec/exec-all.h" #include "tcg-op.h" #include "exec/cpu_ldst.h" - #include "exec/helper-proto.h" #include "exec/helper-gen.h" - #include "trace-tcg.h" +#include "exec/translator.h" #include "exec/log.h" =20 =20 @@ -73,32 +72,12 @@ struct DisasContext { bool singlestep_enabled; }; =20 -/* Return values from translate_one, indicating the state of the TB. - Note that zero indicates that we are not exiting the TB. */ - -typedef enum { - NO_EXIT, - - /* We have emitted one or more goto_tb. No fixup required. */ - EXIT_GOTO_TB, - - /* We are not using a goto_tb (for whatever reason), but have updated - the PC (for whatever reason), so there's no need to do it again on - exiting the TB. */ - EXIT_PC_UPDATED, - EXIT_PC_UPDATED_NOCHAIN, - - /* We are exiting the TB, but have neither emitted a goto_tb, nor - updated the PC for the next instruction to be executed. */ - EXIT_PC_STALE, - - /* We are exiting the TB due to page crossing or space constraints. */ - EXIT_FALLTHRU, - - /* We are ending the TB with a noreturn function call, e.g. longjmp. - No following code will be executed. */ - EXIT_NORETURN, -} ExitStatus; +/* Target-specific return values from translate_one, indicating the + state of the TB. Note that DISAS_NEXT indicates that we are not + exiting the TB. */ +#define DISAS_PC_UPDATED_NOCHAIN DISAS_TARGET_0 +#define DISAS_PC_UPDATED DISAS_TARGET_1 +#define DISAS_PC_STALE DISAS_TARGET_2 =20 /* global register indexes */ static TCGv_env cpu_env; @@ -301,14 +280,14 @@ static void gen_excp_1(int exception, int error_code) tcg_temp_free_i32(tmp1); } =20 -static ExitStatus gen_excp(DisasContext *ctx, int exception, int error_cod= e) +static DisasJumpType gen_excp(DisasContext *ctx, int exception, int error_= code) { tcg_gen_movi_i64(cpu_pc, ctx->pc); gen_excp_1(exception, error_code); - return EXIT_NORETURN; + return DISAS_NORETURN; } =20 -static inline ExitStatus gen_invalid(DisasContext *ctx) +static inline DisasJumpType gen_invalid(DisasContext *ctx) { return gen_excp(ctx, EXCP_OPCDEC, 0); } @@ -434,9 +413,9 @@ static inline void gen_store_mem(DisasContext *ctx, tcg_temp_free(tmp); } =20 -static ExitStatus gen_store_conditional(DisasContext *ctx, int ra, int rb, - int32_t disp16, int mem_idx, - TCGMemOp op) +static DisasJumpType gen_store_conditional(DisasContext *ctx, int ra, int = rb, + int32_t disp16, int mem_idx, + TCGMemOp op) { TCGLabel *lab_fail, *lab_done; TCGv addr, val; @@ -468,7 +447,7 @@ static ExitStatus gen_store_conditional(DisasContext *c= tx, int ra, int rb, =20 gen_set_label(lab_done); tcg_gen_movi_i64(cpu_lock_addr, -1); - return NO_EXIT; + return DISAS_NEXT; } =20 static bool in_superpage(DisasContext *ctx, int64_t addr) @@ -507,7 +486,7 @@ static bool use_goto_tb(DisasContext *ctx, uint64_t des= t) #endif } =20 -static ExitStatus gen_bdirect(DisasContext *ctx, int ra, int32_t disp) +static DisasJumpType gen_bdirect(DisasContext *ctx, int ra, int32_t disp) { uint64_t dest =3D ctx->pc + (disp << 2); =20 @@ -522,15 +501,15 @@ static ExitStatus gen_bdirect(DisasContext *ctx, int = ra, int32_t disp) tcg_gen_goto_tb(0); tcg_gen_movi_i64(cpu_pc, dest); tcg_gen_exit_tb((uintptr_t)ctx->tb); - return EXIT_GOTO_TB; + return DISAS_NORETURN; } else { tcg_gen_movi_i64(cpu_pc, dest); - return EXIT_PC_UPDATED; + return DISAS_PC_UPDATED; } } =20 -static ExitStatus gen_bcond_internal(DisasContext *ctx, TCGCond cond, - TCGv cmp, int32_t disp) +static DisasJumpType gen_bcond_internal(DisasContext *ctx, TCGCond cond, + TCGv cmp, int32_t disp) { uint64_t dest =3D ctx->pc + (disp << 2); TCGLabel *lab_true =3D gen_new_label(); @@ -547,7 +526,7 @@ static ExitStatus gen_bcond_internal(DisasContext *ctx,= TCGCond cond, tcg_gen_movi_i64(cpu_pc, dest); tcg_gen_exit_tb((uintptr_t)ctx->tb + 1); =20 - return EXIT_GOTO_TB; + return DISAS_NORETURN; } else { TCGv_i64 z =3D tcg_const_i64(0); TCGv_i64 d =3D tcg_const_i64(dest); @@ -558,16 +537,16 @@ static ExitStatus gen_bcond_internal(DisasContext *ct= x, TCGCond cond, tcg_temp_free_i64(z); tcg_temp_free_i64(d); tcg_temp_free_i64(p); - return EXIT_PC_UPDATED; + return DISAS_PC_UPDATED; } } =20 -static ExitStatus gen_bcond(DisasContext *ctx, TCGCond cond, int ra, - int32_t disp, int mask) +static DisasJumpType gen_bcond(DisasContext *ctx, TCGCond cond, int ra, + int32_t disp, int mask) { if (mask) { TCGv tmp =3D tcg_temp_new(); - ExitStatus ret; + DisasJumpType ret; =20 tcg_gen_andi_i64(tmp, load_gpr(ctx, ra), 1); ret =3D gen_bcond_internal(ctx, cond, tmp, disp); @@ -609,11 +588,11 @@ static void gen_fold_mzero(TCGCond cond, TCGv dest, T= CGv src) } } =20 -static ExitStatus gen_fbcond(DisasContext *ctx, TCGCond cond, int ra, - int32_t disp) +static DisasJumpType gen_fbcond(DisasContext *ctx, TCGCond cond, int ra, + int32_t disp) { TCGv cmp_tmp =3D tcg_temp_new(); - ExitStatus ret; + DisasJumpType ret; =20 gen_fold_mzero(cond, cmp_tmp, load_fpr(ctx, ra)); ret =3D gen_bcond_internal(ctx, cond, cmp_tmp, disp); @@ -1159,7 +1138,7 @@ static void gen_rx(DisasContext *ctx, int ra, int set) tcg_temp_free(tmp); } =20 -static ExitStatus gen_call_pal(DisasContext *ctx, int palcode) +static DisasJumpType gen_call_pal(DisasContext *ctx, int palcode) { /* We're emulating OSF/1 PALcode. Many of these are trivial access to internal cpu registers. */ @@ -1185,7 +1164,7 @@ static ExitStatus gen_call_pal(DisasContext *ctx, int= palcode) palcode &=3D 0xbf; goto do_call_pal; } - return NO_EXIT; + return DISAS_NEXT; } =20 #ifndef CONFIG_USER_ONLY @@ -1232,7 +1211,7 @@ static ExitStatus gen_call_pal(DisasContext *ctx, int= palcode) =20 /* Allow interrupts to be recognized right away. */ tcg_gen_movi_i64(cpu_pc, ctx->pc); - return EXIT_PC_UPDATED_NOCHAIN; + return DISAS_PC_UPDATED_NOCHAIN; =20 case 0x36: /* RDPS */ @@ -1270,7 +1249,7 @@ static ExitStatus gen_call_pal(DisasContext *ctx, int= palcode) palcode &=3D 0x3f; goto do_call_pal; } - return NO_EXIT; + return DISAS_NEXT; } #endif return gen_invalid(ctx); @@ -1307,10 +1286,10 @@ static ExitStatus gen_call_pal(DisasContext *ctx, i= nt palcode) tcg_gen_goto_tb(0); tcg_gen_movi_i64(cpu_pc, entry); tcg_gen_exit_tb((uintptr_t)ctx->tb); - return EXIT_GOTO_TB; + return DISAS_NORETURN; } else { tcg_gen_movi_i64(cpu_pc, entry); - return EXIT_PC_UPDATED; + return DISAS_PC_UPDATED; } } #endif @@ -1344,7 +1323,7 @@ static int cpu_pr_data(int pr) return 0; } =20 -static ExitStatus gen_mfpr(DisasContext *ctx, TCGv va, int regno) +static DisasJumpType gen_mfpr(DisasContext *ctx, TCGv va, int regno) { void (*helper)(TCGv); int data; @@ -1366,7 +1345,7 @@ static ExitStatus gen_mfpr(DisasContext *ctx, TCGv va= , int regno) gen_io_start(); helper(va); gen_io_end(); - return EXIT_PC_STALE; + return DISAS_PC_STALE; } else { helper(va); } @@ -1393,10 +1372,10 @@ static ExitStatus gen_mfpr(DisasContext *ctx, TCGv = va, int regno) break; } =20 - return NO_EXIT; + return DISAS_NEXT; } =20 -static ExitStatus gen_mtpr(DisasContext *ctx, TCGv vb, int regno) +static DisasJumpType gen_mtpr(DisasContext *ctx, TCGv vb, int regno) { int data; =20 @@ -1424,7 +1403,7 @@ static ExitStatus gen_mtpr(DisasContext *ctx, TCGv vb= , int regno) case 252: /* HALT */ gen_helper_halt(vb); - return EXIT_PC_STALE; + return DISAS_PC_STALE; =20 case 251: /* ALARM */ @@ -1438,7 +1417,7 @@ static ExitStatus gen_mtpr(DisasContext *ctx, TCGv vb= , int regno) that ended with a CALL_PAL. Since the base register usually on= ly changes during boot, flushing everything works well. */ gen_helper_tb_flush(cpu_env); - return EXIT_PC_STALE; + return DISAS_PC_STALE; =20 case 32 ... 39: /* Accessing the "non-shadow" general registers. */ @@ -1467,7 +1446,7 @@ static ExitStatus gen_mtpr(DisasContext *ctx, TCGv vb= , int regno) break; } =20 - return NO_EXIT; + return DISAS_NEXT; } #endif /* !USER_ONLY*/ =20 @@ -1499,7 +1478,7 @@ static ExitStatus gen_mtpr(DisasContext *ctx, TCGv vb= , int regno) } \ } while (0) =20 -static ExitStatus translate_one(DisasContext *ctx, uint32_t insn) +static DisasJumpType translate_one(DisasContext *ctx, uint32_t insn) { int32_t disp21, disp16, disp12 __attribute__((unused)); uint16_t fn11; @@ -1507,7 +1486,7 @@ static ExitStatus translate_one(DisasContext *ctx, ui= nt32_t insn) bool islit, real_islit; TCGv va, vb, vc, tmp, tmp2; TCGv_i32 t32; - ExitStatus ret; + DisasJumpType ret; =20 /* Decode all instruction fields */ opc =3D extract32(insn, 26, 6); @@ -1530,7 +1509,7 @@ static ExitStatus translate_one(DisasContext *ctx, ui= nt32_t insn) lit =3D 0; } =20 - ret =3D NO_EXIT; + ret =3D DISAS_NEXT; switch (opc) { case 0x00: /* CALL_PAL */ @@ -2432,7 +2411,7 @@ static ExitStatus translate_one(DisasContext *ctx, ui= nt32_t insn) gen_io_start(); gen_helper_load_pcc(va, cpu_env); gen_io_end(); - ret =3D EXIT_PC_STALE; + ret =3D DISAS_PC_STALE; } else { gen_helper_load_pcc(va, cpu_env); } @@ -2480,7 +2459,7 @@ static ExitStatus translate_one(DisasContext *ctx, ui= nt32_t insn) if (ra !=3D 31) { tcg_gen_movi_i64(ctx->ir[ra], ctx->pc); } - ret =3D EXIT_PC_UPDATED; + ret =3D DISAS_PC_UPDATED; break; =20 case 0x1B: @@ -2738,7 +2717,7 @@ static ExitStatus translate_one(DisasContext *ctx, ui= nt32_t insn) tcg_temp_free(tmp); tcg_gen_andi_i64(cpu_pc, vb, ~3); /* Allow interrupts to be recognized right away. */ - ret =3D EXIT_PC_UPDATED_NOCHAIN; + ret =3D DISAS_PC_UPDATED_NOCHAIN; break; #else goto invalid_opc; @@ -2959,7 +2938,7 @@ void gen_intermediate_code(CPUState *cs, struct Trans= lationBlock *tb) target_ulong pc_start; target_ulong pc_mask; uint32_t insn; - ExitStatus ret; + DisasJumpType ret; int num_insns; int max_insns; =20 @@ -3040,41 +3019,40 @@ void gen_intermediate_code(CPUState *cs, struct Tra= nslationBlock *tb) =20 /* If we reach a page boundary, are single stepping, or exhaust instruction count, stop generation. */ - if (ret =3D=3D NO_EXIT + if (ret =3D=3D DISAS_NEXT && ((ctx.pc & pc_mask) =3D=3D 0 || tcg_op_buf_full() || num_insns >=3D max_insns || singlestep || ctx.singlestep_enabled)) { - ret =3D EXIT_FALLTHRU; + ret =3D DISAS_TOO_MANY; } - } while (ret =3D=3D NO_EXIT); + } while (ret =3D=3D DISAS_NEXT); =20 if (tb->cflags & CF_LAST_IO) { gen_io_end(); } =20 switch (ret) { - case EXIT_GOTO_TB: - case EXIT_NORETURN: + case DISAS_NORETURN: break; - case EXIT_FALLTHRU: + case DISAS_TOO_MANY: if (use_goto_tb(&ctx, ctx.pc)) { tcg_gen_goto_tb(0); tcg_gen_movi_i64(cpu_pc, ctx.pc); tcg_gen_exit_tb((uintptr_t)ctx.tb); } /* FALLTHRU */ - case EXIT_PC_STALE: + case DISAS_PC_STALE: tcg_gen_movi_i64(cpu_pc, ctx.pc); /* FALLTHRU */ - case EXIT_PC_UPDATED: + case DISAS_PC_UPDATED: if (!use_exit_tb(&ctx)) { tcg_gen_lookup_and_goto_ptr(cpu_pc); break; } /* FALLTHRU */ - case EXIT_PC_UPDATED_NOCHAIN: + case DISAS_PC_UPDATED_NOCHAIN: if (ctx.singlestep_enabled) { gen_excp_1(EXCP_DEBUG, 0); } else { --=20 2.13.5 From nobody Fri May 3 03:26:33 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) client-ip=208.118.235.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org Return-Path: Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) by mx.zohomail.com with SMTPS id 1504808629952142.42825339091485; Thu, 7 Sep 2017 11:23:49 -0700 (PDT) Received: from localhost ([::1]:41667 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dq1Su-0006ck-R8 for importer@patchew.org; Thu, 07 Sep 2017 14:23:48 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41817) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dq1P4-0003Je-Nu for qemu-devel@nongnu.org; Thu, 07 Sep 2017 14:19:56 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dq1Oz-0007aT-Bp for qemu-devel@nongnu.org; Thu, 07 Sep 2017 14:19:50 -0400 Received: from mail-pf0-x230.google.com ([2607:f8b0:400e:c00::230]:35517) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dq1Oz-0007a2-3J for qemu-devel@nongnu.org; Thu, 07 Sep 2017 14:19:45 -0400 Received: by mail-pf0-x230.google.com with SMTP id g13so718476pfm.2 for ; Thu, 07 Sep 2017 11:19:45 -0700 (PDT) Received: from pike.twiddle.net (97-126-108-236.tukw.qwest.net. [97.126.108.236]) by smtp.gmail.com with ESMTPSA id s68sm412051pfd.72.2017.09.07.11.19.41 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Thu, 07 Sep 2017 11:19:41 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=jakNWEEO0c8nmSTrLjHi9fismG+hgCLE8FRpFEcaAhs=; b=ZSFI6rcKt0p5fbe357Tupm+3YErpWWqZPmbQAn24mD9W3XVi7iPCd9M9FI0LPyJtEc fGLD6tkP8J0nkC1Bv7rH0ue+XVhnsf0lCdqvXZ+Sb7KamMSYOwF2ApZJcQl6TtfHInUs 0cv//ifcnEmRvnMiTvoMpXDPRNgILETrXxX1w= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=jakNWEEO0c8nmSTrLjHi9fismG+hgCLE8FRpFEcaAhs=; b=S4Jl8M0ZOIWzjypMV2sy2aeG5iGAAzM5ArzZBUUmKaq5covkUHenZ8Q8QVB7Ugrtif b6jHlUPpBiNQqTe+locqbqzHkuoO9jb8hUp0XjBU3d7Q4W6bpKdiYJbAmE3vosTSJpT1 1LFnQ8DgcyWZJoBo4gEUeJYf7rpSeA1l3FYFTpQLzEQhCLgd2bnczygSoxr6oSP+DC8h b64FP1VSkS41t/+U5KMImTGIh9GKT9ZU5tQmNK0IqQAcc2J7m8W9ZdGBdci8vtdA6OCL mpeMn35RGHFyI4W9Fzkc2J5bIKSeuI4WT2oz3/b07Jo7YQdvoxFRIAiaNUIRN9rTFBwK Z/Qg== X-Gm-Message-State: AHPjjUhDsrNoHwIjpKyytyMT4QKgmHjjPLjzNXZPUaXn2KhdS+1LweIc X2bXQYuGOSyPVNHMPtjUoA== X-Google-Smtp-Source: ADKCNb7mSkgE6CNw3TBty+oRQwE1kEthTrub96bDO7caEjNRmG8v/Mvq9LioG1SOlugNsAu01rFaUg== X-Received: by 10.84.131.12 with SMTP id 12mr230804pld.69.1504808383398; Thu, 07 Sep 2017 11:19:43 -0700 (PDT) From: Richard Henderson To: qemu-devel@nongnu.org Date: Thu, 7 Sep 2017 11:19:36 -0700 Message-Id: <20170907181938.3948-3-richard.henderson@linaro.org> X-Mailer: git-send-email 2.13.5 In-Reply-To: <20170907181938.3948-1-richard.henderson@linaro.org> References: <20170907181938.3948-1-richard.henderson@linaro.org> X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 2607:f8b0:400e:c00::230 Subject: [Qemu-devel] [PULL 2/4] target/alpha: Convert to DisasContextBase X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: peter.maydell@linaro.org, Richard Henderson Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) X-ZohoMail: RDKM_2 RSF_0 Z_629925259 SPT_0 Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" From: Richard Henderson Signed-off-by: Richard Henderson --- target/alpha/translate.c | 78 ++++++++++++++++++++++++--------------------= ---- 1 file changed, 39 insertions(+), 39 deletions(-) diff --git a/target/alpha/translate.c b/target/alpha/translate.c index a75bf5dd90..ba38717e0f 100644 --- a/target/alpha/translate.c +++ b/target/alpha/translate.c @@ -43,8 +43,8 @@ =20 typedef struct DisasContext DisasContext; struct DisasContext { - struct TranslationBlock *tb; - uint64_t pc; + DisasContextBase base; + #ifndef CONFIG_USER_ONLY uint64_t palbr; #endif @@ -68,8 +68,6 @@ struct DisasContext { TCGv sink; /* Temporary for immediate constants. */ TCGv lit; - - bool singlestep_enabled; }; =20 /* Target-specific return values from translate_one, indicating the @@ -282,7 +280,7 @@ static void gen_excp_1(int exception, int error_code) =20 static DisasJumpType gen_excp(DisasContext *ctx, int exception, int error_= code) { - tcg_gen_movi_i64(cpu_pc, ctx->pc); + tcg_gen_movi_i64(cpu_pc, ctx->base.pc_next); gen_excp_1(exception, error_code); return DISAS_NORETURN; } @@ -463,8 +461,8 @@ static bool in_superpage(DisasContext *ctx, int64_t add= r) =20 static bool use_exit_tb(DisasContext *ctx) { - return ((ctx->tb->cflags & CF_LAST_IO) - || ctx->singlestep_enabled + return ((ctx->base.tb->cflags & CF_LAST_IO) + || ctx->base.singlestep_enabled || singlestep); } =20 @@ -480,7 +478,7 @@ static bool use_goto_tb(DisasContext *ctx, uint64_t des= t) return true; } /* Check for the dest on the same page as the start of the TB. */ - return ((ctx->tb->pc ^ dest) & TARGET_PAGE_MASK) =3D=3D 0; + return ((ctx->base.tb->pc ^ dest) & TARGET_PAGE_MASK) =3D=3D 0; #else return true; #endif @@ -488,10 +486,10 @@ static bool use_goto_tb(DisasContext *ctx, uint64_t d= est) =20 static DisasJumpType gen_bdirect(DisasContext *ctx, int ra, int32_t disp) { - uint64_t dest =3D ctx->pc + (disp << 2); + uint64_t dest =3D ctx->base.pc_next + (disp << 2); =20 if (ra !=3D 31) { - tcg_gen_movi_i64(ctx->ir[ra], ctx->pc); + tcg_gen_movi_i64(ctx->ir[ra], ctx->base.pc_next); } =20 /* Notice branch-to-next; used to initialize RA with the PC. */ @@ -500,7 +498,7 @@ static DisasJumpType gen_bdirect(DisasContext *ctx, int= ra, int32_t disp) } else if (use_goto_tb(ctx, dest)) { tcg_gen_goto_tb(0); tcg_gen_movi_i64(cpu_pc, dest); - tcg_gen_exit_tb((uintptr_t)ctx->tb); + tcg_gen_exit_tb((uintptr_t)ctx->base.tb); return DISAS_NORETURN; } else { tcg_gen_movi_i64(cpu_pc, dest); @@ -511,26 +509,26 @@ static DisasJumpType gen_bdirect(DisasContext *ctx, i= nt ra, int32_t disp) static DisasJumpType gen_bcond_internal(DisasContext *ctx, TCGCond cond, TCGv cmp, int32_t disp) { - uint64_t dest =3D ctx->pc + (disp << 2); + uint64_t dest =3D ctx->base.pc_next + (disp << 2); TCGLabel *lab_true =3D gen_new_label(); =20 if (use_goto_tb(ctx, dest)) { tcg_gen_brcondi_i64(cond, cmp, 0, lab_true); =20 tcg_gen_goto_tb(0); - tcg_gen_movi_i64(cpu_pc, ctx->pc); - tcg_gen_exit_tb((uintptr_t)ctx->tb); + tcg_gen_movi_i64(cpu_pc, ctx->base.pc_next); + tcg_gen_exit_tb((uintptr_t)ctx->base.tb); =20 gen_set_label(lab_true); tcg_gen_goto_tb(1); tcg_gen_movi_i64(cpu_pc, dest); - tcg_gen_exit_tb((uintptr_t)ctx->tb + 1); + tcg_gen_exit_tb((uintptr_t)ctx->base.tb + 1); =20 return DISAS_NORETURN; } else { TCGv_i64 z =3D tcg_const_i64(0); TCGv_i64 d =3D tcg_const_i64(dest); - TCGv_i64 p =3D tcg_const_i64(ctx->pc); + TCGv_i64 p =3D tcg_const_i64(ctx->base.pc_next); =20 tcg_gen_movcond_i64(cond, cpu_pc, cmp, z, d, p); =20 @@ -1210,7 +1208,7 @@ static DisasJumpType gen_call_pal(DisasContext *ctx, = int palcode) } =20 /* Allow interrupts to be recognized right away. */ - tcg_gen_movi_i64(cpu_pc, ctx->pc); + tcg_gen_movi_i64(cpu_pc, ctx->base.pc_next); return DISAS_PC_UPDATED_NOCHAIN; =20 case 0x36: @@ -1260,7 +1258,7 @@ static DisasJumpType gen_call_pal(DisasContext *ctx, = int palcode) #else { TCGv tmp =3D tcg_temp_new(); - uint64_t exc_addr =3D ctx->pc; + uint64_t exc_addr =3D ctx->base.pc_next; uint64_t entry =3D ctx->palbr; =20 if (ctx->tbflags & ENV_FLAG_PAL_MODE) { @@ -1285,7 +1283,7 @@ static DisasJumpType gen_call_pal(DisasContext *ctx, = int palcode) if (!use_exit_tb(ctx)) { tcg_gen_goto_tb(0); tcg_gen_movi_i64(cpu_pc, entry); - tcg_gen_exit_tb((uintptr_t)ctx->tb); + tcg_gen_exit_tb((uintptr_t)ctx->base.tb); return DISAS_NORETURN; } else { tcg_gen_movi_i64(cpu_pc, entry); @@ -2407,7 +2405,7 @@ static DisasJumpType translate_one(DisasContext *ctx,= uint32_t insn) case 0xC000: /* RPCC */ va =3D dest_gpr(ctx, ra); - if (ctx->tb->cflags & CF_USE_ICOUNT) { + if (ctx->base.tb->cflags & CF_USE_ICOUNT) { gen_io_start(); gen_helper_load_pcc(va, cpu_env); gen_io_end(); @@ -2457,7 +2455,7 @@ static DisasJumpType translate_one(DisasContext *ctx,= uint32_t insn) vb =3D load_gpr(ctx, rb); tcg_gen_andi_i64(cpu_pc, vb, ~3); if (ra !=3D 31) { - tcg_gen_movi_i64(ctx->ir[ra], ctx->pc); + tcg_gen_movi_i64(ctx->ir[ra], ctx->base.pc_next); } ret =3D DISAS_PC_UPDATED; break; @@ -2944,13 +2942,14 @@ void gen_intermediate_code(CPUState *cs, struct Tra= nslationBlock *tb) =20 pc_start =3D tb->pc; =20 - ctx.tb =3D tb; - ctx.pc =3D pc_start; + ctx.base.tb =3D tb; + ctx.base.pc_next =3D pc_start; + ctx.base.singlestep_enabled =3D cs->singlestep_enabled; + ctx.tbflags =3D tb->flags; ctx.mem_idx =3D cpu_mmu_index(env, false); ctx.implver =3D env->implver; ctx.amask =3D env->amask; - ctx.singlestep_enabled =3D cs->singlestep_enabled; =20 #ifdef CONFIG_USER_ONLY ctx.ir =3D cpu_std_ir; @@ -2992,39 +2991,40 @@ void gen_intermediate_code(CPUState *cs, struct Tra= nslationBlock *tb) tcg_clear_temp_count(); =20 do { - tcg_gen_insn_start(ctx.pc); + tcg_gen_insn_start(ctx.base.pc_next); num_insns++; =20 - if (unlikely(cpu_breakpoint_test(cs, ctx.pc, BP_ANY))) { + if (unlikely(cpu_breakpoint_test(cs, ctx.base.pc_next, BP_ANY))) { ret =3D gen_excp(&ctx, EXCP_DEBUG, 0); /* The address covered by the breakpoint must be included in [tb->pc, tb->pc + tb->size) in order to for it to be properly cleared -- thus we increment the PC here so that the logic setting tb->size below does the right thing. */ - ctx.pc +=3D 4; + ctx.base.pc_next +=3D 4; break; } if (num_insns =3D=3D max_insns && (tb->cflags & CF_LAST_IO)) { gen_io_start(); } - insn =3D cpu_ldl_code(env, ctx.pc); + insn =3D cpu_ldl_code(env, ctx.base.pc_next); =20 - ctx.pc +=3D 4; + ctx.base.pc_next +=3D 4; ret =3D translate_one(ctxp, insn); free_context_temps(ctxp); =20 if (tcg_check_temp_count()) { - qemu_log("TCG temporary leak before "TARGET_FMT_lx"\n", ctx.pc= ); + qemu_log("TCG temporary leak before "TARGET_FMT_lx"\n", + ctx.base.pc_next); } =20 /* If we reach a page boundary, are single stepping, or exhaust instruction count, stop generation. */ if (ret =3D=3D DISAS_NEXT - && ((ctx.pc & pc_mask) =3D=3D 0 + && ((ctx.base.pc_next & pc_mask) =3D=3D 0 || tcg_op_buf_full() || num_insns >=3D max_insns || singlestep - || ctx.singlestep_enabled)) { + || ctx.base.singlestep_enabled)) { ret =3D DISAS_TOO_MANY; } } while (ret =3D=3D DISAS_NEXT); @@ -3037,14 +3037,14 @@ void gen_intermediate_code(CPUState *cs, struct Tra= nslationBlock *tb) case DISAS_NORETURN: break; case DISAS_TOO_MANY: - if (use_goto_tb(&ctx, ctx.pc)) { + if (use_goto_tb(&ctx, ctx.base.pc_next)) { tcg_gen_goto_tb(0); - tcg_gen_movi_i64(cpu_pc, ctx.pc); - tcg_gen_exit_tb((uintptr_t)ctx.tb); + tcg_gen_movi_i64(cpu_pc, ctx.base.pc_next); + tcg_gen_exit_tb((uintptr_t)ctx.base.tb); } /* FALLTHRU */ case DISAS_PC_STALE: - tcg_gen_movi_i64(cpu_pc, ctx.pc); + tcg_gen_movi_i64(cpu_pc, ctx.base.pc_next); /* FALLTHRU */ case DISAS_PC_UPDATED: if (!use_exit_tb(&ctx)) { @@ -3053,7 +3053,7 @@ void gen_intermediate_code(CPUState *cs, struct Trans= lationBlock *tb) } /* FALLTHRU */ case DISAS_PC_UPDATED_NOCHAIN: - if (ctx.singlestep_enabled) { + if (ctx.base.singlestep_enabled) { gen_excp_1(EXCP_DEBUG, 0); } else { tcg_gen_exit_tb(0); @@ -3065,7 +3065,7 @@ void gen_intermediate_code(CPUState *cs, struct Trans= lationBlock *tb) =20 gen_tb_end(tb, num_insns); =20 - tb->size =3D ctx.pc - pc_start; + tb->size =3D ctx.base.pc_next - pc_start; tb->icount =3D num_insns; =20 #ifdef DEBUG_DISAS @@ -3073,7 +3073,7 @@ void gen_intermediate_code(CPUState *cs, struct Trans= lationBlock *tb) && qemu_log_in_addr_range(pc_start)) { qemu_log_lock(); qemu_log("IN: %s\n", lookup_symbol(pc_start)); - log_target_disas(cs, pc_start, ctx.pc - pc_start, 1); + log_target_disas(cs, pc_start, ctx.base.pc_next - pc_start, 1); qemu_log("\n"); qemu_log_unlock(); } --=20 2.13.5 From nobody Fri May 3 03:26:33 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) client-ip=208.118.235.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org Return-Path: Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) by mx.zohomail.com with SMTPS id 1504808495975502.13429321634214; Thu, 7 Sep 2017 11:21:35 -0700 (PDT) Received: from localhost ([::1]:41659 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dq1Qk-0004Z5-LX for importer@patchew.org; Thu, 07 Sep 2017 14:21:34 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41828) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dq1P5-0003KK-RH for qemu-devel@nongnu.org; Thu, 07 Sep 2017 14:19:59 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dq1P0-0007bG-Lj for qemu-devel@nongnu.org; Thu, 07 Sep 2017 14:19:51 -0400 Received: from mail-pf0-x22f.google.com ([2607:f8b0:400e:c00::22f]:36653) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dq1P0-0007ah-Ag for qemu-devel@nongnu.org; Thu, 07 Sep 2017 14:19:46 -0400 Received: by mail-pf0-x22f.google.com with SMTP id e199so713726pfh.3 for ; Thu, 07 Sep 2017 11:19:46 -0700 (PDT) Received: from pike.twiddle.net (97-126-108-236.tukw.qwest.net. [97.126.108.236]) by smtp.gmail.com with ESMTPSA id s68sm412051pfd.72.2017.09.07.11.19.43 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Thu, 07 Sep 2017 11:19:43 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=rbLGxLy2YQASfI0h8KdMcb+i/UlDRG30LRR7g/jcY/M=; b=SWT4/AY2yHLrH5Xtn0fpM/8cFk2DJ5pkKRHetXCZTE0+Vb0CrWLL/YbP/EcjFng+OL AvmRNoJlLyi/qsMxLXHPAbdStXXFo0fdYMe4SdnJp20Z7WOc98te8T/4a68F1EGNDSL0 PWycpi0XvFoiGwR6i3nqREzfDfFEi38md9gXs= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=rbLGxLy2YQASfI0h8KdMcb+i/UlDRG30LRR7g/jcY/M=; b=GIMbjtQrrA3uU/Ud7yI5Z4cOOcIM6ewzxCrq1aCLmPGiFvVy7iZoaOeLqdxZ+aWl0o c5II077SKWtBmanYgiU+5IjC2xsM9aN7DZ5zIQCDycgaax+GP0RgilDezYyz7MBwFq89 0jVw6jr0HE8U0BlgMqnITDiGorxeAbJMPNC958oGqZrO7/Ypfkflj9YUfllFdFUrstJX 6kUMCRQ8ATyj//ihnLuKzsWK1db/foMVybIbw6olqUMIZY0JeLv7lL60X94Fw3WEnUE5 HVUElBx4Yliad5aNf8K6ZHTbiL7ZYQVC4qABjJPAV80Lvo7HzbYHJUfzUq7QNE7SzKIE oUGw== X-Gm-Message-State: AHPjjUjC+7SzaQfZIbzp4iw6qKuGDnJ3F0oSH0QRZM1+TaUvyOBkbnWV UKSHu9PslEClf+4Hvpl0Eg== X-Google-Smtp-Source: ADKCNb7P0wy6kT+dXoY1z+QDHcBxR1pK0eVh8v9kd/GTCbMly8MHmSNyiFy77RFioBgzErt/V2pI0Q== X-Received: by 10.84.232.6 with SMTP id h6mr274681plk.66.1504808385055; Thu, 07 Sep 2017 11:19:45 -0700 (PDT) From: Richard Henderson To: qemu-devel@nongnu.org Date: Thu, 7 Sep 2017 11:19:37 -0700 Message-Id: <20170907181938.3948-4-richard.henderson@linaro.org> X-Mailer: git-send-email 2.13.5 In-Reply-To: <20170907181938.3948-1-richard.henderson@linaro.org> References: <20170907181938.3948-1-richard.henderson@linaro.org> X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 2607:f8b0:400e:c00::22f Subject: [Qemu-devel] [PULL 3/4] target/alpha: Convert to TranslatorOps X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: peter.maydell@linaro.org, Richard Henderson Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) X-ZohoMail: RDKM_2 RSF_0 Z_629925259 SPT_0 Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" From: Richard Henderson Signed-off-by: Richard Henderson --- target/alpha/translate.c | 185 ++++++++++++++++++++++---------------------= ---- 1 file changed, 86 insertions(+), 99 deletions(-) diff --git a/target/alpha/translate.c b/target/alpha/translate.c index ba38717e0f..5a92c4accb 100644 --- a/target/alpha/translate.c +++ b/target/alpha/translate.c @@ -2929,33 +2929,23 @@ static DisasJumpType translate_one(DisasContext *ct= x, uint32_t insn) return ret; } =20 -void gen_intermediate_code(CPUState *cs, struct TranslationBlock *tb) +static int alpha_tr_init_disas_context(DisasContextBase *dcbase, + CPUState *cpu, int max_insns) { - CPUAlphaState *env =3D cs->env_ptr; - DisasContext ctx, *ctxp =3D &ctx; - target_ulong pc_start; - target_ulong pc_mask; - uint32_t insn; - DisasJumpType ret; - int num_insns; - int max_insns; - - pc_start =3D tb->pc; - - ctx.base.tb =3D tb; - ctx.base.pc_next =3D pc_start; - ctx.base.singlestep_enabled =3D cs->singlestep_enabled; + DisasContext *ctx =3D container_of(dcbase, DisasContext, base); + CPUAlphaState *env =3D cpu->env_ptr; + int64_t bound, mask; =20 - ctx.tbflags =3D tb->flags; - ctx.mem_idx =3D cpu_mmu_index(env, false); - ctx.implver =3D env->implver; - ctx.amask =3D env->amask; + ctx->tbflags =3D ctx->base.tb->flags; + ctx->mem_idx =3D cpu_mmu_index(env, false); + ctx->implver =3D env->implver; + ctx->amask =3D env->amask; =20 #ifdef CONFIG_USER_ONLY - ctx.ir =3D cpu_std_ir; + ctx->ir =3D cpu_std_ir; #else - ctx.palbr =3D env->palbr; - ctx.ir =3D (ctx.tbflags & ENV_FLAG_PAL_MODE ? cpu_pal_ir : cpu_std_ir); + ctx->palbr =3D env->palbr; + ctx->ir =3D (ctx->tbflags & ENV_FLAG_PAL_MODE ? cpu_pal_ir : cpu_std_i= r); #endif =20 /* ??? Every TB begins with unset rounding mode, to be initialized on @@ -2964,96 +2954,87 @@ void gen_intermediate_code(CPUState *cs, struct Tra= nslationBlock *tb) to reset the FP_STATUS to that default at the end of any TB that changes the default. We could even (gasp) dynamiclly figure out what default would be most efficient given the running program. */ - ctx.tb_rm =3D -1; + ctx->tb_rm =3D -1; /* Similarly for flush-to-zero. */ - ctx.tb_ftz =3D -1; + ctx->tb_ftz =3D -1; =20 - TCGV_UNUSED_I64(ctx.zero); - TCGV_UNUSED_I64(ctx.sink); - TCGV_UNUSED_I64(ctx.lit); - - num_insns =3D 0; - max_insns =3D tb->cflags & CF_COUNT_MASK; - if (max_insns =3D=3D 0) { - max_insns =3D CF_COUNT_MASK; - } - if (max_insns > TCG_MAX_INSNS) { - max_insns =3D TCG_MAX_INSNS; - } + TCGV_UNUSED_I64(ctx->zero); + TCGV_UNUSED_I64(ctx->sink); + TCGV_UNUSED_I64(ctx->lit); =20 - if (in_superpage(&ctx, pc_start)) { - pc_mask =3D (1ULL << 41) - 1; + /* Bound the number of insns to execute to those left on the page. */ + if (in_superpage(ctx, ctx->base.pc_first)) { + mask =3D -1ULL << 41; } else { - pc_mask =3D ~TARGET_PAGE_MASK; + mask =3D TARGET_PAGE_MASK; } + bound =3D -(ctx->base.pc_first | mask) / 4; =20 - gen_tb_start(tb); - tcg_clear_temp_count(); + return MIN(max_insns, bound); +} =20 - do { - tcg_gen_insn_start(ctx.base.pc_next); - num_insns++; +static void alpha_tr_tb_start(DisasContextBase *db, CPUState *cpu) +{ +} =20 - if (unlikely(cpu_breakpoint_test(cs, ctx.base.pc_next, BP_ANY))) { - ret =3D gen_excp(&ctx, EXCP_DEBUG, 0); - /* The address covered by the breakpoint must be included in - [tb->pc, tb->pc + tb->size) in order to for it to be - properly cleared -- thus we increment the PC here so that - the logic setting tb->size below does the right thing. */ - ctx.base.pc_next +=3D 4; - break; - } - if (num_insns =3D=3D max_insns && (tb->cflags & CF_LAST_IO)) { - gen_io_start(); - } - insn =3D cpu_ldl_code(env, ctx.base.pc_next); +static void alpha_tr_insn_start(DisasContextBase *dcbase, CPUState *cpu) +{ + tcg_gen_insn_start(dcbase->pc_next); +} =20 - ctx.base.pc_next +=3D 4; - ret =3D translate_one(ctxp, insn); - free_context_temps(ctxp); +static bool alpha_tr_breakpoint_check(DisasContextBase *dcbase, CPUState *= cpu, + const CPUBreakpoint *bp) +{ + DisasContext *ctx =3D container_of(dcbase, DisasContext, base); =20 - if (tcg_check_temp_count()) { - qemu_log("TCG temporary leak before "TARGET_FMT_lx"\n", - ctx.base.pc_next); - } + ctx->base.is_jmp =3D gen_excp(ctx, EXCP_DEBUG, 0); =20 - /* If we reach a page boundary, are single stepping, - or exhaust instruction count, stop generation. */ - if (ret =3D=3D DISAS_NEXT - && ((ctx.base.pc_next & pc_mask) =3D=3D 0 - || tcg_op_buf_full() - || num_insns >=3D max_insns - || singlestep - || ctx.base.singlestep_enabled)) { - ret =3D DISAS_TOO_MANY; - } - } while (ret =3D=3D DISAS_NEXT); + /* The address covered by the breakpoint must be included in + [tb->pc, tb->pc + tb->size) in order to for it to be + properly cleared -- thus we increment the PC here so that + the logic setting tb->size below does the right thing. */ + ctx->base.pc_next +=3D 4; + return true; +} =20 - if (tb->cflags & CF_LAST_IO) { - gen_io_end(); - } +static void alpha_tr_translate_insn(DisasContextBase *dcbase, CPUState *cp= u) +{ + DisasContext *ctx =3D container_of(dcbase, DisasContext, base); + CPUAlphaState *env =3D cpu->env_ptr; + uint32_t insn =3D cpu_ldl_code(env, ctx->base.pc_next); + + ctx->base.pc_next +=3D 4; + ctx->base.is_jmp =3D translate_one(ctx, insn); + + free_context_temps(ctx); + translator_loop_temp_check(&ctx->base); +} + +static void alpha_tr_tb_stop(DisasContextBase *dcbase, CPUState *cpu) +{ + DisasContext *ctx =3D container_of(dcbase, DisasContext, base); =20 - switch (ret) { + switch (ctx->base.is_jmp) { case DISAS_NORETURN: break; case DISAS_TOO_MANY: - if (use_goto_tb(&ctx, ctx.base.pc_next)) { + if (use_goto_tb(ctx, ctx->base.pc_next)) { tcg_gen_goto_tb(0); - tcg_gen_movi_i64(cpu_pc, ctx.base.pc_next); - tcg_gen_exit_tb((uintptr_t)ctx.base.tb); + tcg_gen_movi_i64(cpu_pc, ctx->base.pc_next); + tcg_gen_exit_tb((uintptr_t)ctx->base.tb); } /* FALLTHRU */ case DISAS_PC_STALE: - tcg_gen_movi_i64(cpu_pc, ctx.base.pc_next); + tcg_gen_movi_i64(cpu_pc, ctx->base.pc_next); /* FALLTHRU */ case DISAS_PC_UPDATED: - if (!use_exit_tb(&ctx)) { + if (!use_exit_tb(ctx)) { tcg_gen_lookup_and_goto_ptr(cpu_pc); break; } /* FALLTHRU */ case DISAS_PC_UPDATED_NOCHAIN: - if (ctx.base.singlestep_enabled) { + if (ctx->base.singlestep_enabled) { gen_excp_1(EXCP_DEBUG, 0); } else { tcg_gen_exit_tb(0); @@ -3062,22 +3043,28 @@ void gen_intermediate_code(CPUState *cs, struct Tra= nslationBlock *tb) default: g_assert_not_reached(); } +} =20 - gen_tb_end(tb, num_insns); - - tb->size =3D ctx.base.pc_next - pc_start; - tb->icount =3D num_insns; +static void alpha_tr_disas_log(const DisasContextBase *dcbase, CPUState *c= pu) +{ + qemu_log("IN: %s\n", lookup_symbol(dcbase->pc_first)); + log_target_disas(cpu, dcbase->pc_first, dcbase->tb->size, 1); +} + +static const TranslatorOps alpha_tr_ops =3D { + .init_disas_context =3D alpha_tr_init_disas_context, + .tb_start =3D alpha_tr_tb_start, + .insn_start =3D alpha_tr_insn_start, + .breakpoint_check =3D alpha_tr_breakpoint_check, + .translate_insn =3D alpha_tr_translate_insn, + .tb_stop =3D alpha_tr_tb_stop, + .disas_log =3D alpha_tr_disas_log, +}; =20 -#ifdef DEBUG_DISAS - if (qemu_loglevel_mask(CPU_LOG_TB_IN_ASM) - && qemu_log_in_addr_range(pc_start)) { - qemu_log_lock(); - qemu_log("IN: %s\n", lookup_symbol(pc_start)); - log_target_disas(cs, pc_start, ctx.base.pc_next - pc_start, 1); - qemu_log("\n"); - qemu_log_unlock(); - } -#endif +void gen_intermediate_code(CPUState *cpu, TranslationBlock *tb) +{ + DisasContext dc; + translator_loop(&alpha_tr_ops, &dc.base, cpu, tb); } =20 void restore_state_to_opc(CPUAlphaState *env, TranslationBlock *tb, --=20 2.13.5 From nobody Fri May 3 03:26:33 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) client-ip=208.118.235.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org Return-Path: Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) by mx.zohomail.com with SMTPS id 150480862109814.201963959927639; Thu, 7 Sep 2017 11:23:41 -0700 (PDT) Received: from localhost ([::1]:41665 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dq1Sl-0006Us-Ok for importer@patchew.org; Thu, 07 Sep 2017 14:23:39 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41840) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dq1P6-0003Kr-Ol for qemu-devel@nongnu.org; Thu, 07 Sep 2017 14:19:59 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dq1P1-0007bn-NI for qemu-devel@nongnu.org; Thu, 07 Sep 2017 14:19:52 -0400 Received: from mail-pf0-x236.google.com ([2607:f8b0:400e:c00::236]:35517) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dq1P1-0007bS-G5 for qemu-devel@nongnu.org; Thu, 07 Sep 2017 14:19:47 -0400 Received: by mail-pf0-x236.google.com with SMTP id g13so718624pfm.2 for ; Thu, 07 Sep 2017 11:19:47 -0700 (PDT) Received: from pike.twiddle.net (97-126-108-236.tukw.qwest.net. [97.126.108.236]) by smtp.gmail.com with ESMTPSA id s68sm412051pfd.72.2017.09.07.11.19.45 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Thu, 07 Sep 2017 11:19:45 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=p0w2LpuRgVCzFXHPmhLKnY8BIvj5eDTupLbjXxouJWo=; b=UpuvbImH8LyU6ghQywqm4LJlEOZ+BzDiEj0sdcbDTy24L0lMdbkM+kNhJ8KSlpsHCP aM4oqsTbKF7kQzhmdN614AyNu5HmvGCp9ieDWXEhc3EykqVhu0znej9164m/Oja+C+cQ /BGtAuvIcBzch5KcuQqJz+ULuA1Tsi8FH6xys= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=p0w2LpuRgVCzFXHPmhLKnY8BIvj5eDTupLbjXxouJWo=; b=tiI8tx0vVuC977yTLt0SyYE9DErTuonGCdYTGuKY0mzAQtCQs3nP2RFo6IQqVFUhus mlBlzxJLzu9lKPwxnlLOZ6/V+G8LDmxwQ0ItGP0sY0pxCXQodTHbOjy9WYiJDhvwXd7+ y7wFz6v8IpMhUGFNZXr0XEU0JK9TeQKmSHC+NFfP9QGQC6ThS95C3d1iYca9jbsEPOpH P0XkjNjYWQb/RbHy0SmCfd8Ta9OYaYNIkGTTDk+ZsUAOeVMj0T5fSNoWHIjZuYLn28rH ZXWm2KcQmP6KMXqh3b89dGQibQumGXXtUJ9zLxTQTf288CdTX/iugF3LKzi/q+4H6k4i hBnQ== X-Gm-Message-State: AHPjjUjUuPqYootd/DuBkLhyKOw8omlhphp9JQH+4lVr3mKGJwaNWuTr kLCl/6WBmYcm/UL0hFZHxg== X-Google-Smtp-Source: ADKCNb70vl5LFfVSrDBjiX1FtRnYBXD/b0BTh1f+Ymp2e45xCjnbQHb3gPPVONn2UT4xcZi+ZwD3BQ== X-Received: by 10.84.210.226 with SMTP id a89mr270305pli.160.1504808386323; Thu, 07 Sep 2017 11:19:46 -0700 (PDT) From: Richard Henderson To: qemu-devel@nongnu.org Date: Thu, 7 Sep 2017 11:19:38 -0700 Message-Id: <20170907181938.3948-5-richard.henderson@linaro.org> X-Mailer: git-send-email 2.13.5 In-Reply-To: <20170907181938.3948-1-richard.henderson@linaro.org> References: <20170907181938.3948-1-richard.henderson@linaro.org> X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 2607:f8b0:400e:c00::236 Subject: [Qemu-devel] [PULL 4/4] target/alpha: Switch to do_transaction_failed() hook X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: peter.maydell@linaro.org Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) X-ZohoMail: RDKM_2 RSF_0 Z_629925259 SPT_0 Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" From: Peter Maydell Switch the alpha target from the old unassigned_access hook to the new do_transaction_failed hook. This allows us to resolve a ??? in the old hook implementation. The only part of the alpha target that does physical memory accesses is reading the page table -- add a TODO comment there to the effect that we should handle bus faults on page table walks. (Since the palcode doesn't actually do anything useful on a bus fault anyway it's a bit moot for now.) Signed-off-by: Peter Maydell Message-Id: <1502196172-13818-1-git-send-email-peter.maydell@linaro.org> Signed-off-by: Richard Henderson --- target/alpha/cpu.c | 2 +- target/alpha/cpu.h | 8 +++++--- target/alpha/helper.c | 8 ++++++++ target/alpha/mem_helper.c | 19 ++++++++++--------- 4 files changed, 24 insertions(+), 13 deletions(-) diff --git a/target/alpha/cpu.c b/target/alpha/cpu.c index 1ea597b9dd..e6c6aabdf0 100644 --- a/target/alpha/cpu.c +++ b/target/alpha/cpu.c @@ -297,7 +297,7 @@ static void alpha_cpu_class_init(ObjectClass *oc, void = *data) #ifdef CONFIG_USER_ONLY cc->handle_mmu_fault =3D alpha_cpu_handle_mmu_fault; #else - cc->do_unassigned_access =3D alpha_cpu_unassigned_access; + cc->do_transaction_failed =3D alpha_cpu_do_transaction_failed; cc->do_unaligned_access =3D alpha_cpu_do_unaligned_access; cc->get_phys_page_debug =3D alpha_cpu_get_phys_page_debug; dc->vmsd =3D &vmstate_alpha_cpu; diff --git a/target/alpha/cpu.h b/target/alpha/cpu.h index 0738e97d6d..6ae240969b 100644 --- a/target/alpha/cpu.h +++ b/target/alpha/cpu.h @@ -486,9 +486,11 @@ void cpu_alpha_store_fpcr (CPUAlphaState *env, uint64_= t val); uint64_t cpu_alpha_load_gr(CPUAlphaState *env, unsigned reg); void cpu_alpha_store_gr(CPUAlphaState *env, unsigned reg, uint64_t val); #ifndef CONFIG_USER_ONLY -QEMU_NORETURN void alpha_cpu_unassigned_access(CPUState *cpu, hwaddr addr, - bool is_write, bool is_exec, - int unused, unsigned size); +void alpha_cpu_do_transaction_failed(CPUState *cs, hwaddr physaddr, + vaddr addr, unsigned size, + MMUAccessType access_type, + int mmu_idx, MemTxAttrs attrs, + MemTxResult response, uintptr_t retad= dr); #endif =20 static inline void cpu_get_tb_cpu_state(CPUAlphaState *env, target_ulong *= pc, diff --git a/target/alpha/helper.c b/target/alpha/helper.c index 34121f4cad..36407f77f5 100644 --- a/target/alpha/helper.c +++ b/target/alpha/helper.c @@ -163,6 +163,14 @@ static int get_physical_address(CPUAlphaState *env, ta= rget_ulong addr, =20 pt =3D env->ptbr; =20 + /* TODO: rather than using ldq_phys() to read the page table we should + * use address_space_ldq() so that we can handle the case when + * the page table read gives a bus fault, rather than ignoring it. + * For the existing code the zero data that ldq_phys will return for + * an access to invalid memory will result in our treating the page + * table as invalid, which may even be the right behaviour. + */ + /* L1 page table read. */ index =3D (addr >> (TARGET_PAGE_BITS + 20)) & 0x3ff; L1pte =3D ldq_phys(cs->as, pt + index*8); diff --git a/target/alpha/mem_helper.c b/target/alpha/mem_helper.c index 78a7d45590..3c06baa93a 100644 --- a/target/alpha/mem_helper.c +++ b/target/alpha/mem_helper.c @@ -49,22 +49,23 @@ void alpha_cpu_do_unaligned_access(CPUState *cs, vaddr = addr, cpu_loop_exit(cs); } =20 -void alpha_cpu_unassigned_access(CPUState *cs, hwaddr addr, - bool is_write, bool is_exec, int unused, - unsigned size) +void alpha_cpu_do_transaction_failed(CPUState *cs, hwaddr physaddr, + vaddr addr, unsigned size, + MMUAccessType access_type, + int mmu_idx, MemTxAttrs attrs, + MemTxResult response, uintptr_t retad= dr) { AlphaCPU *cpu =3D ALPHA_CPU(cs); CPUAlphaState *env =3D &cpu->env; =20 + if (retaddr) { + cpu_restore_state(cs, retaddr); + } + env->trap_arg0 =3D addr; - env->trap_arg1 =3D is_write ? 1 : 0; + env->trap_arg1 =3D access_type =3D=3D MMU_DATA_STORE ? 1 : 0; cs->exception_index =3D EXCP_MCHK; env->error_code =3D 0; - - /* ??? We should cpu_restore_state to the faulting insn, but this hook - does not have access to the retaddr value from the original helper. - It's all moot until the QEMU PALcode grows an MCHK handler. */ - cpu_loop_exit(cs); } =20 --=20 2.13.5