On Wed Mar 18, 2026 at 5:59 PM JST, Alice Ryhl wrote:
> On Tue, Mar 17, 2026 at 04:18:46PM -0400, Joel Fernandes wrote:
>> 
>> 
>> On 3/17/2026 4:17 PM, Joel Fernandes wrote:
>> > Add a new module `kernel::interop::list` for working with C's doubly
>> > circular linked lists. Provide low-level iteration over list nodes.
>> > 
>> > Typed iteration over actual items is provided with a `clist_create`
>> > macro to assist in creation of the `CList` type.
>> > 
>> > Cc: Nikola Djukic <[email protected]>
>> > Reviewed-by: Daniel Almeida <[email protected]>
>> > Reviewed-by: Alexandre Courbot <[email protected]>
>> > Acked-by: Alexandre Courbot <[email protected]>
>> > Acked-by: Gary Guo <[email protected]>
>> > Acked-by: Miguel Ojeda <[email protected]>
>> > Signed-off-by: Joel Fernandes <[email protected]>
>> > ---
>> >  MAINTAINERS                 |   8 +
>> >  rust/helpers/helpers.c      |   1 +
>> >  rust/helpers/list.c         |  17 ++
>> >  rust/kernel/interop.rs      |   9 +
>> >  rust/kernel/interop/list.rs | 342 ++++++++++++++++++++++++++++++++++++
>> >  rust/kernel/lib.rs          |   2 +
>> >  6 files changed, 379 insertions(+)
>> >  create mode 100644 rust/helpers/list.c
>> >  create mode 100644 rust/kernel/interop.rs
>> >  create mode 100644 rust/kernel/interop/list.rs
>> > 
>> > diff --git a/MAINTAINERS b/MAINTAINERS
>> > index 4bd6b538a51f..e847099efcc2 100644
>> > --- a/MAINTAINERS
>> > +++ b/MAINTAINERS
>> > @@ -23251,6 +23251,14 @@ T:        git 
>> > https://github.com/Rust-for-Linux/linux.git alloc-next
>> >  F:        rust/kernel/alloc.rs
>> >  F:        rust/kernel/alloc/
>> >  
>> > +RUST [INTEROP]
>> > +M:        Joel Fernandes <[email protected]>
>> > +M:        Alexandre Courbot <[email protected]>
>> > +L:        [email protected]
>> > +S:        Maintained
>> > +T:        git https://github.com/Rust-for-Linux/linux.git interop-next
>> > +F:        rust/kernel/interop/
>> 
>> Sorry, I forgot to add an additional F: for the rust/kernel/interop.rs file.
>> Danilo/Miguel, do you mind adding this when applying?
>
> I think you should consider a mod.rs file to avoid this. It's tiny, and
> just re-exports submodules, so I don't think the "mod.rs name in file
> view" concern is that big, and IMO having files related to interop/
> inside the directory is much better than having them outside.

Ah, so there is a rationale for using a `mod.rs` file after all. What
are the project-wide guidelines re: `foo.rs` vs `foo/mod.rs`?

Reply via email to