Skip to content

pmp: require execute permission for HLVX accesses - #3413

Open
Maanvi212006 wants to merge 4 commits into
openhwgroup:masterfrom
Maanvi212006:fix/hlvx-pmp-execute-check
Open

pmp: require execute permission for HLVX accesses#3413
Maanvi212006 wants to merge 4 commits into
openhwgroup:masterfrom
Maanvi212006:fix/hlvx-pmp-execute-check

Conversation

@Maanvi212006

@Maanvi212006 Maanvi212006 commented Jul 24, 2026

Copy link
Copy Markdown
  • I have searched for similar pull requests
  • I am a human engaging in an interpersonal interaction. During this interaction, my words are my own and are not generated. If relevant, I provide links to my sources.

Why is this PR needed for the project?

HLVX accesses must require both read and execute permissions from PMP, as specified by the RISC-V Privileged Specification. Previously, HLVX accesses were checked as normal read accesses, allowing execution through read-only PMP regions.

This change forwards the HLVX flag to the PMP data interface and performs the permission check with both ACCESS_READ and ACCESS_EXEC.

Issue

fixes #3400

Are there limitations with the current state of this contribution?

This change only updates the PMP permission check for HLVX accesses. No additional verification tests are included in this PR.

Signed-off-by: Maanvi212006 <maanvichetwani21@gmail.com>
Signed-off-by: Maanvi212006 <maanvichetwani21@gmail.com>
Signed-off-by: Maanvi212006 <maanvichetwani21@gmail.com>
@Maanvi212006

Copy link
Copy Markdown
Author

Hii @JeanRochCoulon , the Verible / format (pull_request_target) is unrelated to my PR. The PR is ready for review. Thanks!

image

@cainria

cainria commented Jul 27, 2026

Copy link
Copy Markdown
Member

Indeed a PR is open to fix Verible #3403

@cainria

cainria commented Jul 27, 2026

Copy link
Copy Markdown
Member

#3403 was merged so Verible should pass if we restart the job or on your next push.

There is another issue, see #3415 -> merged

@Maanvi212006

Copy link
Copy Markdown
Author

Hi @JeanRochCoulon a gentle ping!
Thanks

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.

[BUG] HLVX physical access is not checked for execute permission

3 participants