[PATCH] ieee80211_geo.c: remove frivolous BUG_ON's
authorPete Zaitcev <zaitcev@redhat.com>
Mon, 27 Feb 2006 00:29:51 +0000 (16:29 -0800)
committerJohn W. Linville <linville@tuxdriver.com>
Tue, 28 Feb 2006 01:14:58 +0000 (20:14 -0500)
commit07981aa43f6aec32b875f360755ed3d14f9d5139
tree21baf289bc23882f058e7ed90f80c76d01408119
parenta23f460dd0c7c9c58b03494c7819e126b2c72383
[PATCH] ieee80211_geo.c: remove frivolous BUG_ON's

I have come to consider BUG_ON generally harmful. The idea of an assert is
to prevent a program to execute past a point where its state is known
erroneous, thus preventing it from dealing more damage to the data
(or hiding the traces of malfunction). The problem is, in kernel this harm
has to be balanced against the harm of forced reboot.

The last straw was our softmac tree, where "iwlist eth1 scan" causes
a lockup. It is absolutely frivolus and provides no advantages a normal
assert has to provide. In fact, doing this impedes debugging.

Signed-off-by: Pete Zaitcev <zaitcev@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
net/ieee80211/ieee80211_geo.c