Re: [PATCH 3/3] rust: sync: Use safe synchronize_rcu() abstraction in poll
Onur Özkan <[email protected]>
| Newsgroups | org.kernel.vger.rcu,org.kernel.vger.linux-fsdevel,org.kernel.vger.linux-kernel,org.kernel.vger.rust-for-linux |
|---|---|
| Message-ID | <[email protected]> |
On Fri, 19 Jun 2026 08:55:44 +0200 Philipp Stanner <[email protected]> wrote: > On Thu, 2026-06-18 at 21:11 +0300, Onur Özkan wrote: > > On Thu, 18 Jun 2026 11:59:52 +0200 > > Philipp Stanner <[email protected]> wrote: > > > > > We now have a safe wrapper for the foreign function synchronize_rcu(). > > > > > > Use it in poll.rs. > > > > > > Signed-off-by: Philipp Stanner <[email protected]> > > > --- > > > rust/kernel/sync/poll.rs | 6 ++---- > > > 1 file changed, 2 insertions(+), 4 deletions(-) > > > > > > diff --git a/rust/kernel/sync/poll.rs b/rust/kernel/sync/poll.rs > > > index 0ec985d560c8..401cf4d3f35f 100644 > > > --- a/rust/kernel/sync/poll.rs > > > +++ b/rust/kernel/sync/poll.rs > > > @@ -8,7 +8,7 @@ > > > bindings, > > > fs::File, > > > prelude::*, > > > - sync::{CondVar, LockClassKey}, > > > + sync::{rcu::synchronize_rcu, CondVar, LockClassKey}, > > > > nit: Missing vertical import. Same in the other module as well. > > Was already like that when I came here ;) > > But if it helps I can format it while I'm at it.. Yeah there are still horizontal imports in certain modules and it's a perfect opportunity to fix them if we already making changes on them. - Onur > > > P.