[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 13/27] target/arm: Decode Load/store register
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH v2 13/27] target/arm: Decode Load/store register (pac) |
Date: |
Fri, 4 Jan 2019 18:52:22 +0000 |
On Fri, 14 Dec 2018 at 05:24, Richard Henderson
<address@hidden> wrote:
>
> Not that there are any stores involved, but why argue with ARM's
> naming convention.
>
> Signed-off-by: Richard Henderson <address@hidden>
> ---
> target/arm/translate-a64.c | 62 ++++++++++++++++++++++++++++++++++++++
> 1 file changed, 62 insertions(+)
>
> diff --git a/target/arm/translate-a64.c b/target/arm/translate-a64.c
> index e62d248894..c57c89d98a 100644
> --- a/target/arm/translate-a64.c
> +++ b/target/arm/translate-a64.c
> @@ -3146,6 +3146,65 @@ static void disas_ldst_atomic(DisasContext *s,
> uint32_t insn,
> s->be_data | size | MO_ALIGN);
> }
>
> +/* PAC memory operations
> + *
> + * 31 30 27 26 24 22 21 12 11 10 5 0
> + * +------+-------+---+-----+-----+---+--------+---+---+----+-----+
> + * | size | 1 1 1 | V | 0 0 | M S | 1 | imm9 | W | 1 | Rn | Rt |
> + * +------+-------+---+-----+-----+------------+---+---+----+-----+
> + *
> + * Rt: the result register
> + * Rn: base address or SP
> + * Rs: the source register for the operation
> + * V: vector flag (always 0 as of v8.3)
> + * M: clear for key DA, set for key DB
> + * W: pre-indexing flag
> + * S: sign for imm9.
> + */
> +static void disas_ldst_pac(DisasContext *s, uint32_t insn,
> + int size, int rt, bool is_vector)
> +{
> + int rn = extract32(insn, 5, 5);
> + bool is_wback = extract32(insn, 11, 1);
> + bool use_key_a = !extract32(insn, 23, 1);
> + int offset, memidx;
> + TCGv_i64 tcg_addr, tcg_rt;
> +
> + if (size != 3 || is_vector || !dc_isar_feature(aa64_pauth, s)) {
> + unallocated_encoding(s);
> + return;
> + }
> +
> + if (rn == 31) {
> + gen_check_sp_alignment(s);
> + }
> + tcg_addr = read_cpu_reg_sp(s, rn, 1);
> +
> + if (s->pauth_active) {
> + if (use_key_a) {
> + gen_helper_autda(tcg_addr, cpu_env, tcg_addr, cpu_X[31]);
> + } else {
> + gen_helper_autdb(tcg_addr, cpu_env, tcg_addr, cpu_X[31]);
> + }
> + }
> +
> + /* Form the 10-bit signed, scaled offset. */
> + offset = (extract32(insn, 22, 1) << 9) | extract32(insn, 12, 9);
> + offset = sextract32(offset << size, 10 + size, 0);
> + tcg_gen_addi_i64(tcg_addr, tcg_addr, offset);
> +
> + tcg_rt = cpu_reg(s, rt);
> + memidx = get_mem_index(s);
> + do_gpr_ld_memidx(s, tcg_rt, tcg_addr, size,
> + /* is_signed */ false, /* extend */ false, memidx,
> + /* iss_valid */ true, /* iss_srt */ rt,
> + /* iss_sf */ true, /* iss_ar */ false);
Since you don't have the "memidx might be something other than
the result of get_mem_index()" case to worry about, you could
use do_gpr_ld() here.
ISS information is not valid for accesses which do writeback.
(We seem to get this wrong in the existing disas_ldst_reg_imm9()...)
> +
> + if (is_wback) {
> + tcg_gen_mov_i64(cpu_reg_sp(s, rn), tcg_addr);
> + }
> +}
> +
> /* Load/store register (all forms) */
> static void disas_ldst_reg(DisasContext *s, uint32_t insn)
> {
> @@ -3171,6 +3230,9 @@ static void disas_ldst_reg(DisasContext *s, uint32_t
> insn)
> case 2:
> disas_ldst_reg_roffset(s, insn, opc, size, rt, is_vector);
> return;
> + default:
> + disas_ldst_pac(s, insn, size, rt, is_vector);
> + return;
> }
> break;
> case 1:
> --
> 2.17.2
>
thanks
-- PMM
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH v2 13/27] target/arm: Decode Load/store register (pac),
Peter Maydell <=