linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Bob Copeland" <me@bobcopeland.com>
To: "Chris Mason" <chris.mason@oracle.com>
Cc: "Christoph Hellwig" <hch@infradead.org>,
	"Alan Cox" <alan@lxorguk.ukuu.org.uk>,
	"Andrew Morton" <akpm@linux-foundation.org>,
	"David Woodhouse" <dwmw2@infradead.org>,
	"Miklos Szeredi" <miklos@szeredi.hu>,
	linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org
Subject: Re: [PATCH 0/7] OMFS filesystem version 3
Date: Mon, 14 Apr 2008 12:36:54 -0400	[thread overview]
Message-ID: <b6c5339f0804140936s3dcce48en59a270dc86bf7dc8@mail.gmail.com> (raw)
In-Reply-To: <200804141020.37271.chris.mason@oracle.com>

On Mon, Apr 14, 2008 at 10:20 AM, Chris Mason <chris.mason@oracle.com> wrote:
>  Thanks to Bob Copeland for taking the time to submit this for mainline.
>  Please don't mistake the resulting debate as a sign we don't appreciate the
>  effort of making it available and getting it reviewed.

Sure, no worries...

>  Unlike all the device drivers we don't want floating around out of the tree,
>  filesystem authors do have a choice between FUSE and being in-kernel.  Since
>  OMFS has been maintained out of tree since 2.6.12 or so, Bob probably has a
>  very good idea of how much time he has needed to spend updating things for
>  each release.

The recent deprecation of iget was probably the most involved of any.  It
wasn't that bad, but the #ifdef jungle going back to at least .17 is getting
somewhat hairy (I have 19 of them).  I would say over the past year or so
most of the changes have been <10 liners with kmem_cache_* interface changes.
Luckily through review here, that got zapped anyway.

In the normal case, I'll grab an -rc3 kernel or so, try to compile it, if
it fails, look at minix to see the minimum set of changes I need to make.
The challenge for me maintaining it out-of-tree is that sometimes the git
logs do not carry enough context to know why XYZ changed and if there's a
better, new way to do things.  Then it's time to scour the mailing lists.

>  So, there are two big questions I'd ask:
>
>  * Which setup (kernel or fuse) would Bob find easier to maintain things?
>  * Is this a first step toward more development and an increased user base?

Well, FUSE is easier from the standpoint of having somewhat more control.
On the other hand, that also results in an inferior product if it isn't given
the many-eyes treatment.  For the code itself, it's really not that much
different since FUSE is so close to the VFS API anyway.

For your second question:

More users are always welcome.  At present, the FUSE version implements a few
things that the kernel module does not - better/different block allocation
algorithm, growing truncate, automatic byte-swapping for the ReplayTV model
that had a busted block layer.  However, the FUSE version is known to be buggy
with respect to every day use (I had one MD5 fail among 20G of files in one
copy test).

I could see adding 1 and 2 to the kernel version, but probably not the
byte-swapping bit.  I'm not sure what to do from there on.

