add tests for the SPMP extension#647
Conversation
aswaterman
left a comment
There was a problem hiding this comment.
Thanks, @ybc-alkaid. I see these are not included in isa/Makefile, presumably because they don't yet work on upstream Spike.
My preference is to add them to isa/Makefile, but wait to merge this PR until Spike can execute them.
Signed-off-by: Bicheng Yang <SuperYbc@outlook.com>
|
Hi @aswaterman, |
|
@ybc-alkaid My mistake, it looks like you did it correctly. How did you test these tests? Do you have a prototype of SPMP in Spike so that you can successfully execute |
|
@aswaterman Yes, I test it via my Spike prototype: https://github.com/ybc-alkaid/riscv-isa-sim
That requires a minor modification to the Makefile. I've updated it. |
Cool. Once the Spike stuff is merged, we can merge this PR, too. |
This PR introduces basic tests for the RISC-V S-level Physical Memory Protection (SPMP) extension.
Here is the pointer to the SPMP specification:
riscv/riscv-isa-manual#2573
Here is a pre-built PDF of SPMP:
https://github.com/riscv/riscv-spmp/blob/main/rv-spmp-spec.pdf