Skip to content

Moved PMP regions to data region.#1310

Merged
UmerShahidengr merged 84 commits intoriscv:act4from
hamza-1821:act4-work
Apr 27, 2026
Merged

Moved PMP regions to data region.#1310
UmerShahidengr merged 84 commits intoriscv:act4from
hamza-1821:act4-work

Conversation

@hamza-1821
Copy link
Copy Markdown
Contributor

Added the tests for PMPU, PMPS, PMPF, PMPZalrsc, PMPZicbo, PMPZaamo.
All the pmp regions have now been moved inside the data region, under the test case strings.
Tests from draft PR #699 have been added as well.

hamza-1821 and others added 30 commits February 24, 2026 04:40
At this point, the repo holds a number of different projects that all
require different people to review things. To make organizing this
easier, add issue templates that automatically apply labels for common
types of issues:
- CTP bugs/suggestions
- CRD bugs/suggestions
- Test framework bugs
- Test bugs
- Feature requests

---------

Signed-off-by: Jordan Carlin <jordanmcarlin@gmail.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Umer Shahid <umer.shahid@10xengineers.ai>
Co-authored-by: Umer Shahid <umer.shahid@10xengineers.ai>
Signed-off-by: Hamza <humza.ali1821@gmail.com>
Comment thread tests/priv/pmp/pmp32/PMPF/pmpf_cfg_wr.S Outdated
Comment thread tests/priv/pmp/pmp32/PMPF/pmpf_cfg_wr.S
Comment thread tests/priv/pmp/pmp32/PMPF/pmpf_cfg_wr.S Outdated
Copy link
Copy Markdown
Collaborator

@jordancarlin jordancarlin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM once CI is passing.

Copy link
Copy Markdown
Collaborator

@UmerShahidengr UmerShahidengr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thank you @hamza-1821

@UmerShahidengr UmerShahidengr added this pull request to the merge queue Apr 27, 2026
Merged via the queue into riscv:act4 with commit bc63759 Apr 27, 2026
35 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.

5 participants