linux-kernel - Re: [PATCH] use printk_ratelimit() inside DRM

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <20061101120616.3274bb37.akpm@osdl.org>
Date:	Wed, 1 Nov 2006 12:06:16 -0800
From:	Andrew Morton <akpm@...l.org>
To:	Glauber de Oliveira Costa <gcosta@...hat.com>
Cc:	dri-devel@...ts.sourceforge.net, airlied@...ux.ie,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] use printk_ratelimit() inside DRM_DEBUG

On Wed, 1 Nov 2006 10:50:51 -0300
Glauber de Oliveira Costa <gcosta@...hat.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@...hat.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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/