Page MenuHomeFreeBSD
Differential D49998

efidev: add support for memory attribute
Needs ReviewPublic
Actions

Authored by aokblast on Apr 24 2025, 12:42 PM.
Tags
None
Referenced Files
F119123485: D49998.diff
Thu, Jun 5, 1:05 PM
Unknown Object (File)
Fri, May 9, 8:27 PM
Unknown Object (File)
May 2 2025, 7:43 PM
Unknown Object (File)
May 1 2025, 10:19 AM
Unknown Object (File)
Apr 29 2025, 10:59 PM
Unknown Object (File)
Apr 29 2025, 4:27 PM
Unknown Object (File)
Apr 26 2025, 8:48 PM
Unknown Object (File)
Apr 26 2025, 11:25 AM
View All 10 Files
Subscribers
emaste
imp

Details

Reviewers
pavel.balaev_3mdeb.com
Summary

The EFI_PROPERTIES_TABLE of EFI is deprecated.
The suggested table is EFI_MEMORY_ATTRIBUTES_TABLE that has a
new header and data format

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 63698
Build 60582: arc lint + arc unit

Event Timeline

aokblast created this revision.Apr 24 2025, 12:42 PM
Herald added a subscriber: imp. · View Herald TranscriptApr 24 2025, 12:42 PM
aokblast requested review of this revision.Apr 24 2025, 12:42 PM
Harbormaster completed remote builds in B63698: Diff 154209.Apr 24 2025, 12:42 PM
aokblast added a reviewer: pavel.balaev_3mdeb.com.Apr 24 2025, 12:43 PM
imp added a comment.Apr 24 2025, 7:49 PM

Do you have docs for this new table?
And I'm uneasy about this being an enum, but I have to think about that a bit more.

aokblast added a comment.EditedApr 25 2025, 2:52 AM
In D49998#1140497, @imp wrote:

Do you have docs for this new table?
And I'm uneasy about this being an enum, but I have to think about that a bit more.

Is the documentation from UEFI forum enough for you? https://uefi.org/specs/UEFI/2.10/04_EFI_System_Table.html

When you search the EFI_PROPERTIES_TABLE, it will tell you this table is deprecated and recommend you to switch to MEMORY_ATTRIBUTE.

Both my intel ultra framework laptop and zen5 desktop don't have EFI_PROPERTIES_TABLE now.

emaste added a subscriber: emaste.Apr 29 2025, 3:03 PM

Revision Contents
Changeset List

PathSize
sys/
dev/
efidev/
50 lines
modules/
efirt/
2 lines
sys/
18 lines
usr.sbin/
efitable/
20 lines

Diff 154209

View Options

sys/dev/efidev/efirt.c

Loading...
View Options

sys/modules/efirt/Makefile

Loading...
View Options

sys/sys/efi.h

Loading...
View Options

usr.sbin/efitable/efitable.c

Loading...