Page MenuHomeFreeBSD
Differential D49998

efidev: add support for memory attribute
Needs ReviewPublic
Actions

Authored by aokblast on Thu, Apr 24, 12:42 PM.

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.Thu, Apr 24, 12:42 PM
Herald added a subscriber: imp. · View Herald TranscriptThu, Apr 24, 12:42 PM
aokblast requested review of this revision.Thu, Apr 24, 12:42 PM
Harbormaster completed remote builds in B63698: Diff 154209.Thu, Apr 24, 12:42 PM
aokblast added a reviewer: pavel.balaev_3mdeb.com.Thu, Apr 24, 12:43 PM
imp added a comment.Thu, Apr 24, 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.EditedFri, Apr 25, 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.

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...