aboutsummaryrefslogtreecommitdiff
path: root/rust/kernel/init.rs
diff options
context:
space:
mode:
authorAlice Ryhl <[email protected]>2024-09-15 14:41:28 +0000
committerMiguel Ojeda <[email protected]>2024-09-26 17:51:49 +0200
commita8ee30f45d5d57467ddb7877ed6914d0eba0af7f (patch)
tree72f92eaa3c8669de7724bb6dc5c01f6842ce46a4 /rust/kernel/init.rs
parentece207a83e464af710d641f29e32b7a144c48e79 (diff)
rust: sync: require `T: Sync` for `LockedBy::access`
The `LockedBy::access` method only requires a shared reference to the owner, so if we have shared access to the `LockedBy` from several threads at once, then two threads could call `access` in parallel and both obtain a shared reference to the inner value. Thus, require that `T: Sync` when calling the `access` method. An alternative is to require `T: Sync` in the `impl Sync for LockedBy`. This patch does not choose that approach as it gives up the ability to use `LockedBy` with `!Sync` types, which is okay as long as you only use `access_mut`. Cc: [email protected] Fixes: 7b1f55e3a984 ("rust: sync: introduce `LockedBy`") Signed-off-by: Alice Ryhl <[email protected]> Suggested-by: Boqun Feng <[email protected]> Reviewed-by: Gary Guo <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Miguel Ojeda <[email protected]>
Diffstat (limited to 'rust/kernel/init.rs')
0 files changed, 0 insertions, 0 deletions