lkml.org 
[lkml]   [2006]   [Nov]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH] use printk_ratelimit() inside DRM_DEBUG
    On Wed, 1 Nov 2006 10:50:51 -0300
    Glauber de Oliveira Costa <gcosta@redhat.com> wrote:

    > the DRM_DEBUG macro can be called within functions very oftenly
    > triggered, thus generating lots of message load and potentially
    > compromising system
    >
    > Signed-off-by: Glauber de Oliveira Costa <gcosta@redhat.com>
    >
    > --
    > Glauber de Oliveira Costa
    > Red Hat Inc.
    > "Free as in Freedom"
    >
    >
    > [drm_debug.patch text/plain (444B)]
    > --- linux-2.6.18.x86_64/drivers/char/drm/drmP.h.orig 2006-11-01 08:00:18.000000000 -0500
    > +++ linux-2.6.18.x86_64/drivers/char/drm/drmP.h 2006-11-01 08:06:27.000000000 -0500
    > @@ -185,7 +185,7 @@
    > #if DRM_DEBUG_CODE
    > #define DRM_DEBUG(fmt, arg...) \
    > do { \
    > - if ( drm_debug ) \
    > + if ( drm_debug && printk_ratelimit() ) \
    > printk(KERN_DEBUG \
    > "[" DRM_NAME ":%s] " fmt , \
    > __FUNCTION__ , ##arg); \


    DRM_DEBUG() should be disabled in production code, and enabled only when
    developers are developing stuff. In the latter case, the developer wants
    to see all the messages.

    IOW, don't load the drm module with the `debug' parameter.
    -
    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-11-01 21:11    [W:0.041 / U:0.136 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site