Re: [patch] change WARN_ON back to "BUG: at ..."

From: Ingo Molnar
Date: Sat Dec 23 2006 - 06:09:52 EST



* Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:

> WARN_ON is still a BUG, but we know enough about it that we can just
> cripple the system so that it doesn't break anything. [...]

well - a WARN_ON() can be /anything/. It is the same as BUG_ON(), but it
doesnt crash the box immediately and on purpose. In that sense all
existing BUG_ON()s should be converted to WARN_ON()s or to panic()s
(whichever fits best for that particular BUG_ON()).

[ or all WARN_ON()s should be converted to BUG_ON()s and the behavior of
BUG_ON() should be changed to /not/ crash the system on purpose - and
for those cases where we really do not want to continue, panic()
should be used. That way the user can set panic policy himself. ]

i can whip up a patch for any of these conversions, but i dont think we
need this flux right now.

Ingo
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/