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

Jörn Engel
2006-04-03T13:07:00Z

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