All of lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Zijlstra <peterz@infradead.org>
To: Alice Ryhl <aliceryhl@google.com>
Cc: Steven Rostedt <rostedt@goodmis.org>,
	Miguel Ojeda <miguel.ojeda.sandonis@gmail.com>,
	Masami Hiramatsu <mhiramat@kernel.org>,
	Mathieu Desnoyers <mathieu.desnoyers@efficios.com>,
	Josh Poimboeuf <jpoimboe@kernel.org>,
	Jason Baron <jbaron@akamai.com>, Ard Biesheuvel <ardb@kernel.org>,
	linux-kernel@vger.kernel.org, linux-trace-kernel@vger.kernel.org,
	rust-for-linux@vger.kernel.org
Subject: Re: [PATCH] MAINTAINERS: add Rust files to STATIC BRANCH/CALL and TRACING
Date: Tue, 27 Jan 2026 09:24:01 +0100	[thread overview]
Message-ID: <20260127082401.GS171111@noisy.programming.kicks-ass.net> (raw)
In-Reply-To: <CAH5fLggpBodWYv2EQy5sgg5eSp9=7PM9ZqXcKaTuozNKN1eGGg@mail.gmail.com>

On Mon, Jan 26, 2026 at 10:58:24PM +0100, Alice Ryhl wrote:

> > > What about the STATIC BRANCH/CALL subsystem? Should I also leave you or
> > > someone else as 'M:' there? It's unclear to me who usually picks up
> > > patches for STATIC BRANCH/CALL when they are not a dependency to a patch
> > > for somewhere else.
> >
> > I think that'd be me -- I typically do the static branch/call bits.
> 
> Ah, thanks for the clarification.
> 
> Are you ok with using the approach Steven suggested for STATIC
> BRANCH/CALL subsystem too? That is, add a [RUST] entry below the
> current one, list you and me as M:, and anyone else in the main entry
> as R:, and patches land through the same tree as where they would have
> landed if they were a C patch.
> 
> I'm open to whichever setup you prefer, but I think it'd be nice to
> get these files into MAINTAINERS somewhere.

Yeah, I suppose that'll work. That [RUST] entry seems to be the
predominant style in MAINTAINERS.

My only concern is that most of the [RUST] entries don't actually
include the F entries for the !rust part, which means that if the C bits
change the Rust people aren't notified.

So I would suggest having all F duplicated from the main entry and then
add the rust files. Or, like we did with ATOMIC, just add you as M to
the main entry, along with a few rust files.

Some day I might actually learn enough to not see it as line noise :/

See 2387fb2a9b84 ("rust: sync: Add basic atomic operation mapping framework")


  reply	other threads:[~2026-01-27  8:24 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-01-07 12:22 [PATCH] MAINTAINERS: add Rust files to STATIC BRANCH/CALL and TRACING Alice Ryhl
2026-01-11 18:36 ` Miguel Ojeda
2026-01-12 15:00   ` Steven Rostedt
2026-01-12 15:10     ` Alice Ryhl
2026-01-12 16:54       ` Steven Rostedt
2026-01-26 12:27         ` Alice Ryhl
2026-01-26 21:37           ` Peter Zijlstra
2026-01-26 21:58             ` Alice Ryhl
2026-01-27  8:24               ` Peter Zijlstra [this message]
2026-01-27  8:26                 ` Alice Ryhl
2026-01-12 15:11     ` Miguel Ojeda
2026-01-12 16:54       ` Steven Rostedt

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260127082401.GS171111@noisy.programming.kicks-ass.net \
    --to=peterz@infradead.org \
    --cc=aliceryhl@google.com \
    --cc=ardb@kernel.org \
    --cc=jbaron@akamai.com \
    --cc=jpoimboe@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-trace-kernel@vger.kernel.org \
    --cc=mathieu.desnoyers@efficios.com \
    --cc=mhiramat@kernel.org \
    --cc=miguel.ojeda.sandonis@gmail.com \
    --cc=rostedt@goodmis.org \
    --cc=rust-for-linux@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.