[for-6.2,27/43] target/arm: Use MO_128 for 16 byte atomics

Message ID	20210729004647.282017-28-richard.henderson@linaro.org
State	Superseded
Headers	show Delivered-To: patch@linaro.org Received-SPF: pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; From: Richard Henderson <richard.henderson@linaro.org> To: qemu-devel@nongnu.org Subject: [PATCH for-6.2 27/43] target/arm: Use MO_128 for 16 byte atomics Date: Wed, 28 Jul 2021 14:46:31 -1000 Message-Id: <20210729004647.282017-28-richard.henderson@linaro.org> In-Reply-To: <20210729004647.282017-1-richard.henderson@linaro.org> References: <20210729004647.282017-1-richard.henderson@linaro.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Received-SPF: pass client-ip=2607:f8b0:4864:20::1030; envelope-from=richard.henderson@linaro.org; helo=mail-pj1-x1030.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=unavailable autolearn_force=no X-Spam_action: no action Precedence: list Cc: qemu-arm@nongnu.org Errors-To: qemu-devel-bounces+patch=linaro.org@nongnu.org Sender: "Qemu-devel" <qemu-devel-bounces+patch=linaro.org@nongnu.org>
Series	Unaligned accesses for user-only \| expand [for-6.2,00/43] Unaligned accesses for user-only [for-6.2,01/43] hw/core: Make do_unaligned_access available to user-only [for-6.2,02/43] target/alpha: Implement do_unaligned_access for user-only [for-6.2,03/43] target/arm: Implement do_unaligned_access for user-only [for-6.2,04/43] target/hppa: Implement do_unaligned_access for user-only [for-6.2,05/43] target/microblaze: Implement do_unaligned_access for user-only [for-6.2,06/43] target/mips: Implement do_unaligned_access for user-only [for-6.2,07/43] target/ppc: Set fault address in ppc_cpu_do_unaligned_access [for-6.2,08/43] target/ppc: Implement do_unaligned_access for user-only [for-6.2,09/43] target/riscv: Implement do_unaligned_access for user-only [for-6.2,10/43] target/s390x: Implement do_unaligned_access for user-only [for-6.2,11/43] target/sh4: Set fault address in superh_cpu_do_unaligned_access [for-6.2,12/43] target/sh4: Implement do_unaligned_access for user-only [for-6.2,13/43] target/sparc: Remove DEBUG_UNALIGNED [for-6.2,14/43] target/sparc: Set fault address in sparc_cpu_do_unaligned_access [for-6.2,15/43] target/sparc: Implement do_unaligned_access for user-only [for-6.2,16/43] target/xtensa: Implement do_unaligned_access for user-only [for-6.2,17/43] accel/tcg: Report unaligned atomics for user-only [for-6.2,18/43] accel/tcg: Drop signness in tracing in cputlb.c [for-6.2,19/43] tcg: Expand MO_SIZE to 3 bits [for-6.2,20/43] tcg: Rename TCGMemOpIdx to MemOpIdx [for-6.2,21/43] tcg: Split out MemOpIdx to exec/memopidx.h [for-6.2,22/43] trace/mem: Pass MemOpIdx to trace_mem_get_info [for-6.2,23/43] accel/tcg: Remove double bswap for helper_atomic_sto__mmu [for-6.2,24/43] accel/tcg: Pass MemOpIdx to atomic_trace__post [for-6.2,25/43] plugins: Reorg arguments to qemu_plugin_vcpu_mem_cb [for-6.2,26/43] trace: Split guest_mem_before [for-6.2,27/43] target/arm: Use MO_128 for 16 byte atomics [for-6.2,28/43] target/i386: Use MO_128 for 16 byte atomics [for-6.2,29/43] target/ppc: Use MO_128 for 16 byte atomics [for-6.2,30/43] target/s390x: Use MO_128 for 16 byte atomics [for-6.2,31/43] target/hexagon: Implement cpu_mmu_index [for-6.2,32/43] accel/tcg: Add cpu_{ld,st}_mmu interfaces [for-6.2,33/43] accel/tcg: Move cpu_atomic decls to exec/cpu_ldst.h [for-6.2,34/43] target/mips: Use cpu__data_ra for msa load/store [for-6.2,35/43] target/mips: Use 8-byte memory ops for msa load/store [for-6.2,36/43] target/s390x: Use cpu__mmu instead of helper__mmu [for-6.2,37/43] target/sparc: Use cpu__mmu instead of helper__mmu [for-6.2,38/43] target/arm: Use cpu__mmu instead of helper__mmu [for-6.2,39/43] tcg: Move helper_*_mmu decls to tcg/tcg-ldst.h [for-6.2,40/43] linux-user/alpha: Remove TARGET_ALIGNED_ONLY [for-6.2,41/43] tcg: Add helper_unaligned_mmu for user-only sigbus [for-6.2,42/43] tcg/i386: Support raising sigbus for user-only [for-6.2,43/43] tests/tcg/multiarch: Add sigbus.c

