[PATCH v2 1/4] drivers: android: binder: Update ARef imports from sync::aref

Shankari Anand posted 4 patches 1 month ago
[PATCH v2 1/4] drivers: android: binder: Update ARef imports from sync::aref
Posted by Shankari Anand 1 month ago
Update call sites in binder files to import `ARef`
from `sync::aref` instead of `types`.

This aligns with the ongoing effort to move `ARef` and
`AlwaysRefCounted` to sync.

Suggested-by: Benno Lossin <lossin@kernel.org>
Link: https://github.com/Rust-for-Linux/linux/issues/1173
Signed-off-by: Shankari Anand <shankari.ak0208@gmail.com>
Acked-by: Alice Ryhl <aliceryhl@google.com>
---
No change from v1 -> v2. Carry-forwarded the acked-by.

Link of v1: https://lore.kernel.org/all/20251123092438.182251-2-shankari.ak0208@gmail.com/

---
 drivers/android/binder/process.rs | 2 +-
 drivers/android/binder/thread.rs  | 3 +--
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/android/binder/process.rs b/drivers/android/binder/process.rs
index 132055b4790f..db25307b5a4a 100644
--- a/drivers/android/binder/process.rs
+++ b/drivers/android/binder/process.rs
@@ -28,11 +28,11 @@
     seq_print,
     sync::poll::PollTable,
     sync::{
+        aref::ARef,
         lock::{spinlock::SpinLockBackend, Guard},
         Arc, ArcBorrow, CondVar, CondVarTimeoutResult, Mutex, SpinLock, UniqueArc,
     },
     task::Task,
-    types::ARef,
     uaccess::{UserSlice, UserSliceReader},
     uapi,
     workqueue::{self, Work},
diff --git a/drivers/android/binder/thread.rs b/drivers/android/binder/thread.rs
index 1a8e6fdc0dc4..6a5c90547844 100644
--- a/drivers/android/binder/thread.rs
+++ b/drivers/android/binder/thread.rs
@@ -16,9 +16,8 @@
     seq_file::SeqFile,
     seq_print,
     sync::poll::{PollCondVar, PollTable},
-    sync::{Arc, SpinLock},
+    sync::{aref::ARef, Arc, SpinLock},
     task::Task,
-    types::ARef,
     uaccess::UserSlice,
     uapi,
 };
-- 
2.34.1
Re: [PATCH v2 1/4] drivers: android: binder: Update ARef imports from sync::aref
Posted by Shankari Anand 6 days, 17 hours ago
On Sat, Jan 03, 2026 at 01:57:11AM +0530, Shankari Anand wrote:
> Update call sites in binder files to import `ARef`
> from `sync::aref` instead of `types`.
> 
> This aligns with the ongoing effort to move `ARef` and
> `AlwaysRefCounted` to sync.
> 
> Suggested-by: Benno Lossin <lossin@kernel.org>
> Link: https://github.com/Rust-for-Linux/linux/issues/1173
> Signed-off-by: Shankari Anand <shankari.ak0208@gmail.com>
> Acked-by: Alice Ryhl <aliceryhl@google.com>
> ---
> No change from v1 -> v2. Carry-forwarded the acked-by.
> 
> Link of v1: https://lore.kernel.org/all/20251123092438.182251-2-shankari.ak0208@gmail.com/
> 
> ---
>  drivers/android/binder/process.rs | 2 +-
>  drivers/android/binder/thread.rs  | 3 +--
>  2 files changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/android/binder/process.rs b/drivers/android/binder/process.rs
> index 132055b4790f..db25307b5a4a 100644
> --- a/drivers/android/binder/process.rs
> +++ b/drivers/android/binder/process.rs
> @@ -28,11 +28,11 @@
>      seq_print,
>      sync::poll::PollTable,
>      sync::{
> +        aref::ARef,
>          lock::{spinlock::SpinLockBackend, Guard},
>          Arc, ArcBorrow, CondVar, CondVarTimeoutResult, Mutex, SpinLock, UniqueArc,
>      },
>      task::Task,
> -    types::ARef,
>      uaccess::{UserSlice, UserSliceReader},
>      uapi,
>      workqueue::{self, Work},
> diff --git a/drivers/android/binder/thread.rs b/drivers/android/binder/thread.rs
> index 1a8e6fdc0dc4..6a5c90547844 100644
> --- a/drivers/android/binder/thread.rs
> +++ b/drivers/android/binder/thread.rs
> @@ -16,9 +16,8 @@
>      seq_file::SeqFile,
>      seq_print,
>      sync::poll::{PollCondVar, PollTable},
> -    sync::{Arc, SpinLock},
> +    sync::{aref::ARef, Arc, SpinLock},
>      task::Task,
> -    types::ARef,
>      uaccess::UserSlice,
>      uapi,
>  };
> -- 
> 2.34.1
> 

