Re: [PATCH 3/3] rust: sync: Use safe synchronize_rcu() abstraction in poll

["Onur Özkan" <work@onurozkan.dev>]

On Fri, 19 Jun 2026 08:55:44 +0200
Philipp Stanner <phasta@mailbox.org> wrote:

> On Thu, 2026-06-18 at 21:11 +0300, Onur Özkan wrote:
> > On Thu, 18 Jun 2026 11:59:52 +0200
> > Philipp Stanner <phasta@kernel.org> wrote:
> > 
> > > We now have a safe wrapper for the foreign function synchronize_rcu().
> > > 
> > > Use it in poll.rs.
> > > 
> > > Signed-off-by: Philipp Stanner <phasta@kernel.org>
> > > ---
> > >  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.