Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755587AbZAVVwR (ORCPT ); Thu, 22 Jan 2009 16:52:17 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753058AbZAVVwA (ORCPT ); Thu, 22 Jan 2009 16:52:00 -0500 Received: from mx2.redhat.com ([66.187.237.31]:40438 "EHLO mx2.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753047AbZAVVv6 (ORCPT ); Thu, 22 Jan 2009 16:51:58 -0500 Date: Thu, 22 Jan 2009 16:50:41 -0500 From: Dave Jones To: Greg KH Cc: Ingo Molnar , Geert Uytterhoeven , Andrew Morton , =?iso-8859-1?Q?J=F6rn?= Engel , David Brown , Phil Oester , Kay Sievers , Phillip Lougher , Christoph Hellwig , torvalds@linux-foundation.org, linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org Subject: Re: [GIT PULL] Squashfs pull request for 2.6.29 Message-ID: <20090122215041.GA29369@redhat.com> Mail-Followup-To: Dave Jones , Greg KH , Ingo Molnar , Geert Uytterhoeven , Andrew Morton , =?iso-8859-1?Q?J=F6rn?= Engel , David Brown , Phil Oester , Kay Sievers , Phillip Lougher , Christoph Hellwig , torvalds@linux-foundation.org, linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org References: <20090109165422.GF24884@logfs.org> <20090109193738.GA9827@linode.davidb.org> <20090109211937.GA14342@logfs.org> <20090110124335.GB30744@elte.hu> <20090110165033.GA23943@logfs.org> <20090110101235.7ca24c44.akpm@linux-foundation.org> <20090110221528.GA31774@elte.hu> <20090111153920.GC7401@elte.hu> <20090111163018.GA9300@suse.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20090111163018.GA9300@suse.de> User-Agent: Mutt/1.5.18 (2008-05-17) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2504 Lines: 55 On Sun, Jan 11, 2009 at 08:30:18AM -0800, Greg KH wrote: > I was wanting to stick with drivers to start with, but I really have no > objection to adding filesystems, if they are self-contained, to the > drivers/staging/ directory. > > I looked at adding squashfs, but at the time, it touched other portions > of the kernel which wouldn't have made it a good canidate for staging. > This was later resolved, and now that it is merged, it's a moot issue :) > > So, if anyone wants to send me filesystems, I'll be glad to take them > into drivers/staging, as long as they are self-contained (novfs for > example would fit this category.) Filesystems in staging worries me. * The number of people who competently review filesystem code (and I mean real review here, not checkpatch & codingstyle crap) is significantly less than those who review drivers. I foresee stuff just lingering there for years. (Look how long fs stuff hangs around unmerged in -mm for example). * The fallout of staging is already starting to drift into distros. Within a week of Fedora shipping a kernel that had staging/ we had requests to enable drivers from it. And of course, those drivers were garbage. This is only going to increase as time goes on. * For crap drivers that a minority cares about, this isn't a big deal to tell the users "build it yourself, we don't support it when stuff breaks". (And a lot of that crap will break. NetworkManager won't work properly with some of the wireless crap in staging for example), but by continually adding to the shitpile the potential for review dramatically gets reduced, and for something as critical as a filesystem, I find this absolutely terrifying from a support perspective. I don't mean to piss on your parade, but from my viewpoint, staging is a trainwreck so far, and I'd hate to see it get worse. We've already demonstrated "look how much stuff we can merge" time and time again, but no-one ever seems to have a proposal for how we increase the amount of review code gets before it's merged. There's lowering the barrier for entry, and there's not having a barrier at all. The latter is what I'm concerned that staging/ has become. Dave -- http://www.codemonkey.org.uk -- 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/