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.
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
Add Rust std support for x86_64-unknown-uefi #100316
Add Rust std support for x86_64-unknown-uefi #100316
Changes from all commits
f872477
2be029d
ff55531
b988418
2800931
868e3e1
6769cb5
a14d62e
289c445
645647d
3e891b3
5689d37
372f6e1
65849ae
ebc5ca2
d5e78bf
368ab5a
ce43ad6
72641b8
95e8f68
aa980a6
3d9a83b
767650c
4f32221
0bb8203
e85df7a
1f52c23
d84c8e3
3bf5104
176361a
88c615c
8cd315c
4bc53e3
c257c00
7262c22
36c628f
73852ec
d610cde
3702334
c4320c1
944bf59
c83cbd2
a95933b
3df71e6
545899b
945049c
1baa38f
a9c1bb8
939e11e
67329ff
2345867
d343822
dae6b19
195633a
4f7ff72
77283ae
f89fceb
000228b
d48d07d
824f067
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using
OnceLock
here leads to cleaner code IMHO.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well, I'm not sure if using
NonNull
instead ofAtomicPtr
is correct here. It is technically possible for some other driver to change the underlying System Table and thus atomic operations seem to make more sense to me.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure I catch your meaning here?
NonNull
does not mandate that the underlying data stays constant, so it should be fine to use even if that situation arises.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was referring to the
AtomicPtr::load()
operation since there will be multiple pointers to SystemTable floating around even in Rust code simultaneously.