[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <5ssx3vakl3enjeaygpxfqtb26ufehwfjvcwqfna4xbhc645xpj@lzuvaf3my3f6>
Date: Tue, 19 Aug 2025 09:36:10 -0400
From: "Liam R. Howlett" <Liam.Howlett@...cle.com>
To: Danilo Krummrich <dakr@...nel.org>
Cc: Alice Ryhl <aliceryhl@...gle.com>,
Andrew Morton <akpm@...ux-foundation.org>,
Lorenzo Stoakes <lorenzo.stoakes@...cle.com>,
Miguel Ojeda <ojeda@...nel.org>,
Andrew Ballance <andrewjballance@...il.com>,
Boqun Feng <boqun.feng@...il.com>, Gary Guo <gary@...yguo.net>,
Björn Roy Baron <bjorn3_gh@...tonmail.com>,
Benno Lossin <lossin@...nel.org>,
Andreas Hindborg <a.hindborg@...nel.org>,
Trevor Gross <tmgross@...ch.edu>, linux-kernel@...r.kernel.org,
maple-tree@...ts.infradead.org, rust-for-linux@...r.kernel.org,
linux-mm@...ck.org
Subject: Re: [PATCH v2 5/5] rust: maple_tree: add MAINTAINERS entry
* Danilo Krummrich <dakr@...nel.org> [250819 07:49]:
> On Tue Aug 19, 2025 at 12:34 PM CEST, Alice Ryhl wrote:
> > Similar to and just below the existing MAPLE TREE entry.
> >
> > Signed-off-by: Alice Ryhl <aliceryhl@...gle.com>
> > ---
> > Liam: I'm not sure what you prefer for the MAINTAINERS entry, so let me
> > know if you want anything changed. There are also a few other options,
> > for example, I could just add the files under the existing MAPLE TREE
> > entry? The get_maintainers script should still send any relevant patches
> > my way because they also match the RUST entry that has a wildcard on the
> > rust/ directory.
>
> From v1 [1]:
>
> >> We should have another section for the maple tree, since it's not just
> >> used for mm. Your stated plan is to use it for GPU allocations and the
> >> code doesn't live in mm/, wdyt?
>
> > Sure, I can add a new section if you prefer that.
>
> Maple tree is already used outside of mm, e.g. for regmap stuff and I also use
> it in nouveau. Hence, I read this as moving maple tree to e.g. lib/ adjusting
> the existing entry.
>
> I personally think that - of course unless the affected people prefer otherwise
> - it is usually best to keep a single maintainers entry for the C and the Rust
> code. Maybe Alice can simply be added to the existing maintainers entry?
>
> What do you think?
I'm not sure what you mean by lib/ since the lib files are spread into
other entries by the looks of it?
I'm okay with the entry below or adjusting the existing one.
>
> [1] https://lore.kernel.org/all/aJW3L3SEVUl_AVvN@google.com/
>
> > Which tree does maple tree patches usually land through?
> >
> > Andrew Ballance: You mentioned being interested in being listed here as
> > well?
> > ---
> > MAINTAINERS | 10 ++++++++++
> > 1 file changed, 10 insertions(+)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 26053163fe5aed2fc4b4e39d47062c93b873ac13..4a52d884e36eafe1c819227207123c51caf02ee5 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -14674,6 +14674,16 @@ F: lib/test_maple_tree.c
> > F: tools/testing/radix-tree/maple.c
> > F: tools/testing/shared/linux/maple_tree.h
> >
> > +MAPLE TREE [RUST]
> > +M: Liam R. Howlett <Liam.Howlett@...cle.com>
> > +M: Alice Ryhl <aliceryhl@...gle.com>
> > +L: maple-tree@...ts.infradead.org
> > +L: linux-mm@...ck.org
> > +L: rust-for-linux@...r.kernel.org
> > +S: Supported
> > +F: rust/helpers/maple_tree.c
> > +F: rust/kernel/maple_tree.rs
> > +
> > MARDUK (CREATOR CI40) DEVICE TREE SUPPORT
> > M: Rahul Bedarkar <rahulbedarkar89@...il.com>
> > L: linux-mips@...r.kernel.org
> >
> > --
> > 2.51.0.rc1.167.g924127e9c0-goog
>
>
Powered by blists - more mailing lists