-
-
Notifications
You must be signed in to change notification settings - Fork 3k
std.os.uefi.tables: ziggify boot and runtime services #23441
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
linusg
merged 41 commits into
ziglang:master
from
dotcarmen:ziggify-uefi-boot-runtime-services
Jul 12, 2025
Merged
Changes from all commits
Commits
Show all changes
41 commits
Select commit
Hold shift + click to select a range
29bcdcf
std.os.uefi.tables: ziggify boot and runtime services
dotcarmen 7e1f689
avoid T{} syntax
dotcarmen 46daf84
misc fixes
dotcarmen 8f1ab0d
work
dotcarmen ddecbd1
self-review quickfixes
dotcarmen 64e9c9a
dont make MemoryMapSlice generic
dotcarmen 33650ee
more review fixes, work
dotcarmen c868ccf
more work
dotcarmen 597b6da
more work
dotcarmen 06b6c8b
review fixes
dotcarmen 136b10f
update boot/runtime services references throughout codebase
dotcarmen 8f125ad
self-review fixes
dotcarmen 2f65ded
couple of fixes i forgot to commit earlier
dotcarmen 6d03cce
fixes from integrating in my own project
dotcarmen 8e2d24c
fixes from refAllDeclsRecursive
dotcarmen 003fbc7
Apply suggestions from code review
dotcarmen 6d0e2bf
more fixes from review
dotcarmen 5fc4b18
fixes from project integration
dotcarmen 86c191f
make natural alignment of Guid align-8
dotcarmen 347c003
EventRegistration is a new opaque type
dotcarmen ccc5f3d
fix getNextHighMonotonicCount
dotcarmen 855cf17
fix locateProtocol
dotcarmen 18600c1
fix exit
dotcarmen e316b7e
partly revert 7372d65
dotcarmen e426856
oops exit data_len is num of bytes
dotcarmen 80cbffb
fixes from project integration
dotcarmen b2e35f4
MapInfo consistency, MemoryType update per review
dotcarmen 99d7a34
turn EventRegistration back into a pointer
dotcarmen 2fff399
forgot to finish updating MemoryType methods
dotcarmen cefa6cd
fix IntFittingRange calls
dotcarmen 1027e0d
set uefi.Page nat alignment
dotcarmen 3f32679
Back out "set uefi.Page nat alignment"
dotcarmen 0ecda35
get rid of some error.NotFound-s
dotcarmen 9f1bd86
fix .exit call in panic
dotcarmen e13de1c
review comments, add format method
dotcarmen 6cc6434
fix resetSystem data alignment
dotcarmen 819a6c6
oops, didnt do a final refAllDeclsRecursive i guess
dotcarmen a6c60c3
review comments
dotcarmen 2517ac7
merge latest master
dotcarmen cc6f15a
writergate update MemoryType.format
dotcarmen 8676b6a
fix rename
dotcarmen File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -654,9 +654,8 @@ pub fn defaultPanic( | |
|
||
if (uefi.system_table.boot_services) |bs| { | ||
// ExitData buffer must be allocated using boot_services.allocatePool (spec: page 220) | ||
const exit_data: []u16 = uefi.raw_pool_allocator.alloc(u16, exit_msg.len + 1) catch @trap(); | ||
@memcpy(exit_data, exit_msg[0..exit_data.len]); // Includes null terminator. | ||
_ = bs.exit(uefi.handle, .aborted, exit_data.len, exit_data.ptr); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. afaict this was a bug 😬 |
||
const exit_data = uefi.raw_pool_allocator.dupeZ(u16, exit_msg) catch @trap(); | ||
bs.exit(uefi.handle, .aborted, exit_data) catch {}; | ||
} | ||
@trap(); | ||
}, | ||
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.