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

From: Philipp Stanner

Date: Fri Jun 19 2026 - 02:56:31 EST


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


P.