[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v8 19/35] RISC-V: Allow S-mode mxr access when p
From: |
Alistair Francis |
Subject: |
Re: [Qemu-devel] [PATCH v8 19/35] RISC-V: Allow S-mode mxr access when priv ISA >= v1.10 |
Date: |
Thu, 26 Apr 2018 20:02:17 +0000 |
On Wed, Apr 25, 2018 at 5:10 PM Michael Clark <address@hidden> wrote:
> The mstatus.MXR alias in sstatus should only be writable
> by S-mode if the privileged ISA version >= v1.10. Also MXR
> was masked in sstatus CSR read but not sstatus CSR writes.
> Now we correctly mask sstatus.mxr in both read and write.
> Cc: Sagar Karandikar <address@hidden>
> Cc: Bastian Koppelmann <address@hidden>
> Cc: Palmer Dabbelt <address@hidden>
> Cc: Alistair Francis <address@hidden>
> Signed-off-by: Michael Clark <address@hidden>
Reviewed-by: Alistair Francis <address@hidden>
Alistair
> ---
> target/riscv/op_helper.c | 7 +++++--
> 1 file changed, 5 insertions(+), 2 deletions(-)
> diff --git a/target/riscv/op_helper.c b/target/riscv/op_helper.c
> index b81b9b6..88c263a 100644
> --- a/target/riscv/op_helper.c
> +++ b/target/riscv/op_helper.c
> @@ -236,7 +236,10 @@ void csr_write_helper(CPURISCVState *env,
target_ulong val_to_write,
> target_ulong ms = env->mstatus;
> target_ulong mask = SSTATUS_SIE | SSTATUS_SPIE | SSTATUS_UIE
> | SSTATUS_UPIE | SSTATUS_SPP | SSTATUS_FS | SSTATUS_XS
> - | SSTATUS_SUM | SSTATUS_MXR | SSTATUS_SD;
> + | SSTATUS_SUM | SSTATUS_SD;
> + if (env->priv_ver >= PRIV_VERSION_1_10_0) {
> + mask |= SSTATUS_MXR;
> + }
> ms = (ms & ~mask) | (val_to_write & mask);
> csr_write_helper(env, ms, CSR_MSTATUS);
> break;
> @@ -439,7 +442,7 @@ target_ulong csr_read_helper(CPURISCVState *env,
target_ulong csrno)
> case CSR_SSTATUS: {
> target_ulong mask = SSTATUS_SIE | SSTATUS_SPIE | SSTATUS_UIE
> | SSTATUS_UPIE | SSTATUS_SPP | SSTATUS_FS | SSTATUS_XS
> - | SSTATUS_SUM | SSTATUS_SD;
> + | SSTATUS_SUM | SSTATUS_SD;
> if (env->priv_ver >= PRIV_VERSION_1_10_0) {
> mask |= SSTATUS_MXR;
> }
> --
> 2.7.0
- [Qemu-devel] [PATCH v8 14/35] RISC-V: Update E order and I extension order, (continued)
- [Qemu-devel] [PATCH v8 14/35] RISC-V: Update E order and I extension order, Michael Clark, 2018/04/25
- [Qemu-devel] [PATCH v8 15/35] RISC-V: Hardwire satp to 0 for no-mmu case, Michael Clark, 2018/04/25
- [Qemu-devel] [PATCH v8 16/35] RISC-V: Make mtvec/stvec ignore vectored traps, Michael Clark, 2018/04/25
- [Qemu-devel] [PATCH v8 17/35] RISC-V: No traps on writes to misa, minstret, mcycle, Michael Clark, 2018/04/25
- [Qemu-devel] [PATCH v8 18/35] RISC-V: Clear mtval/stval on exceptions without info, Michael Clark, 2018/04/25
- [Qemu-devel] [PATCH v8 19/35] RISC-V: Allow S-mode mxr access when priv ISA >= v1.10, Michael Clark, 2018/04/25
- Re: [Qemu-devel] [PATCH v8 19/35] RISC-V: Allow S-mode mxr access when priv ISA >= v1.10,
Alistair Francis <=
- [Qemu-devel] [PATCH v8 20/35] RISC-V: Use [ms]counteren CSRs when priv ISA >= v1.10, Michael Clark, 2018/04/25
- [Qemu-devel] [PATCH v8 21/35] RISC-V: Add mcycle/minstret support for -icount auto, Michael Clark, 2018/04/25
- [Qemu-devel] [PATCH v8 22/35] RISC-V: Use atomic_cmpxchg to update PLIC bitmaps, Michael Clark, 2018/04/25
- [Qemu-devel] [PATCH v8 23/35] RISC-V: Simplify riscv_cpu_local_irqs_pending, Michael Clark, 2018/04/25
- [Qemu-devel] [PATCH v8 24/35] RISC-V: Allow setting and clearing multiple irqs, Michael Clark, 2018/04/25
- [Qemu-devel] [PATCH v8 25/35] RISC-V: Move non-ops from op_helper to cpu_helper, Michael Clark, 2018/04/25