From: Rahul Pathak <rpathak@ventanamicro.com> To: Alistair Francis <alistair23@gmail.com> Cc: Alistair Francis <Alistair.Francis@wdc.com>, Bin Meng <bmeng.cn@gmail.com>, "open list:RISC-V" <qemu-riscv@nongnu.org>, "qemu-devel@nongnu.org Developers" <qemu-devel@nongnu.org> Subject: Re: [PATCH v2 2/2] target/riscv: csr: Implement mconfigptr CSR Date: Thu, 28 Oct 2021 08:01:42 +0530 [thread overview] Message-ID: <CA+Oz1=Ya_j-d1eMJ+jrw=TNWd6EOM613vONR9UU=9ny5MGg=Hg@mail.gmail.com> (raw) In-Reply-To: <CAKmqyKMVEZkKdp0vmFo_tc4oFuVhaTRXBiB-uEksYtfjyRibqw@mail.gmail.com> [-- Attachment #1: Type: text/plain, Size: 2947 bytes --] On Wed, Oct 27, 2021 at 8:13 AM Alistair Francis <alistair23@gmail.com> wrote: > On Mon, Oct 25, 2021 at 10:55 PM Rahul Pathak <rpathak@ventanamicro.com> > wrote: > > > > Signed-off-by: Rahul Pathak <rpathak@ventanamicro.com> > > --- > > target/riscv/cpu_bits.h | 1 + > > target/riscv/csr.c | 19 +++++++++++++++---- > > 2 files changed, 16 insertions(+), 4 deletions(-) > > > > diff --git a/target/riscv/cpu_bits.h b/target/riscv/cpu_bits.h > > index cffcd3a5df..e2f154b7c5 100644 > > --- a/target/riscv/cpu_bits.h > > +++ b/target/riscv/cpu_bits.h > > @@ -140,6 +140,7 @@ > > #define CSR_MARCHID 0xf12 > > #define CSR_MIMPID 0xf13 > > #define CSR_MHARTID 0xf14 > > +#define CSR_MCONFIGPTR 0xf15 > > > > /* Machine Trap Setup */ > > #define CSR_MSTATUS 0x300 > > diff --git a/target/riscv/csr.c b/target/riscv/csr.c > > index 69e4d65fcd..2d7f608d49 100644 > > --- a/target/riscv/csr.c > > +++ b/target/riscv/csr.c > > @@ -209,6 +209,16 @@ static RISCVException epmp(CPURISCVState *env, int > csrno) > > > > return RISCV_EXCP_ILLEGAL_INST; > > } > > + > > +static RISCVException priv1p12(CPURISCVState *env, int csrno) > > +{ > > + if (env->priv_ver >= PRIV_VERSION_1_12_0) { > > + return RISCV_EXCP_NONE; > > + } > > + > > + return RISCV_EXCP_ILLEGAL_INST; > > +} > > + > > #endif > > > > /* User Floating-Point CSRs */ > > @@ -1569,10 +1579,11 @@ riscv_csr_operations csr_ops[CSR_TABLE_SIZE] = { > > [CSR_MINSTRETH] = { "minstreth", any32, read_instreth }, > > > > /* Machine Information Registers */ > > - [CSR_MVENDORID] = { "mvendorid", any, read_zero }, > > - [CSR_MARCHID] = { "marchid", any, read_zero }, > > - [CSR_MIMPID] = { "mimpid", any, read_zero }, > > - [CSR_MHARTID] = { "mhartid", any, read_mhartid }, > > + [CSR_MVENDORID] = { "mvendorid", any, read_zero }, > > + [CSR_MARCHID] = { "marchid", any, read_zero }, > > + [CSR_MIMPID] = { "mimpid", any, read_zero }, > > + [CSR_MHARTID] = { "mhartid", any, read_mhartid }, > > Why change these? > The alignment of all structure entries is consistent in their respective blocks, that's why I aligned these with the mconfigptr line. It's really not necessary and if my observation on the alignment is really not a requirement I will undo this. > > > > > + [CSR_MCONFIGPTR] = {"mconfigptr", priv1p12, read_zero }, > > This looks fine, but there are more changes then this in v1.12. > Looking at the preface we need mret/sret changes at least. It also > looks like some other changes will need to be implemented or at least > checked. > > Agree, I will look into that > Alistair > > > > > /* Machine Trap Setup */ > > [CSR_MSTATUS] = { "mstatus", any, read_mstatus, > write_mstatus }, > > -- > > 2.25.1 > > > > > [-- Attachment #2: Type: text/html, Size: 4932 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Rahul Pathak <rpathak@ventanamicro.com> To: Alistair Francis <alistair23@gmail.com> Cc: "open list:RISC-V" <qemu-riscv@nongnu.org>, "qemu-devel@nongnu.org Developers" <qemu-devel@nongnu.org>, Bin Meng <bmeng.cn@gmail.com>, Alistair Francis <Alistair.Francis@wdc.com> Subject: Re: [PATCH v2 2/2] target/riscv: csr: Implement mconfigptr CSR Date: Thu, 28 Oct 2021 08:01:42 +0530 [thread overview] Message-ID: <CA+Oz1=Ya_j-d1eMJ+jrw=TNWd6EOM613vONR9UU=9ny5MGg=Hg@mail.gmail.com> (raw) In-Reply-To: <CAKmqyKMVEZkKdp0vmFo_tc4oFuVhaTRXBiB-uEksYtfjyRibqw@mail.gmail.com> [-- Attachment #1: Type: text/plain, Size: 2947 bytes --] On Wed, Oct 27, 2021 at 8:13 AM Alistair Francis <alistair23@gmail.com> wrote: > On Mon, Oct 25, 2021 at 10:55 PM Rahul Pathak <rpathak@ventanamicro.com> > wrote: > > > > Signed-off-by: Rahul Pathak <rpathak@ventanamicro.com> > > --- > > target/riscv/cpu_bits.h | 1 + > > target/riscv/csr.c | 19 +++++++++++++++---- > > 2 files changed, 16 insertions(+), 4 deletions(-) > > > > diff --git a/target/riscv/cpu_bits.h b/target/riscv/cpu_bits.h > > index cffcd3a5df..e2f154b7c5 100644 > > --- a/target/riscv/cpu_bits.h > > +++ b/target/riscv/cpu_bits.h > > @@ -140,6 +140,7 @@ > > #define CSR_MARCHID 0xf12 > > #define CSR_MIMPID 0xf13 > > #define CSR_MHARTID 0xf14 > > +#define CSR_MCONFIGPTR 0xf15 > > > > /* Machine Trap Setup */ > > #define CSR_MSTATUS 0x300 > > diff --git a/target/riscv/csr.c b/target/riscv/csr.c > > index 69e4d65fcd..2d7f608d49 100644 > > --- a/target/riscv/csr.c > > +++ b/target/riscv/csr.c > > @@ -209,6 +209,16 @@ static RISCVException epmp(CPURISCVState *env, int > csrno) > > > > return RISCV_EXCP_ILLEGAL_INST; > > } > > + > > +static RISCVException priv1p12(CPURISCVState *env, int csrno) > > +{ > > + if (env->priv_ver >= PRIV_VERSION_1_12_0) { > > + return RISCV_EXCP_NONE; > > + } > > + > > + return RISCV_EXCP_ILLEGAL_INST; > > +} > > + > > #endif > > > > /* User Floating-Point CSRs */ > > @@ -1569,10 +1579,11 @@ riscv_csr_operations csr_ops[CSR_TABLE_SIZE] = { > > [CSR_MINSTRETH] = { "minstreth", any32, read_instreth }, > > > > /* Machine Information Registers */ > > - [CSR_MVENDORID] = { "mvendorid", any, read_zero }, > > - [CSR_MARCHID] = { "marchid", any, read_zero }, > > - [CSR_MIMPID] = { "mimpid", any, read_zero }, > > - [CSR_MHARTID] = { "mhartid", any, read_mhartid }, > > + [CSR_MVENDORID] = { "mvendorid", any, read_zero }, > > + [CSR_MARCHID] = { "marchid", any, read_zero }, > > + [CSR_MIMPID] = { "mimpid", any, read_zero }, > > + [CSR_MHARTID] = { "mhartid", any, read_mhartid }, > > Why change these? > The alignment of all structure entries is consistent in their respective blocks, that's why I aligned these with the mconfigptr line. It's really not necessary and if my observation on the alignment is really not a requirement I will undo this. > > > > > + [CSR_MCONFIGPTR] = {"mconfigptr", priv1p12, read_zero }, > > This looks fine, but there are more changes then this in v1.12. > Looking at the preface we need mret/sret changes at least. It also > looks like some other changes will need to be implemented or at least > checked. > > Agree, I will look into that > Alistair > > > > > /* Machine Trap Setup */ > > [CSR_MSTATUS] = { "mstatus", any, read_mstatus, > write_mstatus }, > > -- > > 2.25.1 > > > > > [-- Attachment #2: Type: text/html, Size: 4932 bytes --]
next prev parent reply other threads:[~2021-10-28 2:34 UTC|newest] Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-10-25 12:43 [PATCH v2 0/2] mconfigptr support Rahul Pathak 2021-10-25 12:43 ` [PATCH v2 1/2] target/riscv: Add priv spec 1.12.0 version check Rahul Pathak 2021-10-27 2:38 ` Alistair Francis 2021-10-27 2:38 ` Alistair Francis 2021-10-28 2:32 ` Rahul Pathak 2021-10-28 2:32 ` Rahul Pathak 2021-10-25 12:43 ` [PATCH v2 2/2] target/riscv: csr: Implement mconfigptr CSR Rahul Pathak 2021-10-27 2:42 ` Alistair Francis 2021-10-27 2:42 ` Alistair Francis 2021-10-28 2:31 ` Rahul Pathak [this message] 2021-10-28 2:31 ` Rahul Pathak 2021-10-27 2:44 ` [PATCH v2 0/2] mconfigptr support Alistair Francis 2021-10-27 2:44 ` Alistair Francis 2021-10-28 2:34 ` Rahul Pathak 2021-10-28 2:34 ` Rahul Pathak
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to='CA+Oz1=Ya_j-d1eMJ+jrw=TNWd6EOM613vONR9UU=9ny5MGg=Hg@mail.gmail.com' \ --to=rpathak@ventanamicro.com \ --cc=Alistair.Francis@wdc.com \ --cc=alistair23@gmail.com \ --cc=bmeng.cn@gmail.com \ --cc=qemu-devel@nongnu.org \ --cc=qemu-riscv@nongnu.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.