diff options
| author | Greentime Hu <[email protected]> | 2023-09-06 09:01:14 +0000 |
|---|---|---|
| committer | Anup Patel <[email protected]> | 2023-09-22 11:17:26 +0530 |
| commit | 130e65dd9d44c11ef3a3048eb9a491d33e5f14b8 (patch) | |
| tree | 72ba3ea9399ee53c941ff2f95a4dc8695af10543 | |
| parent | 5bd969477f2c710ee5945f1a66cf534c478c48de (diff) | |
lib: sbi: Implement SET_FS_DIRTY() to make sure the mstatus FS dirty is set
We found the mstatus.FS status is not set correctly after the SET_F64_REG()
and SET_F32_REG(). We should set mstatus.FS dirty after we emulate the FPU
instructions.
Co-developed-by: Roy Lin <[email protected]>
Signed-off-by: Roy Lin <[email protected]>
Signed-off-by: Greentime Hu <[email protected]>
Reviewed-by: Anup Patel <[email protected]>
| -rw-r--r-- | include/sbi/riscv_fp.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/include/sbi/riscv_fp.h b/include/sbi/riscv_fp.h index b3dc08b9..3141c1c5 100644 --- a/include/sbi/riscv_fp.h +++ b/include/sbi/riscv_fp.h @@ -84,7 +84,7 @@ #define GET_FFLAGS() csr_read(CSR_FFLAGS) #define SET_FFLAGS(value) csr_write(CSR_FFLAGS, (value)) -#define SET_FS_DIRTY() ((void)0) +#define SET_FS_DIRTY(regs) (regs->mstatus |= MSTATUS_FS) #define GET_F32_RS1(insn, regs) (GET_F32_REG(insn, 15, regs)) #define GET_F32_RS2(insn, regs) (GET_F32_REG(insn, 20, regs)) @@ -93,9 +93,9 @@ #define GET_F64_RS2(insn, regs) (GET_F64_REG(insn, 20, regs)) #define GET_F64_RS3(insn, regs) (GET_F64_REG(insn, 27, regs)) #define SET_F32_RD(insn, regs, val) \ - (SET_F32_REG(insn, 7, regs, val), SET_FS_DIRTY()) + (SET_F32_REG(insn, 7, regs, val), SET_FS_DIRTY(regs)) #define SET_F64_RD(insn, regs, val) \ - (SET_F64_REG(insn, 7, regs, val), SET_FS_DIRTY()) + (SET_F64_REG(insn, 7, regs, val), SET_FS_DIRTY(regs)) #define GET_F32_RS2C(insn, regs) (GET_F32_REG(insn, 2, regs)) #define GET_F32_RS2S(insn, regs) (GET_F32_REG(RVC_RS2S(insn), 0, regs)) |