riscv-isa-sim
riscv-isa-sim copied to clipboard
add hlvx pmp protect to fix issue 1557
for fix issue #1557, add hlvx check x check for pmp
i fix it,now i think it's r && x
use a space before , now and rebase into one fix
const bool normal_rwx = ((typer && cfgr) && (!hlvx || cfgx)) || (typew && cfgw) || (typex && cfgx); I think hlvx is typer. if hlvx is low , the logic is same with old, and if hlvx is high, it need cfgx is high . if PMP not have the X ,the cfgx ||!hlvx is 0 and the normal_rwx will be 0 and go to access fault. and i run a small case which only have R to test it .
Can you squash those commits into one commit? Then I think this is ready to go.
Can you squash those commits into one commit? Then I think this is ready to go.
i do it