diff --git a/lib/sbi/sbi_illegal_insn.c b/lib/sbi/sbi_illegal_insn.c index 84c04f8..ecd3508 100644 --- a/lib/sbi/sbi_illegal_insn.c +++ b/lib/sbi/sbi_illegal_insn.c @@ -30,6 +30,7 @@ static int truly_illegal_insn(ulong insn, struct sbi_trap_regs *regs) trap.tval = insn; trap.tval2 = 0; trap.tinst = 0; + trap.gva = 0; return sbi_trap_redirect(regs, &trap); } diff --git a/lib/sbi/sbi_misaligned_ldst.c b/lib/sbi/sbi_misaligned_ldst.c index fd11798..92a2393 100644 --- a/lib/sbi/sbi_misaligned_ldst.c +++ b/lib/sbi/sbi_misaligned_ldst.c @@ -129,6 +129,7 @@ int sbi_misaligned_load_handler(ulong addr, ulong tval2, ulong tinst, uptrap.tval = addr; uptrap.tval2 = tval2; uptrap.tinst = tinst; + uptrap.gva = 0; return sbi_trap_redirect(regs, &uptrap); } @@ -244,6 +245,7 @@ int sbi_misaligned_store_handler(ulong addr, ulong tval2, ulong tinst, uptrap.tval = addr; uptrap.tval2 = tval2; uptrap.tinst = tinst; + uptrap.gva = 0; return sbi_trap_redirect(regs, &uptrap); } diff --git a/lib/sbi/sbi_trap.c b/lib/sbi/sbi_trap.c index ee3e4e9..db70c26 100644 --- a/lib/sbi/sbi_trap.c +++ b/lib/sbi/sbi_trap.c @@ -311,6 +311,20 @@ struct sbi_trap_regs *sbi_trap_handler(struct sbi_trap_regs *regs) trap.tval = mtval; trap.tval2 = mtval2; trap.tinst = mtinst; + + /* + * If the hypervisor extension is not implemented, + * mstatus[h].GVA is a WPRI field, which is guaranteed to read + * as zero. In addition, in this case we don't read mstatush and + * instead pretend it is zero, which handles privileged spec + * version < 1.12. + */ +#if __riscv_xlen == 32 + trap.gva = (regs->mstatusH & MSTATUSH_GVA) ? 1 : 0; +#else + trap.gva = (regs->mstatus & MSTATUS_GVA) ? 1 : 0; +#endif + rc = sbi_trap_redirect(regs, &trap); break; };