Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753197AbZISFPf (ORCPT ); Sat, 19 Sep 2009 01:15:35 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752079AbZISFPc (ORCPT ); Sat, 19 Sep 2009 01:15:32 -0400 Received: from sh.osrg.net ([192.16.179.4]:50556 "EHLO sh.osrg.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751848AbZISFPc (ORCPT ); Sat, 19 Sep 2009 01:15:32 -0400 Date: Sat, 19 Sep 2009 14:14:30 +0900 To: jens.axboe@oracle.com Cc: neilb@suse.de, hch@infradead.org, James.Bottomley@suse.de, lars.ellenberg@linbit.com, linux-kernel@vger.kernel.org, drbd-dev@lists.linbit.com, akpm@linux-foundation.org, bart.vanassche@gmail.com, davej@redhat.com, gregkh@suse.de, kosaki.motohiro@jp.fujitsu.com, kyle@moffetthome.net, lmb@suse.de, torvalds@linux-foundation.org, nab@linux-iscsi.org, knikanth@suse.de, philipp.reisner@linbit.com, sam@ravnborg.org Subject: Re: [GIT PULL] DRBD for 2.6.32 From: FUJITA Tomonori In-Reply-To: <20090918200803.GM23126@kernel.dk> References: <20090917161108.GA3361@infradead.org> <19122.65335.126937.476968@notabene.brown> <20090918200803.GM23126@kernel.dk> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Message-Id: <20090919141334N.fujita.tomonori@lab.ntt.co.jp> X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-3.0 (sh.osrg.net [192.16.179.4]); Sat, 19 Sep 2009 14:14:36 +0900 (JST) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2571 Lines: 48 On Fri, 18 Sep 2009 22:08:03 +0200 Jens Axboe wrote: > On Fri, Sep 18 2009, Neil Brown wrote: > > On Thursday September 17, hch@infradead.org wrote: > > > On Thu, Sep 17, 2009 at 10:02:45AM -0600, James Bottomley wrote: > > > > So I think Christoph's NAK is rooted in the fact that we have a > > > > proliferation of in-kernel RAID implementations and he's trying to > > > > reunify them all again. > > > > > > > > As part of the review, reusing the kernel RAID (and actually logging) > > > > logic did come up and you added it to your todo list. Perhaps expanding > > > > on the status of that would help, since what's being looked for is that > > > > you're not adding more work to the RAID reunification effort and that > > > > you do have a plan and preferably a time frame for coming into sync with > > > > it. > > > > > > Yes. RDBD has spend tons of time out of tree, and if they want to put > > > it in now I think requiring them to do their homework is a good idea. > > > > What homework? > > > > If there was a sensible unifying framework in the kernel that they > > could plug in to, then requiring them do to that might make sense. But > > there isn't. You/I/We haven't created a solution (i.e. there is no > > equivalent of the VFS for virtual block devices) and saying that > > because we haven't they cannot merge DRBD hardly seems fair. > > > > Indeed, merging DRBD must be seen as a *good* thing as we then have > > more examples of differing requirements against which a proposed > > solution can be measured and tested. > > > > I thought the current attitude was "merge then fix". That is what the > > drivers/staging tree seems to be all about. Maybe you could argue > > that DRBD should go in to 'staging' first (though I don't think that > > is appropriate or require myself), but keeping it out just seems > > wrong. > > FWIW, I agree with Neil here. If drbd is merge clean, lets go ahead and > merge it. While it would be nice to offload the raid unification onto > drbd, it's not exactly fair. I guess that Christoph is worry about adding another user interface for kinda device management; once we merge this, we can't fix it (for the raid unification). BTW, DM already has something like drbd? I thought that there is a talk about that new target at LinuxCon. -- 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/