On Thu, Mar 17, 2011 at 09:41:34AM -0500, Steve French wrote:
> On Thu, Mar 17, 2011 at 9:35 AM, Christoph Hellwig <[email protected]> wrote:
> > Btw, what branch are these commits for? ?I dearly hope you're not trying
> > to push half-assed code to mainline. ?Please do your development on a
> > branch first, and once there is a useable implementation it can be
> > reviewed and synced over. ?Take a look at pnfs development for example.
> 
> We have an implementation that was in the test tree for a year, went
> through 3 test events that is being ported.  I don't think it will
> take more than a week or so to get enough of it in.  Unless it gets
> upstream, it is not going to get much wider review than from the
> original group (Pavel, Jeremy etc).  Note that all code is marked as
> "broken" and the majority in distinct c files - this is not that
> different than pNFS, which only recently got usable in mainline but
> was checked in many releases back.

Sorry, but that's bullshit.  Please stop merging that crap in it's
current state, and prepare a full patch series for review.  Please
include some broader list like linux-fsdevel when posting the full
series.

--
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to