|
From: | Richard Henderson |
Subject: | Re: [PATCH 2/3] target/ppc: Update VMX storage access insns to use tcg_gen_qemu_ld/st_i128. |
Date: | Fri, 21 Jun 2024 09:34:33 -0700 |
User-agent: | Mozilla Thunderbird |
On 6/21/24 04:46, Chinmay Rath wrote:
+ tcg_gen_qemu_ld_i128(avr, EA, ctx->mem_idx, DEF_MEMOP(MO_128)); + set_avr_full(a->rt, avr);
This needs to specify atomicity as well. This is much more important to for 16 byte operations than smaller accesses, as this might require stop-the-world semantics depending on the host.
According to section 1.4 Storage Atomicity, we need no more than 8-byte atomicity for these vector operations, and then the following the alignment bits down.
So: MO_128 | MO_ATOM_IFALIGN_PAIR, r~
[Prev in Thread] | Current Thread | [Next in Thread] |