Skip to content

ext: spmp (1.0.0-rc2): new PMP entry field - #388

Merged
Andrew Waterman (aswaterman) merged 1 commit into
riscv:masterfrom
binno:spmp_ext
Nov 7, 2025
Merged

ext: spmp (1.0.0-rc2): new PMP entry field#388
Andrew Waterman (aswaterman) merged 1 commit into
riscv:masterfrom
binno:spmp_ext

Conversation

@binno

@binno Binno Shen (binno) commented Nov 7, 2025

Copy link
Copy Markdown
Contributor

These two fields are defined separately for PMP entry in SPMP extension.

@codecov

codecov Bot commented Nov 7, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.08%. Comparing base (383cbca) to head (04d913e).
⚠️ Report is 41 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #388      +/-   ##
==========================================
+ Coverage   96.53%   97.08%   +0.55%     
==========================================
  Files          10       14       +4     
  Lines         750      926     +176     
==========================================
+ Hits          724      899     +175     
- Misses         26       27       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@chihminchao

Copy link
Copy Markdown
Contributor

I think it is based on 1.0.0-rc2. It is better to describe it in commit log

@aswaterman

Copy link
Copy Markdown
Member

OK, ping me after the commit message is updated and I'll merge.

Signed-off-by: Binno <binno.shen@sifive.com>
@binno Binno Shen (binno) changed the title ext: spmp: new PMP entry field ext: spmp (1.0.0-rc2): new PMP entry field Nov 7, 2025
@binno

Copy link
Copy Markdown
Contributor Author

Hi Andrew Waterman (@aswaterman) ,
The commit message is updated

@aswaterman
Andrew Waterman (aswaterman) merged commit a37bbd3 into riscv:master Nov 7, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants