Re: [RFC 1/5] locking: MAINTAINERS: Start watching Rust locking primitives

From: Boqun Feng
Date: Thu Dec 12 2024 - 11:58:33 EST


On Thu, Dec 12, 2024 at 11:06:41AM +0100, Peter Zijlstra wrote:
> On Wed, Nov 27, 2024 at 09:40:18PM -0800, Boqun Feng wrote:
> > It makes sense to add Rust locking primitives under the watch of general
> > locking primitives maintainers. This will encourage more reviews and
> > find potential issues earlier. Hence add related Rust files into the
> > LOCKING PRIMITIVES entry in MAINTAINERS.
> >
> > While we are at it, change the role of myself into the maintainer of
> > LOCKDEP and RUST to reflect my responsibility for the corresponding
> > code.
> >
> > Signed-off-by: Boqun Feng <boqun.feng@xxxxxxxxx>
>
> This seems fine with me; thanks for taking that up Boqun. I'm assuming
> this whole series is aimed at the rust tree, in which case:
>
> Acked-by: Peter Zijlstra (Intel) <peterz@xxxxxxxxxxxxx>
>

Thanks! Actually I'm planning to combine this series into my upcoming PR
to tip, the PR candidate is something like:

https://git.kernel.org/pub/scm/linux/kernel/git/boqun/linux.git/log/?h=lockdep-for-tip

(I still need a few days to catch up with other locking patches, so
probably after -rc3)

Would this (targeting tip/locking/core) work for you?

Regards,
Boqun

> > ---
> > MAINTAINERS | 7 ++++++-
> > 1 file changed, 6 insertions(+), 1 deletion(-)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 443217066eb9..3a6a193fad99 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -13351,8 +13351,8 @@ LOCKING PRIMITIVES
> > M: Peter Zijlstra <peterz@xxxxxxxxxxxxx>
> > M: Ingo Molnar <mingo@xxxxxxxxxx>
> > M: Will Deacon <will@xxxxxxxxxx>
> > +M: Boqun Feng <boqun.feng@xxxxxxxxx> (LOCKDEP & RUST)
> > R: Waiman Long <longman@xxxxxxxxxx>
> > -R: Boqun Feng <boqun.feng@xxxxxxxxx> (LOCKDEP)
> > L: linux-kernel@xxxxxxxxxxxxxxx
> > S: Maintained
> > T: git git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git locking/core
> > @@ -13366,6 +13366,11 @@ F: include/linux/seqlock.h
> > F: include/linux/spinlock*.h
> > F: kernel/locking/
> > F: lib/locking*.[ch]
> > +F: rust/helpers/mutex.c
> > +F: rust/helpers/spinlock.c
> > +F: rust/kernel/sync/lock.rs
> > +F: rust/kernel/sync/lock/
> > +F: rust/kernel/sync/locked_by.rs
> > X: kernel/locking/locktorture.c
> >
> > LOGICAL DISK MANAGER SUPPORT (LDM, Windows 2000/XP/Vista Dynamic Disks)
> > --
> > 2.39.5 (Apple Git-154)
> >