lkml.org 
[lkml]   [2006]   [Mar]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH] reduce syslog clutter (take 2)
    On Thu, Mar 09, 2006 at 01:18:47PM -0800, Pete Zaitcev wrote:
    > On Thu, 9 Mar 2006 13:03:27 -0800, Andrew Morton <akpm@osdl.org> wrote:
    > > "Jesper Juhl" <jesper.juhl@gmail.com> wrote:
    > > >
    > > > > Feb 21 00:12:13 gx110 kernel: gigaset: ISDN_CMD_SETL3: invalid protocol 42
    > > > >
    > > > > do not provide any useful information for that clientele. They just push
    > > >
    > > > The filename may not be useful to the user, but the instant the user decides to
    > > > submit a bugreport to LKML or elsewhere it becomes useful.
    > >
    > > But OTOH, there's a difference between messages-to-developers (usually "the
    > > code went wrong") and messages-to-users (hopefully usually "the hardware
    > > went wrong" or "you went wrong").
    >
    > Symbol names are generally unique. As a USB stack developer, I never saw
    > the file name being useful for anything in the error message, let alone
    > the full path! Always hated them, but never bothered to break spears
    > over the issue. We have better things to do. I just quietly remove
    > debugging printouts from the code I touch.

    There's a bit of history here. The dbg(), err(), info() macros came
    from the USB core, back in 2.2 days. Then the whole path of the file
    was not part of __FILE__, but only the single .c file.

    With 2.5, __FILE__ changed as part of the build process changes, and we
    added dev_dbg(), dev_info(), and dev_err(), which are a _much_ better
    way to output information from a driver. It provides the exact driver
    and device that is being talked about, and not just a file.

    So, ideally I'd like to get rid of the USB macros completly, and use the
    dev_* forms instead. But there are a few places in the USB code that do
    not have a valid device and so they can't be dropped entirely.

    Either way, I don't think we need to be making them "prettier" at this
    point in time, but fixing the real problem of using them in the first
    place...

    I'll drop this patch for now, and only take the part that adds the new
    dev_* macro. Is that ok for everyone?

    And if anyone wants to notify the kernel-janitors that this would be a
    good thing to do for the USB subsytem, feel free, I'll gladly accept
    those patches.

    thanks,

    greg k-h
    -
    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: 2006-03-09 23:45    [W:0.023 / U:0.684 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site