From: Kyle Sanderson <kyle.leet@gmail.com>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: linux-bcachefs@vger.kernel.org, linux-fsdevel@vger.kernel.org,
linux-kerenl@vger.kernel.org,
Kent Overstreet <kent.overstreet@linux.dev>
Subject: Re: [GIT PULL] bcachefs fixes for 6.16-rc4
Date: Fri, 27 Jun 2025 12:07:55 -0700 [thread overview]
Message-ID: <065f98ab-885d-4f5e-97e3-beef095b93f0@gmail.com> (raw)
In-Reply-To: <CAHk-=wi+k8E4kWR8c-nREP0+EA4D+=rz5j0Hdk3N6cWgfE03-Q@mail.gmail.com>
On 6/26/2025 8:21 PM, Linus Torvalds wrote:
> On Thu, 26 Jun 2025 at 19:23, Kent Overstreet <kent.overstreet@linux.dev> wrote:
>>
>> per the maintainer thread discussion and precedent in xfs and btrfs
>> for repair code in RCs, journal_rewind is again included
>
> I have pulled this, but also as per that discussion, I think we'll be
> parting ways in the 6.17 merge window.
>
> You made it very clear that I can't even question any bug-fixes and I
> should just pull anything and everything.
>
> Honestly, at that point, I don't really feel comfortable being
> involved at all, and the only thing we both seemed to really
> fundamentally agree on in that discussion was "we're done".
>
> Linus
Linus,
The pushback on rewind makes sense, it wasn’t fully integrated and was
fsck code written to fix the problems with the retail 6.15 release -
this looks like it slipped through Kents CI and there were indeed
multiple people hit by it (myself included).
Quoting someone back to themselves is not cool, however I believe it
highlights what has gone on here which is why I am breaking my own rule:
"One of the things I liked about the Rust side of the kernel was that
there was one maintainer who was clearly much younger than most of the
maintainers and that was the Rust maintainer.
We can clearly see that certain areas in the kernel bring in more young
people.
At the Maintainer Summit, we had this clear division between the
filesystem people, who were very careful and very staid, and cared
deeply about their code being 100% correct - because if you have a bug
in a filesystem, the data on your disk may be gone - so these people
take themselves and their code very seriously.
And then you have the driver people who are a bit more 'okay',
especially the GPU folks, 'where anything goes'.
You notice that on the driver side it’s much easier to find young
people, and that is traditionally how we’ve grown a lot of maintainers.
" (1)
Kent is moving like the older days of rapid development - fast and
driven - and this style clashes with the mature stable filesystem
culture that demands extreme caution today. Almost every single patch
has been in response to reported issues, the primary issue here is
that’s on IRC where his younger users are (not so young, anymore - it is
not tiktok), and not on lkml. The pace of development has kept up, and
the "new feature" part of it like changing out the entire hash table in
rc6 seems to have stopped. This is still experimental, and he's moving
that way now with care and continuing to improve his testing coverage
with each bug.
Kent has deep technical experience here, much earlier in the
interview(1) regarding the 6.7 merge window this filesystem has been in
the works for a decade. Maintainership means adapting to kernel process
as much as code quality, that may be closer to the issue here.
If direct pulls aren’t working, maybe a co-maintainer or routing changes
through a senior fs maintainer can help. If you're open to it, maybe
that is even you.
Dropping bcachefs now would be a monumental step backward from the
filesystems we have today. Enterprises simply do not use them for true
storage at scale which is why vendors have largely taken over this
space. The question is how to balance rigor with supporting new
maintainers in the ecosystem. Everything Kent has written around
supporting users is true, and publicly visible, if only to the 260 users
on irc, and however many more are on matrix. There are plenty more that
are offline, and while this is experimental there are a number of public
sector agencies testing this now (I have seen reference to a number of
emergency service providers, which isn’t great, but for whatever reason
they are doing that).
(1) https://youtu.be/OvuEYtkOH88?t=1044
Kyle.
next prev parent reply other threads:[~2025-06-27 19:07 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-06-27 2:22 [GIT PULL] bcachefs fixes for 6.16-rc4 Kent Overstreet
2025-06-27 3:21 ` Linus Torvalds
2025-06-27 3:34 ` Kent Overstreet
2025-07-01 14:43 ` John Stoffel
2025-07-02 16:34 ` Kent Overstreet
2025-07-02 17:41 ` Carl E. Thompson
2025-07-02 17:53 ` Kent Overstreet
2025-07-02 18:49 ` Malte Schröder
2025-07-07 20:03 ` John Stoffel
2025-07-07 20:39 ` Kent Overstreet
2025-07-07 21:35 ` Jani Partanen
2025-07-07 22:10 ` Kent Overstreet
2025-07-07 21:32 ` Carl E. Thompson
2025-07-07 22:22 ` Kent Overstreet
2025-07-04 7:02 ` Hillf Danton
2025-06-27 19:07 ` Kyle Sanderson [this message]
2025-06-27 19:16 ` Kyle Sanderson
2025-06-27 19:42 ` Kent Overstreet
2025-06-28 8:06 ` Gerhard Wiesinger
2025-06-27 3:33 ` pr-tracker-bot
2025-06-27 14:46 ` Josef Bacik
2025-06-28 1:59 ` 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=065f98ab-885d-4f5e-97e3-beef095b93f0@gmail.com \
--to=kyle.leet@gmail.com \
--cc=kent.overstreet@linux.dev \
--cc=linux-bcachefs@vger.kernel.org \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kerenl@vger.kernel.org \
--cc=torvalds@linux-foundation.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).