[tip: locking/core] rust: revocable: Use safe synchronize_rcu() abstraction
"tip-bot2 for Philipp Stanner" <[email protected]>
| Newsgroups | gmane.linux.kernel |
|---|---|
| Message-ID | <178611874436.708.11445557319136729587.tip-bot2@tip-bot2> |
The following commit has been merged into the locking/core branch of tip: Commit-ID: 042278f5fa9e681420ab486d6cbf464e26667461 Gitweb: https://git.kernel.org/tip/042278f5fa9e681420ab486d6cbf464e26667461 Author: Philipp Stanner <[email protected]> AuthorDate: Wed, 24 Jun 2026 17:07:03 +02:00 Committer: Boqun Feng <[email protected]> CommitterDate: Tue, 04 Aug 2026 05:52:53 -07:00 rust: revocable: Use safe synchronize_rcu() abstraction We now have a safe wrapper for the foreign function synchronize_rcu(). Use it in revocable.rs. Signed-off-by: Philipp Stanner <[email protected]> Reviewed-by: Onur Özkan <[email protected]> Reviewed-by: Danilo Krummrich <[email protected]> Reviewed-by: Gary Guo <[email protected]> Signed-off-by: Boqun Feng <[email protected]> Link: https://patch.msgid.link/[email protected] --- rust/kernel/revocable.rs | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/rust/kernel/revocable.rs b/rust/kernel/revocable.rs index 0f4ae67..f539603 100644 --- a/rust/kernel/revocable.rs +++ b/rust/kernel/revocable.rs @@ -7,7 +7,11 @@ use pin_init::Wrapper; -use crate::{bindings, prelude::*, sync::rcu, types::Opaque}; +use crate::{ + prelude::*, + sync::rcu, + types::Opaque, // +}; use core::{ marker::PhantomData, ops::Deref, @@ -161,8 +165,7 @@ impl<T> Revocable<T> { if revoke { if SYNC { - // SAFETY: Just an FFI call, there are no further requirements. - unsafe { bindings::synchronize_rcu() }; + rcu::synchronize_rcu(); } // SAFETY: We know `self.data` is valid because only one CPU can succeed the