Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Replace pthread RwLock with custom implementation #110211

Merged
merged 12 commits into from
Feb 12, 2024
Merged

Replace pthread RwLock with custom implementation #110211

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift click to select a range
934eb8b
std: replace pthread `RwLock` with custom implementation inspired by …
joboet Apr 11, 2023
2e652e5
adjust code documentation
joboet Apr 12, 2023
280cbc5
use braces to make operator precedence less ambiguous
joboet Apr 12, 2023
709ccf9
avoid unnecessary `Thread` handle allocation
joboet Apr 12, 2023
61ce691
immediately register writer node if threads are queued
joboet Apr 18, 2023
8db64b5
use exponential backoff in `lock_contended`
joboet Apr 18, 2023
16aae04
queue_rwlock: use a separate `QUEUE_LOCKED` bit to synchronize waiter…
joboet Apr 18, 2023
1fd9f78
inline some single-use functions, add documentation
joboet Apr 22, 2023
69f55de
format using latest rustfmt
joboet Nov 30, 2023
3fa5a40
be more explicit about why adding backlinks eagerly makes sense
joboet Feb 9, 2024
ff44ae7
address review comments
joboet Feb 9, 2024
04282db
add doc-comment to `unlock_queue`
joboet Feb 11, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
avoid unnecessary Thread handle allocation
  • Loading branch information
@joboet
joboet committed Feb 9, 2024
commit 709ccf98b8f646629150c9b1dfa536dd2b905d10
3 changes: 2 additions & 1 deletion library/std/src/sys/pal/unix/locks/queue_rwlock.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -210,7 210,8 @@ impl Node {
fn prepare(&mut self) {
// Fall back to creating an unnamed `Thread` handle to allow locking in
// TLS destructors.
self.thread.get_or_init(|| thread_info::current_thread().unwrap_or(Thread::new(None)));
self.thread
.get_or_init(|| thread_info::current_thread().unwrap_or_else(|| Thread::new(None)));
self.completed = AtomicBool::new(false);
}

Expand Down