Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use NAPOT PMP to cover the entire physical address space #154

Open
wants to merge 1 commit into
base: riscv
Choose a base branch
from

Conversation

chongfeng-hu
Copy link

The current PMP setting uses TOR regions, which does not cover the end of the memory (spec: If PMP entry 0’s A field is set to TOR, zero is used for the lower bound, and so it matches any address y < pmpaddr0).

Proposed new NAPOT PMP covers every byte of physical memory in the 56-bit physical address space.

Quote from Andrew: you need to use a NAPOT PMP if you want to grant permissions to the entire address space.

Relevant discussions:

The current PMP setting uses TOR regions, which does not cover the end of the
memory (spec: If PMP entry 0’s A field is set to TOR, zero is used for the lower
bound, and so it matches any address y < pmpaddr0).

Proposed new NAPOT PMP covers every byte of physical memory in the 56-bit
physical address space.

Quote from Andrew: you need to use a NAPOT PMP if you want to grant permissions
to the entire address space.

Relevant discussions:
* riscv/riscv-isa-manual#138
* https://groups.google.com/a/groups.riscv.org/g/sw-dev/c/pn9ZCf0O65c/m/tnLTjg74CgAJ
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.

1 participant