[PATCH V2 3/3] lib/debugobjects.c: convert printk(KERN_DEBUG to pr_debug

[PATCH V2 3/3] lib/debugobjects.c: convert printk(KERN_DEBUG to pr_debug

[Fabian Frederick]

Direct conversion of one KERN_DEBUG message without DEBUG definition
(suggested by Josh Triplett)

That message will now be disabled by default.
(see Documentation/CodingStyle Chapter 13)

Cc: Josh Triplett <josh@joshtriplett.org>
Cc: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Fabian Frederick <fabf@skynet.be>
---
 lib/debugobjects.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/debugobjects.c b/lib/debugobjects.c
index b628247..547f7f9 100644
--- a/lib/debugobjects.c
+++ b/lib/debugobjects.c
@@ -1061,8 +1061,8 @@ static int __init debug_objects_replace_static_objects(void)
 	}
 	local_irq_enable();
 
-	printk(KERN_DEBUG "ODEBUG: %d of %d active objects replaced\n", cnt,
-	       obj_pool_used);
+	pr_debug("%d of %d active objects replaced\n",
+		 cnt, obj_pool_used);
 	return 0;
 free:
 	hlist_for_each_entry_safe(obj, tmp, &objects, node) {
-- 
1.9.1

Re: [PATCH V2 3/3] lib/debugobjects.c: convert printk(KERN_DEBUG to pr_debug

[Josh Triplett]

On Thu, May 29, 2014 at 09:25:32AM +0200, Fabian Frederick wrote:
> Direct conversion of one KERN_DEBUG message without DEBUG definition
> (suggested by Josh Triplett)
> 
> That message will now be disabled by default.
> (see Documentation/CodingStyle Chapter 13)
> 
> Cc: Josh Triplett <josh@joshtriplett.org>
> Cc: Andrew Morton <akpm@linux-foundation.org>
> Signed-off-by: Fabian Frederick <fabf@skynet.be>

Reviewed-by: Josh Triplett <josh@joshtriplett.org>

>  lib/debugobjects.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/lib/debugobjects.c b/lib/debugobjects.c
> index b628247..547f7f9 100644
> --- a/lib/debugobjects.c
> +++ b/lib/debugobjects.c
> @@ -1061,8 +1061,8 @@ static int __init debug_objects_replace_static_objects(void)
>  	}
>  	local_irq_enable();
>  
> -	printk(KERN_DEBUG "ODEBUG: %d of %d active objects replaced\n", cnt,
> -	       obj_pool_used);
> +	pr_debug("%d of %d active objects replaced\n",
> +		 cnt, obj_pool_used);
>  	return 0;
>  free:
>  	hlist_for_each_entry_safe(obj, tmp, &objects, node) {
> -- 
> 1.9.1