stand/efi: Use EFI_PAGE_SIZE here, it's more appropriate
ClosedPublic
Actions

Authored by imp on Wed, May 28, 9:48 PM.

Details

Reviewers

chs
kevans
jhibbits
tsoome
manu

Commits

rG700a2c08dcea: stand/efi: Use EFI_PAGE_SIZE here, it's more appropriate

Summary

No functional change: We're allocating a page from EFI, which is always
4k on amd64. So, this should always be 4k. PAGE_SIZE usually is this,
but might not be in some cases. For the end of the stack pointer in the
ist1, it should point the architecture's phyiscal page past where we've
allocated. EFI_PAGE_SIZE more faithfully reflects that here since
PAGE_SIZE might not be exactly that in the future (if we had a larger
logical page than physical page). Since the AllocPage interface is in
EFI_PAGE_SIZE pages always, this seems safer. No functional change since
they are both 2096 today.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 64533
Build 61417: arc lint + arc unit

Event Timeline

imp created this revision.Wed, May 28, 9:48 PM

Herald added a reviewer: manu. · View Herald TranscriptWed, May 28, 9:49 PM

Herald added subscribers: dab, bcran. · View Herald Transcript

imp requested review of this revision.Wed, May 28, 9:49 PM

Harbormaster completed remote builds in B64515: Diff 156231.Wed, May 28, 9:49 PM

imp added a child revision: D50585: stand: Introduce md_align.Wed, May 28, 9:49 PM

tsoome accepted this revision.Wed, May 28, 9:53 PM

This revision is now accepted and ready to land.Wed, May 28, 9:53 PM

The code change is fine, but the commit comment has several typos (appropraite vs. appropriate, phyiscal vs. physical, 2096 vs. 4096).

imp retitled this revision from stand/efi: Use EFI_PAGE_SIZE here, it's more appropraite to stand/efi: Use EFI_PAGE_SIZE here, it's more appropriate.Thu, May 29, 6:04 PM

Update per review

This revision now requires review to proceed.Thu, May 29, 6:17 PM

Harbormaster completed remote builds in B64533: Diff 156262.Thu, May 29, 6:17 PM

tsoome accepted this revision.Thu, May 29, 9:16 PM

This revision is now accepted and ready to land.Thu, May 29, 9:16 PM

Closed by commit rG700a2c08dcea: stand/efi: Use EFI_PAGE_SIZE here, it's more appropriate (authored by imp). · Explain WhyFri, May 30, 3:57 PM

This revision was automatically updated to reflect the committed changes.

imp added a commit: rG700a2c08dcea: stand/efi: Use EFI_PAGE_SIZE here, it's more appropriate.