All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kyle Moffett <kyle@moffetthome.net>
To: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
Cc: lars.ellenberg@linbit.com, arjan@infradead.org, lmb@suse.de,
	jens.axboe@oracle.com, neilb@suse.de, hch@infradead.org,
	James.Bottomley@suse.de, 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, torvalds@linux-foundation.org,
	nab@linux-iscsi.org, knikanth@suse.de,
	philipp.reisner@linbit.com, sam@ravnborg.org,
	Mauelshagen@redhat.com
Subject: Re: [GIT PULL] DRBD for 2.6.32
Date: Mon, 21 Sep 2009 20:51:32 -0400	[thread overview]
Message-ID: <f73f7ab80909211751p4d797556lcce5e47c3241b5ef@mail.gmail.com> (raw)
In-Reply-To: <20090922072617U.fujita.tomonori@lab.ntt.co.jp>

On Mon, Sep 21, 2009 at 18:27, FUJITA Tomonori
<fujita.tomonori@lab.ntt.co.jp> wrote:
> On Mon, 21 Sep 2009 18:53:21 +0200 Lars Ellenberg <lars.ellenberg@linbit.com> wrote:
>> That's not what I meant, of course that is and needs to be stable.
>> Sorry, I exagerated to make a point.
>>
>> Point was:
>> mdadm configured md.
>> dmsetup configured dm.
>> drbdsetup configure drbd.
>>
>> If and when "something" is done to "unify" things on the implementation
>> level, it is likely to also unify the "kernel<->userspace" configuration
>> interface.
>>
>> If it happens, once that happens, that _will_ be an ABI break.
>
> You misunderstand the raid unification.
>
> We will not unify the kernel<->userspace configuration interface
> because we can't break the kernel<->userspace ABI.
>
> We plan to unify the multiple device frameworks, but the unified
> framework must support the all existing ABIs.
>
> So adding another 'drbd' ABI hurts us.

One major issue for me personally (and I don't think its been mentioned enough):

There is a *VAST* existing user-base for DRBD.  Basically every vendor
builds the modules for their kernels, ships the userspace tools, etc.
*Regardless* of when or how it gets merged, the existing user-base
will need kernel support for the existing tools.

You have to realize that this project is NOT a new one, it's been
around quite a decent number of years (since kernel 2.2-ish).  Yes,
the ABI is unique and has its warts, but there are a lot of things
that depend on it.

Think of it (in concept) like merging mainline support for an
architecture that has been forward-ported as patches since 2.2.  If
the architecture was a simple embedded-only one (like a few recent
ones have been), then you might just say "hell with it, everybody
needs to rebuild libc and the world".  That doesn't seem to be the
case with an enterprise-supported distributed block device.

IMHO, we should treat the kernel<=>userspace ABI as fixed... it's an
existing wart that will need to be supported for a while.  The
benefits of getting the stable and long-out-of-tree drbd modules into
the mainline kernel will far outweigh the pain of having to maintain
the existing ABI.

To put it another way:  Would you really keep a stable SCSI raid
driver for existing hardware out of mainline by claiming they need to
write a new raid-management abstraction first?  If not, then why the
pushback on DRBD?

Cheers,
Kyle Moffett

  reply	other threads:[~2009-09-22  1:13 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-09-15 14:45 [GIT PULL] DRBD for 2.6.32 Philipp Reisner
2009-09-15 23:19 ` Christoph Hellwig
2009-09-16  8:33   ` Philipp Reisner
2009-09-17  8:12   ` Lars Ellenberg
2009-09-17 16:02     ` James Bottomley
2009-09-17 16:11       ` Christoph Hellwig
2009-09-18  3:32         ` Neil Brown
2009-09-18 20:08           ` Jens Axboe
2009-09-19  5:14             ` FUJITA Tomonori
2009-09-19 22:02               ` Lars Marowsky-Bree
2009-09-19 23:56                 ` Dan Williams
2009-09-21 13:39                 ` FUJITA Tomonori
2009-09-21 14:43                   ` Lars Ellenberg
2009-09-21 14:52                     ` Arjan van de Ven
2009-09-21 16:53                       ` Lars Ellenberg
2009-09-21 22:27                         ` FUJITA Tomonori
2009-09-22  0:51                           ` Kyle Moffett [this message]
2009-09-23 11:27                             ` FUJITA Tomonori
2009-09-23 11:57                             ` Christoph Hellwig
2009-09-23 14:01                               ` Kyle Moffett
2009-09-23 23:21                                 ` FUJITA Tomonori
2009-09-22  6:20                           ` Lars Marowsky-Bree
2009-09-23 11:36                             ` FUJITA Tomonori
2009-09-23 23:06                               ` Neil Brown
2009-09-23 23:37                                 ` FUJITA Tomonori
2009-09-25  5:27                                   ` Neil Brown
2009-09-25  9:59                                     ` Lars Marowsky-Bree
2009-09-21 14:55                     ` [Drbd-dev] " Lars Ellenberg
2009-09-22  5:37                     ` Heinz Mauelshagen
2009-09-17  8:50   ` Lars Marowsky-Bree
2009-09-16  0:46 ` KOSAKI Motohiro
2009-09-16  9:19   ` Philipp Reisner
2009-09-17 18:52 devzero
2009-09-23 19:10 devzero

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=f73f7ab80909211751p4d797556lcce5e47c3241b5ef@mail.gmail.com \
    --to=kyle@moffetthome.net \
    --cc=James.Bottomley@suse.de \
    --cc=Mauelshagen@redhat.com \
    --cc=akpm@linux-foundation.org \
    --cc=arjan@infradead.org \
    --cc=bart.vanassche@gmail.com \
    --cc=davej@redhat.com \
    --cc=drbd-dev@lists.linbit.com \
    --cc=fujita.tomonori@lab.ntt.co.jp \
    --cc=gregkh@suse.de \
    --cc=hch@infradead.org \
    --cc=jens.axboe@oracle.com \
    --cc=knikanth@suse.de \
    --cc=kosaki.motohiro@jp.fujitsu.com \
    --cc=lars.ellenberg@linbit.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lmb@suse.de \
    --cc=nab@linux-iscsi.org \
    --cc=neilb@suse.de \
    --cc=philipp.reisner@linbit.com \
    --cc=sam@ravnborg.org \
    --cc=torvalds@linux-foundation.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.