summaryrefslogtreecommitdiffstats
path: root/init
diff options
context:
space:
mode:
authorAlice Ryhl <aliceryhl@google.com>2024-09-15 16:41:28 +0200
committerMiguel Ojeda <ojeda@kernel.org>2024-09-26 17:51:49 +0200
commita8ee30f45d5d57467ddb7877ed6914d0eba0af7f (patch)
tree72f92eaa3c8669de7724bb6dc5c01f6842ce46a4 /init
parentrust: kernel: sort Rust modules (diff)
downloadlinux-a8ee30f45d5d57467ddb7877ed6914d0eba0af7f.tar.xz
linux-a8ee30f45d5d57467ddb7877ed6914d0eba0af7f.zip
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: stable@vger.kernel.org Fixes: 7b1f55e3a984 ("rust: sync: introduce `LockedBy`") Signed-off-by: Alice Ryhl <aliceryhl@google.com> Suggested-by: Boqun Feng <boqun.feng@gmail.com> Reviewed-by: Gary Guo <gary@garyguo.net> Link: https://lore.kernel.org/r/20240915-locked-by-sync-fix-v2-1-1a8d89710392@google.com Signed-off-by: Miguel Ojeda <ojeda@kernel.org>
Diffstat (limited to 'init')
0 files changed, 0 insertions, 0 deletions