ref(vmpu): Standardize vMPU driver and update armv8-r MPU #216
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
miguelafsilva5
force-pushed
the
ref/vmpu
branch
5 times, most recently
from
063a1b2 to
2f167aa
Compare
Signed-off-by: Jose Martins <[email protected]>
Signed-off-by: Jose Martins <[email protected]>
This API simplifies the implementation of vMPU entries coalescense. Updating a region requires a broadcast to the other CPUs on the same address space. This commit also fixes the install of the VM images where MPU entries on other cores were not correctly unmaped. Signed-off-by: Daniel Oliveira <[email protected]> Signed-off-by: Miguel Silva <[email protected]>
Add functionality to lock entries on the vMPU to prevent hypervisor MPU entries from being removed. Signed-off-by: Daniel Oliveira <[email protected]>
This reduces the number of vMPU/MPU entries. Signed-off-by: Daniel Oliveira <[email protected]> Signed-off-by: Miguel Silva <[email protected]>
miguelafsilva5
force-pushed
the
ref/vmpu
branch
3 times, most recently
from
b5d134b to
c058b4e
Compare
|
@miguelafsilva5 Please check for typos in commit messages. Also, the sentence after : should start lowecase. |
miguelafsilva5
force-pushed
the
ref/vmpu
branch
from
c058b4e to
91ac275
Compare
To simplify boot code we run with mpu disabled until the mem init. Signed-off-by: Daniel Oliveira <[email protected]>
Initally, when mapping and copying, the destination region was being created with the same size of the original region, instead of the size of the data being copied Signed-off-by: Miguel Silva <[email protected]>
miguelafsilva5
force-pushed
the
ref/vmpu
branch
from
91ac275 to
8e8c6a1
Compare
@josecm should be fixed already. Thank you |
@miguelafsilva5 commit 2922a2d still has a typo. Should be "deallocate" |
miguelafsilva5
force-pushed
the
ref/vmpu
branch
from
8e8c6a1 to
658439b
Compare
@danielRep my bad- Missed it. Its fixed now. Thank you |
Removed the complexity from the MPU driver related to the merge of regions, which was moved to the vMPU level. Removed the creation of MPU entries during boot. Signed-off-by: Miguel Silva <[email protected]>
During the context switch of MPU regions, locked entries are not disabled. Signed-off-by: Miguel Silva <[email protected]>
Added a function that deallocates a vMPU entry whenever the mpu fails to insert a region. Signed-off-by: Miguel Silva <[email protected]>
Signed-off-by: Miguel Silva <[email protected]>
miguelafsilva5
force-pushed
the
ref/vmpu
branch
from
658439b to
44db8da
Compare
josecm
approved these changes
Aug 20, 2025
DavidMCerdeira
approved these changes
Aug 20, 2025
danielRep
requested review from
danielRep
and removed request for
AfonsoSantos96
danielRep
approved these changes
Aug 20, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR Description
This PR aims to stantardize the vMPU driver and to the some extend the MPU API. This was needed to support different MPU implementations.
A major goal of this PR is remove the complexity from the MPU driver and move it to the vMPU.
In order to comply with these modifications to the vMPU, the MPU driver for armv8-r was also updated