[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v1 01/36] target/riscv: Convert MIP CSR to target_ulong
From: |
Alistair Francis |
Subject: |
Re: [PATCH v1 01/36] target/riscv: Convert MIP CSR to target_ulong |
Date: |
Thu, 2 Jan 2020 18:08:40 -0800 |
On Thu, Jan 2, 2020 at 10:18 AM Palmer Dabbelt <address@hidden> wrote:
>
> On Mon, 09 Dec 2019 10:10:43 PST (-0800), Alistair Francis wrote:
> > The MIP CSR is a xlen CSR, it was only 32-bits to allow atomic access.
> > Now that we don't use atomics for MIP we can change this back to a xlen
> > CSR.
> >
> > Signed-off-by: Alistair Francis <address@hidden>
> > ---
> > target/riscv/cpu.c | 2 +-
> > target/riscv/cpu.h | 2 +-
> > 2 files changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/target/riscv/cpu.c b/target/riscv/cpu.c
> > index d37861a430..e521ebe2e1 100644
> > --- a/target/riscv/cpu.c
> > +++ b/target/riscv/cpu.c
> > @@ -224,7 +224,7 @@ static void riscv_cpu_dump_state(CPUState *cs, FILE *f,
> > int flags)
> > #ifndef CONFIG_USER_ONLY
> > qemu_fprintf(f, " %s " TARGET_FMT_lx "\n", "mhartid ", env->mhartid);
> > qemu_fprintf(f, " %s " TARGET_FMT_lx "\n", "mstatus ", env->mstatus);
> > - qemu_fprintf(f, " %s 0x%x\n", "mip ", env->mip);
> > + qemu_fprintf(f, " %s " TARGET_FMT_lx "\n", "mip ", env->mip);
> > qemu_fprintf(f, " %s " TARGET_FMT_lx "\n", "mie ", env->mie);
> > qemu_fprintf(f, " %s " TARGET_FMT_lx "\n", "mideleg ", env->mideleg);
> > qemu_fprintf(f, " %s " TARGET_FMT_lx "\n", "medeleg ", env->medeleg);
> > diff --git a/target/riscv/cpu.h b/target/riscv/cpu.h
> > index e59343e13c..f889427869 100644
> > --- a/target/riscv/cpu.h
> > +++ b/target/riscv/cpu.h
> > @@ -121,7 +121,7 @@ struct CPURISCVState {
> > target_ulong mhartid;
> > target_ulong mstatus;
> >
> > - uint32_t mip;
> > + target_ulong mip;
> > uint32_t miclaim;
> >
> > target_ulong mie;
>
> Reviewed-by: Palmer Dabbelt <address@hidden>
Thanks!
Can you just apply the patches from this series as they are reviewed?
Also, can you review this series :)
Alistair