Conversation

Thorsten Leemhuis (acct. 1/4)

Linus is fed up with the pull requests from Kent:

"'The bcachefs patches have become these kinds of "lots of development during the release cycles rather than before it", to the point where I'm starting to regret merging bcachefs.

If bcachefs can't work sanely within the normal upstream kernel release schedule, maybe it shouldn't *be* in the normal upstream kernel.

This is getting beyond ridiculous.

Linus'"

https://lore.kernel.org/all/CAHk-%3Dwj1Oo9-g-yuwWuHQZU8v%3DVAsBceWCRLhWxy7_-QnSa1Ng@mail.gmail.com/

7
4
1

@kernellogger That whole subthread is basically Kent Overstreet throwing a temper tantrum together with a megalomaniac episode. What the heck is wrong with waiting for the next merge cycle to do things according to process? At this point, I'd be avoiding using any of bcachefs until another person who is not Kent takes over the management of that subproject. Carl says it well https://lore.kernel.org/all/1816164937.417.1724473375169@mail.carlthompson.net/

2
2
2

@kernellogger The first reply underlines Linus' concerns; completely missing the point and having the wrong attitude. Not the way to work towards a healthy relationship & following proper process like anybody else:

"But I've got to get this done, and right now that does mean moving fast
and grinding through a lot of issues.”

0
1
2

@kernellogger Reminds me of https://lore.kernel.org/all/8734t0vntp.fsf@intel.com/

I'm all for doing cleanups and stuff, but it's not cool to merge patches behind the maintainers' backs through your own trees. It's not free for all, never has been, no matter how trivial the patches.

2
0
2

@kernellogger Oh funny, the same guy who did not understand why people might need Debian stable.

1
0
0

@uecker

well, not totally sure, but that might be unfair, as Kent claimed he was taking about _unstable_ (I did not look that closely, but I thought I mention it)

https://lore.kernel.org/all/qri2e26cmkjmffya27mx5g2yuasswlbw4opnzokpybnvt2t5et@wvjj3o3lj4qc/

0
0
1

@jani ohh, yeah, I remember.

Corporating in a big project and sticking to its rules sometimes is annoying and time-consuming, but in the end it really is worth it.

Not to mention that Kent here violated something that in a ideal world (e.g. not the one we live in) might not even need a written rule, as it should be obvious to anyone…

0
0
2

@jlin @kernellogger yeah I'm sitting on some locking fixes because bcachefs has some very funny ideas how console_lock and printk work. but I don't want to deal with this mess, so I guess as long as syzcaller has bcachefs enabled we'll just get a bunch of lockdep splats all over in other places

2
0
2

@sima @jlin @kernellogger Wait what? Why would bcachefs have anything to do with console lock?

1
0
2
@jani @kernellogger the pull request should have been declined on that basis.

Met him at a conference, he walked off immediately because I wasn't important enough (I was saying hi for meme value so didn't bother me). But I think that speaks volumes...
0
0
5
@kernellogger Kent's interactions reminds me of another file system creator from long ago, whose name I will not mention.
1
0
6

@sima @jlin @kernellogger talking about lockdep and console lock and printk: it would be neat if there was a nice way to, while the system is booting, inform lockdep about lock nestings that are intentional but rarely observed, so that you can immediately get a lockdep warning when the intended locking order is violated...

1
0
0
@jann @sima @jlin @kernellogger could you just take those locks once and unlock again in some init function?
1
0
4

@rostedt @kernellogger I had the same thought but did not dare go there.

0
0
1
Edited 4 months ago

@vbabka @jlin @kernellogger @jann yeah we do that all over for drm related locks. it also helps to make sure drivers all agree on the same lock order across platforms. we do ifdef the code out if lockdep is disabled

0
0
1

@kernellogger if it's going to be out of tree, I may as well use zfs as my filesystem again instead

1
0
0

@chris

well, we are not yet really at the point where Linus throws bcachefs out afaics.

And even if that happens, my first thought to your toot was:

Yes, I can understand that – but at the same time it's more complicated that that; for example, one of those filesystems is made for Linux and interacts nicely with the vfs and mm, while its way more complicated for the other. It might be naive, but that would scare me a little.

1
0
0

@kernellogger I don't know what I'm doing. I shouldn't be trusted to manage my own data.

1
0
0

@chris

I guess that holds true for most of us (me included) 😬 πŸ₯΄ πŸ˜„

0
0
0

2/ Kent meanwhile send a smaller merge request[1] with just fixes for 6.11-rc, which Linus just merged[2] with minor complains about one of the changes[3].

FWIW, the one Linus refused to merge had "39 files changed, 1309 insertions(+), 671 deletions(-)", the new PR just merged had "25 files changed, 387 insertions(+), 192 deletions(-)"

[1] https://lore.kernel.org/all/awwxatomvqsjf3uao64qm3b4jq34tvdfpobe22wouydgzc534j@c6h7l4k7447c/

[2] https://git.kernel.org/linus/72bea05cb1ad486b1a850f584cc93b651579ad2f

[3] https://lore.kernel.org/all/CAHk-=wh3eEa-X6wAXpVLrh+i5a3dK0atBDPUf_XhFmhs7pKwZA@mail.gmail.com/

1
0
0

@kernellogger That VFS patch has never been on any mailing list and has never been seen by Al, Jan, or myself...

2
0
1
@brauner @kernellogger that is an unreal violation of kernel process... I mean what's the point of maintainers if people can just randomly change your code in their tree :/
2
0
4

Christian Brauner 🦊🐺

Edited 3 months ago

@ljs @vbabka @kernellogger I see it's 's turn to have fun.

1
0
2

HAMMER SMASHED FILESYSTEM πŸ‡ΊπŸ‡¦

@ljs @kernellogger @brauner i thought the point of maintainers is to make us feel small in the vast universe of chaos

1
1
2
@brauner @kernellogger @lkundrak oh but I was reliably informed (by Kent) that bcachefs is in every way more stable than btrfs?
2
0
2

HAMMER SMASHED FILESYSTEM πŸ‡ΊπŸ‡¦

@ljs @kernellogger @brauner oh no what did he do to btrfs

1
0
2

@ljs @kernellogger @brauner @lkundrak oh boy. in my experience btrfs has been pretty good, I've been using it on ubuntu as the fs on top of a raid5, using it for backups of other systems, Mac Time Machine over samba, and rsnapshot for some other linux systems. Compression enabled. No problems for 3 years I've been using it.

1
0
2

@vbabka @kernellogger @brauner @lkundrak @ljs See funny, meaning 2 in https://en.wiktionary.org/wiki/funny:
Strange or unusual, often implying unpleasant.

1
0
2

@brauner @vbabka @ptesarik @lkundrak @ljs

/me looks

/me reads it a second time, as he can't believe what he just read 🧐

/me thinks – followed by a mental and a πŸ₯΄ feeling

3
0
1

HAMMER SMASHED FILESYSTEM πŸ‡ΊπŸ‡¦

Edited 3 months ago

@kernellogger @brauner @vbabka @ptesarik @ljs is this gentleman also going to the vienna conference?
i'm bringing the 🌈 bong β˜€οΈ 🚫 🧠

0
0
2
@kernellogger @brauner @ptesarik @lkundrak @ljs that's enough, forwarded to conduct@kernel.org
I feel sorry for them :(
2
0
8
@vbabka @kernellogger @brauner @lkundrak @ptesarik he has been in flagrant violation of the code of conduct for some time, he is almost who that is for...

I back your report and happy to support.

I am tired of him being involved in conferences and pretending he is part of the community too, when he keeps doing things like this and openly saying he is better than everyone else...
0
0
4