lkml.org 
[lkml]   [2002]   [Sep]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [linux-usb-devel] Re: [BK PATCH] USB changes for 2.5.34

    On 10 Sep 2002, Alan Cox wrote:
    >
    > It drops you politely into the kernel debugger, you fix up the values
    > and step over it. If you want to debug with zen mind power and printk
    > feel free. For the rest of us BUG() is fine on SMP

    Ok, a show of hands..

    Of the millions (whatever) of Linux machines, how many have a kernel
    debugger attached? Count them.

    In other words, if a user is faced with a dead machine with no other way
    to even know what BUG() triggered than to try to set up a cross debugger,
    just how useful is that BUG()? I claim it is pretty useless - simply
    because 99+% of all people won't even make a bug report in that case,
    they'll just push the reset button and consider Linux unreliable.

    In other news, the approach that shows up in the kernel logs might just
    eventually be noticed and acted upon (especially if the machine acts
    strange and kills processes).

    So I claim a BUG() that locks up the machine is useless. If the user can't
    just run ksymoops and email out the BUG message, that BUG() is _not_ fine
    on SMP.

    It has nothing to do with zen mind power or printk's. It has everything to
    do with the fact that a working machine is about a million times easier to
    debug on than a dead one, _and_ is a lot more likely to get acted upon by
    most users.

    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/

    \
     
     \ /
      Last update: 2005-03-22 13:28    [W:2.297 / U:0.028 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site