From nobody Wed Oct 29 11:33:38 2025 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; dmarc=fail(p=none dis=none) header.from=linaro.org Return-Path: Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) by mx.zohomail.com with SMTPS id 1524789137580947.936005486282; Thu, 26 Apr 2018 17:32:17 -0700 (PDT) Received: from localhost ([::1]:45130 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fBrJ9-0006EC-PX for importer@patchew.org; Thu, 26 Apr 2018 20:32:15 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51711) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fBrEF-00026G-L1 for qemu-devel@nongnu.org; Thu, 26 Apr 2018 20:27:12 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1fBrEE-0004Uc-F1 for qemu-devel@nongnu.org; Thu, 26 Apr 2018 20:27:11 -0400 Received: from mail-pg0-x244.google.com ([2607:f8b0:400e:c05::244]:46801) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1fBrEE-0004U5-AR for qemu-devel@nongnu.org; Thu, 26 Apr 2018 20:27:10 -0400 Received: by mail-pg0-x244.google.com with SMTP id z4-v6so134262pgu.13 for ; Thu, 26 Apr 2018 17:27:10 -0700 (PDT) Received: from cloudburst.twiddle.net.com ([2605:e000:112b:41da:c94c:5ee7:de92:7d78]) by smtp.gmail.com with ESMTPSA id g76sm86338pfj.102.2018.04.26.17.27.07 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Thu, 26 Apr 2018 17:27:08 -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=sEh2wGJE6rgMpMZZf3odblZRKaPH0BOzj0JTtUcsDDE=; b=FI1pOc0WIAJOl2HQNSah4DIlYL2rd2/cHstiGpsYsuHJIn+XueVQVzmXYL0mWPdgPd MbNl9A8QziqmF8rtCHmu9JW0sqIJbDVYYVHaVKUaUEv+9NI0wem4S+HT1TTWaCMdmTUl gLcRyx6FHLJ9HyH/brwdBD+p/XRcYfYmFtYbQ= 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=sEh2wGJE6rgMpMZZf3odblZRKaPH0BOzj0JTtUcsDDE=; b=PzFgc97/fkTfbP0sYpS0i4nWHEzpu9GMbHU1tD/57wAKTnL8mhIMsxnL0mYYeHytVS T/bmmhY5XKTpN5uCptD+r8EwWkdZsyaNtltP2M5lx2deOppZIDshWzHpTnqRhS/w2Qw0 5hy38YY4Ube/2VP8qJ6exmNchH6AUN7gHRay/iWuiue4D7JrO4vn1wb+XzngKZsbNM4l vormsvyyDRtayimG3dUatEXeTT3Yp5VYSDnHYUOpZNRYJLiv4dhn9e0KFpwtFjjSS0Pq GB6+MAdhaWXuXY4F3vfOBOmOQYyT24QVAcs4g04hygu8Ntehp44rtaaE2GA1v4hCv0xx xqcQ== X-Gm-Message-State: ALQs6tBzEuCTIWUzbJvJaZLDUgWAcufFBWGiSFReRLiDaDgB2gfb3cSZ 942uvaxG7Ysu/6RuwnZFYoVhtM7ugNM= X-Google-Smtp-Source: AB8JxZoSYXF1B9L18/wu2AG6OEkGg66Wua0Cko0KCkpqI8RAxI4MUnc5L5G9t1ligXbTpo6+xVRyyw== X-Received: by 2002:a63:af44:: with SMTP id s4-v6mr147975pgo.295.1524788829145; Thu, 26 Apr 2018 17:27:09 -0700 (PDT) From: Richard Henderson To: qemu-devel@nongnu.org Date: Thu, 26 Apr 2018 14:26:47 -1000 Message-Id: <20180427002651.28356-6-richard.henderson@linaro.org> X-Mailer: git-send-email 2.14.3 In-Reply-To: <20180427002651.28356-1-richard.henderson@linaro.org> References: <20180427002651.28356-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::244 Subject: [Qemu-devel] [PATCH 5/9] target/riscv: Use new atomic min/max expanders 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: Bastian Koppelmann , qemu-arm@nongnu.org, Michael Clark , Sagar Karandikar , Palmer Dabbelt 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" Cc: Michael Clark Cc: Palmer Dabbelt Cc: Sagar Karandikar Cc: Bastian Koppelmann Signed-off-by: Richard Henderson Reviewed-by: Michael Clark --- target/riscv/translate.c | 72 ++++++++++++++------------------------------= ---- 1 file changed, 20 insertions(+), 52 deletions(-) diff --git a/target/riscv/translate.c b/target/riscv/translate.c index 808eab7f50..9cab717088 100644 --- a/target/riscv/translate.c +++ b/target/riscv/translate.c @@ -725,7 +725,6 @@ static void gen_atomic(DisasContext *ctx, uint32_t opc, TCGv src1, src2, dat; TCGLabel *l1, *l2; TCGMemOp mop; - TCGCond cond; bool aq, rl; =20 /* Extract the size of the atomic operation. */ @@ -823,60 +822,29 @@ static void gen_atomic(DisasContext *ctx, uint32_t op= c, tcg_gen_atomic_fetch_or_tl(src2, src1, src2, ctx->mem_idx, mop); gen_set_gpr(rd, src2); break; - case OPC_RISC_AMOMIN: - cond =3D TCG_COND_LT; - goto do_minmax; - case OPC_RISC_AMOMAX: - cond =3D TCG_COND_GT; - goto do_minmax; - case OPC_RISC_AMOMINU: - cond =3D TCG_COND_LTU; - goto do_minmax; - case OPC_RISC_AMOMAXU: - cond =3D TCG_COND_GTU; - goto do_minmax; - do_minmax: - /* Handle the RL barrier. The AQ barrier is handled along the - parallel path by the SC atomic cmpxchg. On the serial path, - of course, barriers do not matter. */ - if (rl) { - tcg_gen_mb(TCG_MO_ALL | TCG_BAR_STRL); - } - if (tb_cflags(ctx->tb) & CF_PARALLEL) { - l1 =3D gen_new_label(); - gen_set_label(l1); - } else { - l1 =3D NULL; - } - gen_get_gpr(src1, rs1); gen_get_gpr(src2, rs2); - if ((mop & MO_SSIZE) =3D=3D MO_SL) { - /* Sign-extend the register comparison input. */ - tcg_gen_ext32s_tl(src2, src2); - } - dat =3D tcg_temp_local_new(); - tcg_gen_qemu_ld_tl(dat, src1, ctx->mem_idx, mop); - tcg_gen_movcond_tl(cond, src2, dat, src2, dat, src2); - - if (tb_cflags(ctx->tb) & CF_PARALLEL) { - /* Parallel context. Make this operation atomic by verifying - that the memory didn't change while we computed the result.= */ - tcg_gen_atomic_cmpxchg_tl(src2, src1, dat, src2, ctx->mem_idx,= mop); - - /* If the cmpxchg failed, retry. */ - /* ??? There is an assumption here that this will eventually - succeed, such that we don't live-lock. This is not unlike - a similar loop that the compiler would generate for e.g. - __atomic_fetch_and_xor, so don't worry about it. */ - tcg_gen_brcond_tl(TCG_COND_NE, dat, src2, l1); - } else { - /* Serial context. Directly store the result. */ - tcg_gen_qemu_st_tl(src2, src1, ctx->mem_idx, mop); - } - gen_set_gpr(rd, dat); - tcg_temp_free(dat); + tcg_gen_atomic_fetch_smin_tl(src2, src1, src2, ctx->mem_idx, mop); + gen_set_gpr(rd, src2); + break; + case OPC_RISC_AMOMAX: + gen_get_gpr(src1, rs1); + gen_get_gpr(src2, rs2); + tcg_gen_atomic_fetch_smax_tl(src2, src1, src2, ctx->mem_idx, mop); + gen_set_gpr(rd, src2); + break; + case OPC_RISC_AMOMINU: + gen_get_gpr(src1, rs1); + gen_get_gpr(src2, rs2); + tcg_gen_atomic_fetch_umin_tl(src2, src1, src2, ctx->mem_idx, mop); + gen_set_gpr(rd, src2); + break; + case OPC_RISC_AMOMAXU: + gen_get_gpr(src1, rs1); + gen_get_gpr(src2, rs2); + tcg_gen_atomic_fetch_umax_tl(src2, src1, src2, ctx->mem_idx, mop); + gen_set_gpr(rd, src2); break; =20 default: --=20 2.14.3