Firmware/opensbi
8
0 Fork 0
forked from Mirrors/opensbi
Code Pull Requests Activity

lib: sbi_trap: Fix hstatus.SPVP update in sbi_trap_redirect()

Browse Source 
When redirecting from VS/VU-mode to HS-mode, hstatus.SPVP was set
to the value of mstatus.SPP, as according to the specification both
flags should be set to the same value.
However, the assignment of SPVP takes place before SPP itself is
updated, which results in SPVP having an outdated value.

Signed-off-by: Georg Kotheimer <georg.kotheimer@tu-dresden.de>
Reviewed-by: Anup Patel <anup.patel@wdc.com>
This commit is contained in:
Georg Kotheimer authored and Anup Patel committed 2020-08-17 18:17:42 +05:30
1 parent 4b18a2acc2
commit 9d56961b23
1 file changed
+1 -1
Download Patch File Download Diff File Expand all files Collapse all files
lib/sbi/sbi_trap.c
+1 -1
View File
@@ -123,7 +123,7 @@ int sbi_trap_redirect(struct sbi_trap_regs *regs,
/* Update HSTATUS SPVP and SPV bits */
hstatus = csr_read(CSR_HSTATUS);
hstatus &= ~HSTATUS_SPVP;
hstatus |= (regs->mstatus & MSTATUS_SPP) ? HSTATUS_SPVP : 0;
hstatus |= (prev_mode == PRV_S) ? HSTATUS_SPVP : 0;
hstatus &= ~HSTATUS_SPV;
hstatus |= (prev_virt) ? HSTATUS_SPV : 0;
csr_write(CSR_HSTATUS, hstatus);