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
Move thread parking to sys::sync
#124159
base: master
Are you sure you want to change the base?
Move thread parking to sys::sync
#124159
Conversation
f4b0d51
to
6818915
Compare
// Only used on NetBSD. If other platforms start using id-based parking, use | ||
// separate modules for each platform. | ||
#![cfg(target_os = "netbsd")] |
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.
Hmm... I'm not sure I love this being unix/thread_parking.rs
with nothing in the path to indicate it's highly platform specific. If more unix targets do use id-based parking, what should they call their modules? How would that be organised?
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.
Then I'd split them unix/thread_parking/netbsd.rs
/unix/thread_parking/*.rs
and expose the relevant inferface in unix/thread_parking/mod.rs
.
☔ The latest upstream changes (presumably #124491) made this pull request unmergeable. Please resolve the merge conflicts. |
6818915
to
a56fd37
Compare
Part of #117276.
I'll leave the platform-specific API abstractions in
sys::pal
, as per the initial proposal. I'm not entirely sure whether we'll want to keep it that way, but that remains to be seen.r? @ChrisDenton (if you have time)