As a user, I use the kernel module because it works, it's faster, and the
other bells and whistles aren't that important to me.  It's hard to tell
what the user community prefers, because the kernel module was first and
so far no one has had a reason to try the FUSE implementation (if they
have, I don't know about it).

-- 
Bob Copeland %% www.bobcopeland.com

  reply	other threads:[~2008-04-14 16:36 UTC|newest]

Thread overview: 92+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-04-12 22:58 [PATCH 0/7] OMFS filesystem version 3 Bob Copeland
2008-04-13  0:03 ` Andrew Morton
2008-04-13  3:33   ` Bob Copeland
2008-04-13  3:55     ` Andrew Morton
2008-04-13  4:41       ` Bob Copeland
2008-04-13  8:01       ` Christoph Hellwig
2008-04-13  8:20         ` Andrew Morton
2008-04-13  8:28           ` Christoph Hellwig
2008-04-13 10:37             ` Miklos Szeredi
2008-04-13 21:15               ` David Woodhouse
2008-04-13 22:44                 ` Andrew Morton
2008-04-13 22:49                   ` Alan Cox
2008-04-13 23:10                     ` Andrew Morton
2008-04-14  1:32                       ` Bob Copeland
2008-04-14  5:48                         ` Andrew Morton
2008-04-14  8:16                       ` Alan Cox
2008-04-14  8:36                         ` Andrew Morton
2008-04-14  8:41                         ` Christoph Hellwig
2008-04-14 14:20                           ` Chris Mason
2008-04-14 16:36                             ` Bob Copeland [this message]
2008-04-14 16:51                               ` Alan Cox
2008-04-14 17:18                                 ` Bob Copeland
2008-04-14 17:22                                   ` Alan Cox
2008-04-14 20:29                             ` david
2008-04-18 13:13                               ` Szabolcs Szakacsits
2008-04-14 20:55                           ` Jeff Garzik
2008-04-14 21:11                             ` Andrew Morton
2008-04-14 22:32                               ` Evgeniy Polyakov
2008-04-14 23:21                                 ` Andrew Morton
2008-04-14 23:09                               ` SL Baur
2008-04-14 23:24                                 ` Andrew Morton
2008-04-14  8:30                       ` Xavier Bestel
2008-04-14  8:43                         ` Christoph Hellwig
2008-04-14  8:44                         ` Andrew Morton
2008-04-14  9:08                           ` Christoph Hellwig
2008-04-14  9:09                           ` Christoph Hellwig
2008-04-14  9:21                             ` Andrew Morton
2008-04-14 10:09                               ` David Woodhouse
2008-04-14 10:22                                 ` Andrew Morton
2008-04-14 10:36                                   ` David Woodhouse
2008-04-14 11:16                                   ` Christoph Hellwig
2008-04-15 15:16                       ` Adrian Bunk
2008-04-15 16:57                         ` Christoph Hellwig
2008-04-15 18:34                           ` Andrew Morton
2008-04-15 18:53                             ` Alan Cox
2008-04-15 20:02                               ` Andrew Morton
2008-04-15 19:58                                 ` Alan Cox
2008-04-15 21:46                                   ` david m. richter
2008-04-15 19:24                             ` Adrian Bunk
2008-04-15 20:11                               ` Andrew Morton
2008-04-15 20:27                                 ` Adrian Bunk
2008-04-14  7:25                 ` Miklos Szeredi
2008-04-14  7:49                   ` Andrew Morton
2008-04-14  8:11                     ` Miklos Szeredi
2008-04-14  8:11                     ` Anton Altaparmakov
2008-04-14  8:26                       ` Miklos Szeredi
2008-04-14  9:42                   ` Jamie Lokier
2008-04-14  9:58                     ` Miklos Szeredi
2008-04-14 11:05                       ` David Woodhouse
2008-04-14 12:50                         ` Miklos Szeredi
2008-04-14 11:55                       ` Jamie Lokier
2008-04-14 11:57                         ` Christoph Hellwig
2008-04-14 12:26                         ` Miklos Szeredi
2008-04-14 22:35                           ` Jamie Lokier
2008-04-15 11:33                             ` Miklos Szeredi
2008-04-15 15:23                               ` Jamie Lokier
2008-04-17  1:08                                 ` Szabolcs Szakacsits
2008-04-17  6:50                                   ` Jamie Lokier
2008-04-17  8:17                                     ` Miklos Szeredi
2008-04-14  0:45               ` Bob Copeland
2008-04-14  7:35                 ` Miklos Szeredi
2008-04-18 10:30                 ` Szabolcs Szakacsits
2008-04-18 11:52                   ` Jamie Lokier
2008-04-18 12:20                     ` Miklos Szeredi
2008-04-18 12:57                       ` Jamie Lokier
2008-04-18 16:01                         ` Miklos Szeredi
2008-04-18 16:15                           ` Jamie Lokier
2008-04-18 13:51                   ` Bob Copeland
2008-04-18 14:23                     ` Miklos Szeredi
2008-04-18 14:43                       ` Bob Copeland
2008-04-18 17:35                     ` Szabolcs Szakacsits
2008-04-18 17:48                       ` Bob Copeland
2008-04-18 12:46             ` Szabolcs Szakacsits
2008-04-13  9:04     ` Alan Cox
2008-04-13 10:00     ` Pavel Machek
2008-04-15  3:14     ` Erez Zadok
     [not found] <14725485.776281208170999085.JavaMail.szaka@kolumbus.fi>
2008-04-14 12:46 ` Szabolcs Szakacsits
2008-04-14 13:15   ` Anton Altaparmakov
2008-04-14 16:12     ` Szabolcs Szakacsits
     [not found] <10224488.783261208172602425.JavaMail.szaka@kolumbus.fi>
2008-04-15  0:11 ` Szabolcs Szakacsits
2008-04-15 15:05   ` Adrian Bunk
     [not found] <aiwLk-3mc-3@gated-at.bofh.it>
     [not found] ` <aiwLk-3mc-5@gated-at.bofh.it>
     [not found]   ` <aiwLk-3mc-1@gated-at.bofh.it>
2008-04-16  9:56     ` Bodo Eggert

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=b6c5339f0804140936s3dcce48en59a270dc86bf7dc8@mail.gmail.com \
    --to=me@bobcopeland.com \
    --cc=akpm@linux-foundation.org \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=chris.mason@oracle.com \
    --cc=dwmw2@infradead.org \
    --cc=hch@infradead.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=miklos@szeredi.hu \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).