Linux EXT4 FS development
 help / color / mirror / Atom feed
From: "Theodore Tso" <tytso@mit.edu>
To: Matthias Goergens <matthias.goergens@gmail.com>
Cc: Jan Kara <jack@suse.cz>, linux-ext4@vger.kernel.org
Subject: Re: [PATCH] MAINTAINERS: name the ext4 dev branch
Date: Wed, 23 Sep 2026 23:14:31 -0400	[thread overview]
Message-ID: <arSMwd1bGO1vrif-@mit.edu> (raw)
In-Reply-To: <20260923101749.3505886-1-matthias.goergens@gmail.com>

On Wed, Sep 23, 2026 at 06:17:49PM -0500, Matthias Goergens wrote:
> The ext4 T: entry names the repository without a branch, so tools that
> default to the repository's HEAD select master, which still points at a
> merge from October 2020 (96485e446260).  Development happens on dev,
> which is also the branch linux-next merges.  Name it.

Huh.  I wasn't even aware that you *could* specify the branch name in
the T: entry.  A quick "grep ^T: MAINTAINERS" there are only a few
repos using T: that include branch.

I also tried looking at the documentation, and I can't find any
mention that the git branch could be specified.  Maybe we should
update the documentation for T: in a separate patch to the MAINTAINERS
file?  Or did I miss something?  "git grep MAINTAINERS | grep branch"
didn't turn up anything.

> The Sashiko review bot, for one, falls back to HEAD this way and has
> been reviewing ext4 patches against that tree.  Its review of an EA
> inode refcount fix reported a Critical double decrement and an
> undefined function after applying the patch to that stale baseline;
> neither holds on dev.

... and I didn't know that Sashiko uses the branch specifier.  I
wonder if ext4 is the only tree which might have this issue...

   git log --merges linux-next

... nope, apparently not.  I see a lot of branches which are merged
into linux-net which are not mentioned in the MAINTAINERS file.

Thanks for pointing this out for the ext4 MAINTAINERS entry!

						- Ted

  parent reply	other threads:[~2026-09-24  3:15 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-23 10:17 [PATCH] MAINTAINERS: name the ext4 dev branch Matthias Goergens
2026-09-23 10:44 ` Jan Kara
2026-09-24  3:14 ` Theodore Tso [this message]
2026-09-24  3:32   ` Matthias Goergens
2026-09-24  4:00     ` Eric Biggers
2026-09-24  8:30       ` Matthias Goergens
2026-09-24  8:53       ` Jan Kara
2026-09-25 13:06       ` Niklas Cassel
2026-09-25 18:43         ` Eric Biggers
2026-09-26  6:11         ` Matthias Goergens
2026-09-26 15:57           ` Niklas Cassel
2026-10-01 11:25     ` Matthias Goergens
2026-10-08 15:32 ` Theodore Ts'o

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=arSMwd1bGO1vrif-@mit.edu \
    --to=tytso@mit.edu \
    --cc=jack@suse.cz \
    --cc=linux-ext4@vger.kernel.org \
    --cc=matthias.goergens@gmail.com \
    /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