Page MenuHomeFreeBSD
Differential D11301

EFI boot: made 32 bit pad more explicit on EFI_MEMORY_DESCRIPTOR
ClosedPublic
Actions

Authored by guest-svmhdvn on Jun 22 2017, 1:29 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Sep 27, 7:04 AM
Unknown Object (File)
Fri, Sep 19, 9:06 PM
Unknown Object (File)
Sep 5 2025, 5:33 AM
Unknown Object (File)
Sep 5 2025, 3:04 AM
Unknown Object (File)
Sep 5 2025, 3:04 AM
Unknown Object (File)
Aug 21 2025, 2:03 PM
Unknown Object (File)
Aug 2 2025, 12:32 PM
Unknown Object (File)
Jul 17 2025, 5:16 PM
View All 57 Files
Subscribers
andrew
manu

Details

Reviewers
emaste
Commits
rS320234: Make structure padding explicit in EFI_MEMORY_DESCRIPTOR

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

guest-svmhdvn created this revision.Jun 22 2017, 1:29 PM
emaste added a subscriber: manu.Jun 22 2017, 2:08 PM
emaste added inline comments.
sys/boot/efi/include/efidef.h
181 ↗(On Diff #29946)

I found some other instances of this file that name the field Pad and will probably do the same upon commit.

manu added a comment.Jun 22 2017, 2:12 PM

The current code matches the struct in UEFI specification 2.6 (http://www.uefi.org/sites/default/files/resources/UEFI%20Spec%202_6%20Errata%20A%20final.pdf) page 157

manu added a comment.Jun 22 2017, 2:17 PM
In D11301#234045, @manu wrote:

The current code matches the struct in UEFI specification 2.6 (http://www.uefi.org/sites/default/files/resources/UEFI%20Spec%202_6%20Errata%20A%20final.pdf) page 157

But u-boot added a u32 named reserved, not sure yet which one is wrong, I'll investigate.

andrew added a subscriber: andrew.Jun 22 2017, 2:28 PM

On 32-bit ARM there will be implicit padding as EFI_PHYSICAL_ADDRESS is a 64-bit type so will be 64-bit aligned.

Closed by commit rS320234: Make structure padding explicit in EFI_MEMORY_DESCRIPTOR (authored by emaste). · Explain WhyJun 22 2017, 2:30 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
boot/
efi/
include/
1 line

Diff 29947

View Options

head/sys/boot/efi/include/efidef.h

Loading...