Message ID

20210729004647.282017-28-richard.henderson@linaro.org

State

Superseded

Headers

Received-SPF: pass (google.com: domain of
	qemu-devel-bounces+patch=linaro.org@nongnu.org designates
	209.51.188.17 as permitted sender) client-ip=209.51.188.17; 
From: Richard Henderson <richard.henderson@linaro.org>
To: qemu-devel@nongnu.org
Subject: [PATCH for-6.2 27/43] target/arm: Use MO_128 for 16 byte atomics
Date: Wed, 28 Jul 2021 14:46:31 -1000
Message-Id: <20210729004647.282017-28-richard.henderson@linaro.org>
In-Reply-To: <20210729004647.282017-1-richard.henderson@linaro.org>
References: <20210729004647.282017-1-richard.henderson@linaro.org>
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Received-SPF: pass client-ip=2607:f8b0:4864:20::1030;
	envelope-from=richard.henderson@linaro.org;
	helo=mail-pj1-x1030.google.com
X-Spam_score_int: -20
X-Spam_score: -2.1
X-Spam_bar: --
X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1,
	DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1,
	RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001,
	SPF_PASS=-0.001 autolearn=unavailable autolearn_force=no
X-Spam_action: no action
X-BeenThere: qemu-devel@nongnu.org
X-Mailman-Version: 2.1.23
Precedence: list
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <https://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
Cc: qemu-arm@nongnu.org
Errors-To: qemu-devel-bounces+patch=linaro.org@nongnu.org
Sender: "Qemu-devel" <qemu-devel-bounces+patch=linaro.org@nongnu.org>

Series

Unaligned accesses for user-only | expand

Comments

Philippe Mathieu-Daudé July 29, 2021, 6:32 a.m. UTC | #1

On 7/29/21 2:46 AM, Richard Henderson wrote:
> Cc: qemu-arm@nongnu.org

> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>

> ---

>  target/arm/helper-a64.c | 8 ++++----

>  1 file changed, 4 insertions(+), 4 deletions(-)


Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

diff --git a/target/arm/helper-a64.c b/target/arm/helper-a64.c
index 13d1e3f808..f06399f351 100644
--- a/target/arm/helper-a64.c
+++ b/target/arm/helper-a64.c
@@ -560,7 +560,7 @@  uint64_t HELPER(paired_cmpxchg64_le_parallel)(CPUARMState *env, uint64_t addr,
     assert(HAVE_CMPXCHG128);
 
     mem_idx = cpu_mmu_index(env, false);
-    oi = make_memop_idx(MO_LEQ | MO_ALIGN_16, mem_idx);
+    oi = make_memop_idx(MO_LE | MO_128 | MO_ALIGN, mem_idx);
 
     cmpv = int128_make128(env->exclusive_val, env->exclusive_high);
     newv = int128_make128(new_lo, new_hi);
@@ -630,7 +630,7 @@  uint64_t HELPER(paired_cmpxchg64_be_parallel)(CPUARMState *env, uint64_t addr,
     assert(HAVE_CMPXCHG128);
 
     mem_idx = cpu_mmu_index(env, false);
-    oi = make_memop_idx(MO_BEQ | MO_ALIGN_16, mem_idx);
+    oi = make_memop_idx(MO_BE | MO_128 | MO_ALIGN, mem_idx);
 
     /*
      * High and low need to be switched here because this is not actually a
@@ -656,7 +656,7 @@  void HELPER(casp_le_parallel)(CPUARMState *env, uint32_t rs, uint64_t addr,
     assert(HAVE_CMPXCHG128);
 
     mem_idx = cpu_mmu_index(env, false);
-    oi = make_memop_idx(MO_LEQ | MO_ALIGN_16, mem_idx);
+    oi = make_memop_idx(MO_LE | MO_128 | MO_ALIGN, mem_idx);
 
     cmpv = int128_make128(env->xregs[rs], env->xregs[rs + 1]);
     newv = int128_make128(new_lo, new_hi);
@@ -677,7 +677,7 @@  void HELPER(casp_be_parallel)(CPUARMState *env, uint32_t rs, uint64_t addr,
     assert(HAVE_CMPXCHG128);
 
     mem_idx = cpu_mmu_index(env, false);
-    oi = make_memop_idx(MO_LEQ | MO_ALIGN_16, mem_idx);
+    oi = make_memop_idx(MO_LE | MO_128 | MO_ALIGN, mem_idx);
 
     cmpv = int128_make128(env->xregs[rs + 1], env->xregs[rs]);
     newv = int128_make128(new_lo, new_hi);

[for-6.2,27/43] target/arm: Use MO_128 for 16 byte atomics

Commit Message

Comments

Patch