linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: James Bottomley <James.Bottomley@HansenPartnership.com>
To: Steven Rostedt <rostedt@goodmis.org>
Cc: ksummit@lists.linux.dev, linux-fsdevel@vger.kernel.org
Subject: Re: [MAINTAINER SUMMIT] Adding more formality around feature inclusion and ejection
Date: Fri, 22 Aug 2025 08:59:31 +0100	[thread overview]
Message-ID: <fa6de8282a80306d1ecfb9513a63aa97f6d406eb.camel@HansenPartnership.com> (raw)
In-Reply-To: <20250821151512.6b42336b@gandalf.local.home>

On Thu, 2025-08-21 at 15:15 -0400, Steven Rostedt wrote:
> On Thu, 21 Aug 2025 18:44:07 +0100
> James Bottomley <James.Bottomley@HansenPartnership.com> wrote:
> 
> > > I share my scripts and explain how to do a pull request. How to
> > > use linux-next and what to and more importantly, what not to send
> > > during during the -rc releases.  
> > 
> > I'm not sure that covers it.  As I read the situation it was more
> > about how you work with others when there are things in the kernel
> > you'd like to introduce or change to support your feature.  Hence
> > it's really about working with rather than against the community.
> 
> What I'm suggesting is to have a program to help newcomers that are
> taking on a maintainer role. This program can not only teach what
> needs to be done to be a maintainer, but also vet the people that are
> coming into our ecosystem. If there's a lot of push back from the
> individual on how to interact with the community, then that
> individual can be denied becoming a maintainer.

As I said, I think this is a good idea, it just wouldn't have solved
the problems we saw.  The initial pull request has a huge thread which,
when summarized, pretty much predicted the issues that were seen. 
Although he went into MAINTAINERS with an R tag, Brian Foster did
attempt to act as a buffer, something which I don't think we've thanked
Brian for enough, but something which ultimately failed probably due to
lack of empowerment.

> > > I'm sure others have helped developers become maintainers as
> > > well. Perhaps we should get together and come up with a formal
> > > way to become a maintainer? Because honestly, it's currently done
> > > by trial and error. I think that should change.  
> > 
> > That wouldn't hurt, but that problem that I see is that some fairly
> > drastic action has been taken on what can be characterised as a
> > whim, so I think we need some formality around how and when this
> > happens.
> 
> If it was policy for Kent to work with a mentor before he could send
> patches directly to Linus, would this have uncovered the issues
> before they became as large as they had become?

Well, no, the thread I pointed to as part of this proposal pretty much
predicted what actually happened.  So the problems were known ahead of
time and didn't need to be discovered, we just needed a better
mitigation mechanism (which a supervision program could form part of).

Regards,

James


  reply	other threads:[~2025-08-22  7:59 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-08-21  8:56 [MAINTAINER SUMMIT] Adding more formality around feature inclusion and ejection James Bottomley
2025-08-21 16:27 ` Steven Rostedt
2025-08-21 17:44   ` James Bottomley
2025-08-21 19:15     ` Steven Rostedt
2025-08-22  7:59       ` James Bottomley [this message]
2025-08-21 19:32   ` Paul Moore
2025-08-22 11:39   ` Mauro Carvalho Chehab
2025-08-22 13:15     ` Matthew Wilcox
2025-08-21 20:34 ` Theodore Ts'o
2025-08-21 20:59   ` Theodore Ts'o
2025-08-21 22:21   ` Matthew Wilcox
2025-08-22 11:29     ` Mauro Carvalho Chehab
2025-08-22  8:09   ` James Bottomley
2025-08-22 12:03     ` Greg KH
2025-08-22 12:08       ` Rafael J. Wysocki
2025-08-22 12:24       ` Theodore Tso
2025-08-22 15:31         ` James Bottomley
2025-08-22 16:09           ` Theodore Tso
2025-08-25  8:20           ` Mauro Carvalho Chehab
2025-08-25  7:57         ` Mauro Carvalho Chehab
2025-08-22 15:19       ` James Bottomley
2025-08-23  1:03 ` dan.j.williams

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=fa6de8282a80306d1ecfb9513a63aa97f6d406eb.camel@HansenPartnership.com \
    --to=james.bottomley@hansenpartnership.com \
    --cc=ksummit@lists.linux.dev \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=rostedt@goodmis.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).