Messages in this thread | | | Date | Mon, 24 Jul 2006 09:39:39 -0400 | From | Theodore Tso <> | Subject | Re: the " 'official' point of view" expressed by kernelnewbies.org regarding reiser4 inclusion |
| |
On Mon, Jul 24, 2006 at 01:35:34PM +0200, Olivier Galibert wrote: >Ext patches don't get reviewed much > outside of the developpers, and they go in pretty much without > discussion in any case, except when Linus blows a fuse.
Um, you're kidding, right? We certainly don't make the assumption that we can violate CodingStyle willy nilly and stuff in yacc grammers into ext3 and assume that no one will push back.
In fact we did a lot of work to make sure the patches were clean and mostly ready to be accepted to mainline even before we made the first proposal to push extents to LKML.
> I think there is something of a problem currently, tough. It is > getting too hard to get code in if you're not a maintainer for an > existing subsystem (reiser4, suspend2...), and too easy when you're a > maintainer (ext4, uswsusp...).
It's not fair to assume that the only reason why non-maintainers have a harder time getting changes is because their changes are getting more intensive review. (Although it is the case that we probably do need to get better at reviewing changes that go in via git trees.) A much more important effect is that non-maintainers aren't familiar with coding and patch submission guidelines. For example, in suspend2, Nigel first tried with patches that were too monolithic, and then his next series was too broken down such that it was too hard to review (and "git bisect" wouldn't work). And of course, there are people who assume that the rules shouldn't apply to their filesystem...
- Ted - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |