-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Aarch64 mmulib #430
Aarch64 mmulib #430
Conversation
Replace the slightly overcomplicated page table management code with a simplified, recursive implementation that should be far easier to reason about. Note that, as a side effect, this extends the per-entry cache invalidation that we do on page table entries to block and page entries, whereas the previous change inadvertently only affected the creation of table entries. Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org> Reviewed-by: Leif Lindholm <leif@nuviainc.com>
As it turns out, ARMv8 also permits accesses made with the MMU and caches off to hit in the caches, so to ensure that any modifications we make before enabling the MMU are visible afterwards as well, we should invalidate page tables right after allocation like we do now on ARM, if the MMU is still disabled at that point. Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org> Reviewed-by: Leif Lindholm <leif@nuviainc.com>
This is the AARCH64 counterpart of commit 1f3b1eb, to remove a pointless check against the memory type of the allocations that the page tables happened to land in. On ArmV8, we use writeback cacheable attributes exclusively for all memory. Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org> Reviewed-by: Leif Lindholm <leif@nuviainc.com>
Some cosmetic fixups to the AArch64 MMU code: - reflow overly long lines unless it hurts legibility - add/remove whitespace according to the [de facto] coding style - use camel case for goto labels Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org> Reviewed-by: Leif Lindholm <leif@nuviainc.com>
I started a new run of the devops builds to see if mergify would catch the status. **edit. that didn't help. |
"@Mergifyio refresh |
Command |
PR can not be merged due to a PatchCheck failure. Please resolve and resubmit |
20 similar comments
PR can not be merged due to a PatchCheck failure. Please resolve and resubmit |
PR can not be merged due to a PatchCheck failure. Please resolve and resubmit |
PR can not be merged due to a PatchCheck failure. Please resolve and resubmit |
PR can not be merged due to a PatchCheck failure. Please resolve and resubmit |
PR can not be merged due to a PatchCheck failure. Please resolve and resubmit |
PR can not be merged due to a PatchCheck failure. Please resolve and resubmit |
PR can not be merged due to a PatchCheck failure. Please resolve and resubmit |
PR can not be merged due to a PatchCheck failure. Please resolve and resubmit |
PR can not be merged due to a PatchCheck failure. Please resolve and resubmit |
PR can not be merged due to a PatchCheck failure. Please resolve and resubmit |
PR can not be merged due to a PatchCheck failure. Please resolve and resubmit |
PR can not be merged due to a PatchCheck failure. Please resolve and resubmit |
PR can not be merged due to a PatchCheck failure. Please resolve and resubmit |
PR can not be merged due to a PatchCheck failure. Please resolve and resubmit |
PR can not be merged due to a PatchCheck failure. Please resolve and resubmit |
PR can not be merged due to a PatchCheck failure. Please resolve and resubmit |
PR can not be merged due to a PatchCheck failure. Please resolve and resubmit |
PR can not be merged due to a PatchCheck failure. Please resolve and resubmit |
PR can not be merged due to a PatchCheck failure. Please resolve and resubmit |
PR can not be merged due to a PatchCheck failure. Please resolve and resubmit |
The https://edk2.groups.io/g/devel/message/55695 I'm closing this PR now. I'm going to pick up Ard's patches afresh from the list, along with Leif's R-b's, and then submit a brand new PR. |
…tianocore#430) Updates the requirements on [edk2-pytool-library](https://github.com/tianocore/edk2-pytool-library) to permit the latest version. Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Michael Kubacki <michael.kubacki@microsoft.com>
No description provided.