lkml.org 
[lkml]   [2010]   [Nov]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v2 1/4] drivers: hwspinlock: add generic framework
    On Fri, Nov 26, 2010 at 10:53:10AM +0200, Ohad Ben-Cohen wrote:
    > >> +int __hwspin_trylock(struct hwspinlock *hwlock, int mode, unsigned long *flags)
    > >> +{
    > >> +     int ret;
    > >> +
    > >> +     if (unlikely(!hwlock)) {
    > >> +             pr_err("invalid hwlock\n");
    > >
    > > These kind of errors can get very spammy for buggy drivers.
    >
    > Yeah, but that's the purpose - I want to catch such egregious drivers
    > who try to crash the kernel.

    That can be better - because you get a backtrace, and it causes people
    to report the problem rather than just ignore it. It may also prevent
    the driver author releasing his code (as it won't work on their
    initial testing.)

    > > It's likely
    > > more useful to either do a WARN_ON(), and/or move them under a debug
    > > config option.
    >
    > Why would you prefer to compile out reporting of such extremely buggy
    > behavior ?

    If it's "extremely buggy behaviour" then the drivers deserve to crash.
    Such stuff should cause them not to get out the door. A simple printk
    with an error return can just be ignored.
    --
    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: 2010-11-26 10:21    [W:2.363 / U:0.016 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site