Hello,
Can this patch be picked up?

Thanks and regards,
Shankari
Re: [PATCH v2 1/4] drivers: android: binder: Update ARef imports from sync::aref
Posted by Greg Kroah-Hartman 3 days, 17 hours ago
On Sat, Jan 31, 2026 at 05:05:23PM +0530, Shankari Anand wrote:
> On Sat, Jan 03, 2026 at 01:57:11AM +0530, Shankari Anand wrote:
> > Update call sites in binder files to import `ARef`
> > from `sync::aref` instead of `types`.
> > 
> > This aligns with the ongoing effort to move `ARef` and
> > `AlwaysRefCounted` to sync.
> > 
> > Suggested-by: Benno Lossin <lossin@kernel.org>
> > Link: https://github.com/Rust-for-Linux/linux/issues/1173
> > Signed-off-by: Shankari Anand <shankari.ak0208@gmail.com>
> > Acked-by: Alice Ryhl <aliceryhl@google.com>
> > ---
> > No change from v1 -> v2. Carry-forwarded the acked-by.
> > 
> > Link of v1: https://lore.kernel.org/all/20251123092438.182251-2-shankari.ak0208@gmail.com/
> > 
> > ---
> >  drivers/android/binder/process.rs | 2 +-
> >  drivers/android/binder/thread.rs  | 3 +--
> >  2 files changed, 2 insertions(+), 3 deletions(-)
> > 
> > diff --git a/drivers/android/binder/process.rs b/drivers/android/binder/process.rs
> > index 132055b4790f..db25307b5a4a 100644
> > --- a/drivers/android/binder/process.rs
> > +++ b/drivers/android/binder/process.rs
> > @@ -28,11 +28,11 @@
> >      seq_print,
> >      sync::poll::PollTable,
> >      sync::{
> > +        aref::ARef,
> >          lock::{spinlock::SpinLockBackend, Guard},
> >          Arc, ArcBorrow, CondVar, CondVarTimeoutResult, Mutex, SpinLock, UniqueArc,
> >      },
> >      task::Task,
> > -    types::ARef,
> >      uaccess::{UserSlice, UserSliceReader},
> >      uapi,
> >      workqueue::{self, Work},
> > diff --git a/drivers/android/binder/thread.rs b/drivers/android/binder/thread.rs
> > index 1a8e6fdc0dc4..6a5c90547844 100644
> > --- a/drivers/android/binder/thread.rs
> > +++ b/drivers/android/binder/thread.rs
> > @@ -16,9 +16,8 @@
> >      seq_file::SeqFile,
> >      seq_print,
> >      sync::poll::{PollCondVar, PollTable},
> > -    sync::{Arc, SpinLock},
> > +    sync::{aref::ARef, Arc, SpinLock},
> >      task::Task,
> > -    types::ARef,
> >      uaccess::UserSlice,
> >      uapi,
> >  };
> > -- 
> > 2.34.1
> > 
> 
> Hello,
> Can this patch be picked up?

Sorry, trying to pick patches out of the middle of a series is "hard",
next time you do this, please do not make a series if you wish for
different maintainers to be taking them.

I'll go take this now, thanks.

greg k-h