Re: [PATCH/RFC] MTD: Striping layer core

["Jörn Engel" <joern@wohnheim.fh-wedel.de>]

On Fri, 31 March 2006 12:22:50 -0500, Nicolas Pitre wrote:
> 
> You are right here.  This is why MTD drivers should export _flash_ 
> features rather than filesystem requirements.  And it is up to 
> filesystem code to cope with the (lack of) certain flash features not 
> the other way around.

As long as noone is being a fundamentalist zealot, we all agree here.

My fine point of disagreement is that I stress "should" above
everything else.  Quite a few things can and should be improved about
mtd and its users.  If everything were perfect, it would all be about
flash features.  Until then, it may make sense to _temporarily_ hold
some filesystem features.  While this is undeniably a bad thing in its
own, it can allow you to get rid of bigger warts.

Once the bigger warts are gone, we can aim for perfect. ;)

As a step in that direction, please take a look at this patchset:
http://wh.fh-wedel.de/~joern/mtd_type.tgzwh.fh-wedel.de/~joern/mtd_type.tgz

It removes all types except MTD_ABSENT and all flags but MTD_OOB.  As
a replacement, three new flags are introduced.  So now we're at a
total of 4 flags (previously 9) and two types (previously 9).

Jörn

-- 
...one more straw can't possibly matter...
-- Kirby Bakken