Re: MTD concat layer

["Brian J. Fox" <bfox@ua.com>]

   From: David Woodhouse <dwmw2@infradead.org>
   Cc: "Brian J. Fox" <bfox@ua.com>, linux-mtd@lists.infradead.org,
       J?rn Engel <joern@wohnheim.fh-wedel.de>
   Date: Mon, 18 Feb 2002 09:05:55 +0000
   Sender: David Woodhouse <dwmw2@redhat.com>

   There is never any excuse for passing invalid offset/length to the
   erase function - even the ioctl code can do a sanity check before
   passing through the values provided by the user.

That's right.

   For errors which can understandably occur, your concerns are valid
   and we should ensure sanity by failing the request as safely as
   possible, doing nothing else.

   This is not such an error. As long as the ioctl() has an
   appropriate sanity check, the occurrence of such an error indicates
   that the kernel code is completely broken. Better to BUG()

BUG() is completely acceptable.  It allows the driver to be fixed.
It's also orthogonal to the original question.  Erasing blocks is
simply a way to guarantee that data has been irrevocably lost.

Brian
== The Difference Between Cultures: ==
    Einigkeit und Recht und Freiheit
    Liberte', E'galite', Fraternite'
    Sex, drugs and rock'n'roll