[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-riscv] [Qemu-devel] [PATCH v1 5/8] RISC-V: Add priv_ver to Dis
From: |
Alistair Francis |
Subject: |
Re: [Qemu-riscv] [Qemu-devel] [PATCH v1 5/8] RISC-V: Add priv_ver to DisasContext |
Date: |
Thu, 24 Jan 2019 16:45:13 -0800 |
On Thu, Jan 24, 2019 at 4:37 PM Palmer Dabbelt <address@hidden> wrote:
>
> On Tue, 15 Jan 2019 14:25:44 PST (-0800), address@hidden wrote:
> > On Tue, Jan 15, 2019 at 2:24 PM Richard Henderson
> > <address@hidden> wrote:
> >>
> >> On 1/15/19 10:58 AM, Alistair Francis wrote:
> >> > -static void riscv_tr_init_disas_context(DisasContextBase *dcbase,
> >> > CPUState *cs)
> >> > +static void riscv_tr_init_disas_context(DisasContextBase *dcbase,
> >> > CPUState *cpu)
> >>
> >> Why change this? I know there is variation in the naming, but my
> >> preferred default mapping is CPUState *cs, RISCVCPU *cpu.
> >
> > Good point, I have changed it back to cs.
>
> I don't see a v2, so I'm just going to go ahead and squash in
Yeah, I was waiting for more comments before sending a v2 on such a
trivial change.
>
> diff --git a/target/riscv/translate.c b/target/riscv/translate.c
> index 8593c2170af4..b7176cbf98e1 100644
> --- a/target/riscv/translate.c
> +++ b/target/riscv/translate.c
> @@ -2015,10 +2015,10 @@ static void decode_opc(DisasContext *ctx)
> }
> }
>
> -static void riscv_tr_init_disas_context(DisasContextBase *dcbase,
> CPUState *cpu)
> +static void riscv_tr_init_disas_context(DisasContextBase *dcbase,
> CPUState *cs)
> {
> DisasContext *ctx = container_of(dcbase, DisasContext, base);
> - CPURISCVState *env = cpu->env_ptr;
> + CPURISCVState *env = cs->env_ptr;
>
> ctx->pc_succ_insn = ctx->base.pc_first;
> ctx->mem_idx = ctx->base.tb->flags & TB_FLAGS_MMU_MASK;
>
> and add Richard's tag.
Sounds good.
Alistair
>
> >
> > Alistair
> >
> >>
> >> Otherwise,
> >> Reviewed-by: Richard Henderson <address@hidden>
[Qemu-riscv] [PATCH v1 4/8] RISC-V: Use riscv prefix consistently on cpu helpers, Alistair Francis, 2019/01/14
Re: [Qemu-riscv] [Qemu-devel] [PATCH v1 4/8] RISC-V: Use riscv prefix consistently on cpu helpers, Richard Henderson, 2019/01/15
[Qemu-riscv] [PATCH v1 6/8] RISC-V: Add misa to DisasContext, Alistair Francis, 2019/01/14
[Qemu-riscv] [PATCH v1 7/8] RISC-V: Add misa.MAFD checks to translate, Alistair Francis, 2019/01/14
[Qemu-riscv] [PATCH v1 8/8] RISC-V: Add misa runtime write support, Alistair Francis, 2019/01/14