Page MenuHomeFreeBSD

Fix the spelling of EFI_PAGE_SIZE
ClosedPublic

Authored by andrew on Mar 9 2022, 6:42 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, May 7, 1:40 AM
Unknown Object (File)
Thu, May 7, 12:45 AM
Unknown Object (File)
Tue, May 5, 7:55 PM
Unknown Object (File)
Sun, May 3, 10:32 PM
Unknown Object (File)
Sun, Apr 19, 2:46 PM
Unknown Object (File)
Apr 16 2026, 1:24 PM
Unknown Object (File)
Apr 10 2026, 5:07 PM
Unknown Object (File)
Apr 10 2026, 7:47 AM
Subscribers

Details

Summary

We assume EFI_PAGE_SIZE is the same as PAGE_SIZE, however this may not
be the case. Use the former when working with a list of pages from the
UEFI firmware so the correct size is used.

Sponsored by: The FreeBSD Foundation

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

andrew requested review of this revision.Mar 9 2022, 6:42 PM

These look good... but where's EFI_PAGE_SIZE defined? And does it need to be a run-time thing?
It's a good fix, but I'd like to understand the larger context :)

This revision is now accepted and ready to land.Mar 9 2022, 7:20 PM
In D34510#781812, @imp wrote:

These look good... but where's EFI_PAGE_SIZE defined? And does it need to be a run-time thing?
It's a good fix, but I'd like to understand the larger context :)

AFAIK the spec doesn't allow any flexibility here, every arch is forced to 4k pages to be compliant (see various mmu configuration verbiages in "Calling Conventions")

In D34510#781812, @imp wrote:

These look good... but where's EFI_PAGE_SIZE defined? And does it need to be a run-time thing?
It's a good fix, but I'd like to understand the larger context :)

AFAIK the spec doesn't allow any flexibility here, every arch is forced to 4k pages to be compliant (see various mmu configuration verbiages in "Calling Conventions")

oh! It's so *OUR* PAGE_SIZE to change.... :) Thanks for the pointers... my brain glossed over them before....

This revision was automatically updated to reflect the committed changes.