Re: [PATCH 0/7] OMFS filesystem version 3

[Andrew Morton]

On Sat, 12 Apr 2008 18:58:34 -0400 Bob Copeland <me@xxxxxxxxxxxxxxx> wrote:

> (Andrew, perhaps this could bake in -mm for a cycle or two if 
> there are no objections?)

Sure, if it compiles I can stick in in there for some airtime.

> These patches 1-7 add the Optimized MPEG Filesystem, a proprietary
> filesystem used by the embedded devices Rio Karma and ReplayTV 
> which are no longer manufactured.

Adding a new FS to Linux is a pretty major thing.

- A key question (which you don't seem to have addressed at all!) is: why
  is this a useful addition to Linux?  What are the filesystem's strengths?
  What is its application?  How does it improves Linux and by how much?

  I'll go aehad and assume that its sweet-spot is streaming media files
  (perhaps more than one at a time?).  If so, then comparative performance
  measurements would be the key piece of information which we'll need when
  making a merge decision.  Probably against ext3, ext4 and XFS.

- What are the filesystem's features?  Does it journal changes, or is it
  inherently crash-safe?  Or is fsck-style repair needed?

- We'd like to see some documentation:

  - Mount options

  - Location of userspace support tools such as mkfs and fsck

  - Documentation for those tools

- A MAINTAINERS record, please?


If I include this in -mm, some kind people will try it out.  It would be
most helpful for the changelog (or even the Kconfig help) to include
sufficient information for people to be able to create, mount and check an
OMFS partition.

> I've been maintaining this reverse-engineered filesystem
> out-of-tree since around 2.6.12.

Who did the reverse-engineering, and how was it done?

Please make us confident that we won't get our butts sued off or something.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/