[PATCH 0/2] nand: Remove BUG abuse
Ezequiel Garcia
ezequiel at vanguardiasur.com.ar
Sat Apr 2 08:37:06 PDT 2016
On 2 April 2016 at 10:55, Boris Brezillon
<boris.brezillon at free-electrons.com> wrote:
> Hi Ezequiel,
>
> On Fri, 1 Apr 2016 18:29:22 -0300
> Ezequiel Garcia <ezequiel at vanguardiasur.com.ar> wrote:
>
>> Hi,
>>
>> While using nandsim to debug an issue with an UBI image,
>> I hit a BUG() when passing some crazy ID values to nandsim.
>>
>> I've always felt that nand_base.c is sort of abusing the
>> BUG() macro, so decided to fix it.
>>
>> There are other BUG() uses, but they aren't in the nand_scan
>> path, so I'm not touching them.
>
> Now that you've opened the door to this rework, can you also patch other
> locations where BUG() is employed? :)
>
Well, it's true that BUG() is abused all over the place, but most of them
should be of the "not reached" type.
So, I fixed these two because they *are* reached and they are in the core.
--
Ezequiel García, VanguardiaSur
www.vanguardiasur.com.ar
More information about the linux-mtd
mailing list