Skip to content

ext: smpmpmpt: new field for pmp config - #386

Merged
Andrew Waterman (aswaterman) merged 1 commit into
riscv:masterfrom
chihminchao:add-smpmpmt
Nov 5, 2025
Merged

ext: smpmpmpt: new field for pmp config#386
Andrew Waterman (aswaterman) merged 1 commit into
riscv:masterfrom
chihminchao:add-smpmpmt

Conversation

@chihminchao

Copy link
Copy Markdown
Contributor

There are two reserved bits used to controll non-cacheable and io attribute for pmp regsions

ref

  1. https://riscv.atlassian.net/browse/RVS-2948
  2. https://github.com/riscv/riscv-isa-manual/blob/smpmpmt/src/smpmpmt.adoc

There are two reserved bits used to controll non-cacheable and io
attribute for pmp regsions

ref
  1. https://riscv.atlassian.net/browse/RVS-2948
  2. https://github.com/riscv/riscv-isa-manual/blob/smpmpmt/src/smpmpmt.adoc

Signed-off-by: Chih-Min Chao <chihmin.chao@sifive.com>
@codecov

codecov Bot commented Nov 5, 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 (24026cf).
⚠️ Report is 39 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #386      +/-   ##
==========================================
+ 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.

@aswaterman
Andrew Waterman (aswaterman) merged commit 2a00cd9 into riscv:master Nov 5, 2025
12 checks passed
@Stan-Cho

Copy link
Copy Markdown

The field name PMP_A seems to be a duplicate. I think it should be PMP_MT.

@aswaterman

Copy link
Copy Markdown
Member

Ah, crap, I should’ve seen that during code review. Chih-Min Chao (@chihminchao) please issue a follow-up PR to fix this.

@chihminchao

Copy link
Copy Markdown
Contributor Author

It is my fault and the fix pr is #387

@aswaterman

Copy link
Copy Markdown
Member

thank you, Chih-Min Chao (@chihminchao)!

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