Messages in this thread |  | | Date | Mon, 9 Sep 2002 17:17:40 -0700 (PDT) | From | Linus Torvalds <> | Subject | Re: [BK PATCH] USB changes for 2.5.34 |
| |
Greg, please don't do this
> ChangeSet@1.614, 2002-09-05 08:33:20-07:00, greg@kroah.com > USB: storage driver: replace show_trace() with BUG()
that BUG() thing is _way_ out of line, and has killed a few of my machines several times for no good reason. It actively hurts debuggability, because the machine is totally dead after it, and the whole and ONLY point of BUG() messages is to help debugging and make it clear that we can't handle something.
In this case, we _can_ handle it, and we're much better off with a machine that works and that you can look up the messages with than killing it.
Rule of thumb: BUG() is only good for something that never happens and that we really have no other option for (ie state is so corrupt that continuing is deadly).
Linus
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
|  |