[dm-devel] dm-integrity: fix inefficient allocation of stack space

Mikulas Patocka mpatocka at redhat.com
Thu Jul 20 10:59:18 UTC 2017



On Wed, 19 Jul 2017, John Stoffel wrote:

> I'd like to argue that you should never use BUG_ON at all, esp since
> if you have integrity running on just one critical device, but have
> other devices that work just fine, bringing down the entire system
> because you don't think things are ok is terrible.
> 
> We should rip out ALL the BUG_ONs in the dm-integrity and just do
> proper error handling instead.  For testing, sure, use them on your
> code.  But please, not for general use!
> 
> John

Linus sometimes argued against BUG_ON and people are repeating it after 
him like sheep.

If a programmer made a bug in his code, he should fix that bug, not write 
additional recovery code for the bug.

Mikulas




More information about the dm-devel mailing list