Add Sspmp, Sspmpen, and Smpmpdeleg extension definitions#1817
Add Sspmp, Sspmpen, and Smpmpdeleg extension definitions#1817ybc-alkaid wants to merge 7 commits intoriscv:mainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds Unified Database (UDB) extension definitions for the RISC-V Supervisor Physical Memory Protection (SPMP) feature set, capturing the base mechanism and two related CSRs for enable control and delegation from PMP.
Changes:
- Add a new privileged extension definition for
Sspmp(Supervisor Physical Memory Protection). - Add a new privileged extension definition for
Sspmpen(per-entry enable control viaspmpen). - Add a new privileged extension definition for
Smpmpdeleg(delegation control viampmpdeleg).
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| spec/std/isa/ext/Sspmp.yaml | Defines the core SPMP supervisor protection mechanism and its dependencies. |
| spec/std/isa/ext/Sspmpen.yaml | Defines Sspmpen and requires Sspmp as a prerequisite. |
| spec/std/isa/ext/Smpmpdeleg.yaml | Defines Smpmpdeleg (PMP-to-SPMP resource sharing) and its base privilege requirements. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Bicheng Yang <SuperYbc@outlook.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1817 +/- ##
==========================================
- Coverage 72.24% 72.21% -0.04%
==========================================
Files 52 52
Lines 27671 27740 +69
Branches 6009 6012 +3
==========================================
+ Hits 19992 20033 +41
- Misses 7679 7707 +28
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 4 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: Bicheng Yang <SuperYbc@outlook.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 4 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This PR adds Unified Database extension definitions for RISC-V S-level Physical Memory Protection (SPMP), consisting of three extensions:
Sspmp: Core SPMP mechanism for S-mode memory protection.
Sspmpen: Per-entry enable bits via the spmpen CSR.
Smpmpdeleg: PMP-to-SPMP resource sharing via the mpmpdeleg CSR.
References:
SPMP specification: riscv/riscv-isa-manual#2573
SPMP arch test suites: riscv/riscv-arch-test#1166