“no features in RCs” is a very basic rule, this was a forgone conclusion. If these features were so integral to data integrity, he should have kept bcachefs out of the kernel until they were ready.
There is in this case, and why Linus did accept the patch in the end. Previous cases less so though which is why Linus is so pissed at this one.
The reason for this new feature is to help fix data loss on users systems - which is a fine line between a bug and a new feature really. There is precedent for this type on thing in RC releases from other filesystems as well. So the issue in this instance is a lot less black and white.
Honestly, not even in this case (especially given the history). Kent has a kernel tree people can pull if they need to. If it is an emergency, point people there.
“no features in RCs” is a very basic rule, this was a forgone conclusion. If these features were so integral to data integrity, he should have kept bcachefs out of the kernel until they were ready.
There is no reason that Kent Overstreet needed to do this.
I love bcachefs but I am so angry at him for making this happen.
There is in this case, and why Linus did accept the patch in the end. Previous cases less so though which is why Linus is so pissed at this one.
The reason for this new feature is to help fix data loss on users systems - which is a fine line between a bug and a new feature really. There is precedent for this type on thing in RC releases from other filesystems as well. So the issue in this instance is a lot less black and white.
That doesn’t excuse previous behaviour though.
Honestly, not even in this case (especially given the history). Kent has a kernel tree people can pull if they need to. If it is an emergency, point people there.
Sort of the ultimate fuck around and find out.