From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757778AbaE2PvJ (ORCPT ); Thu, 29 May 2014 11:51:09 -0400 Received: from relay3-d.mail.gandi.net ([217.70.183.195]:52258 "EHLO relay3-d.mail.gandi.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757366AbaE2PvH (ORCPT ); Thu, 29 May 2014 11:51:07 -0400 X-Originating-IP: 50.43.32.211 Date: Thu, 29 May 2014 08:50:55 -0700 From: Josh Triplett To: Fabian Frederick Cc: linux-kernel , akpm Subject: Re: [PATCH V2 1/3] lib/debugobjects.c: convert printk to pr_foo() Message-ID: <20140529155055.GA31286@thin> References: <20140529092258.b05c5e133d8b26838e9ffded@skynet.be> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20140529092258.b05c5e133d8b26838e9ffded@skynet.be> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, May 29, 2014 at 09:22:58AM +0200, Fabian Frederick wrote: > Convert all printk to pr_foo() except KERN_DEBUG > (see Documentation/CodingStyle Chapter 13) > > Cc: Josh Triplett > Cc: Andrew Morton > Signed-off-by: Fabian Frederick Reviewed-by: Josh Triplett > lib/debugobjects.c | 12 +++++------- > 1 file changed, 5 insertions(+), 7 deletions(-) > > diff --git a/lib/debugobjects.c b/lib/debugobjects.c > index e0731c3..ea4c737 100644 > --- a/lib/debugobjects.c > +++ b/lib/debugobjects.c > @@ -218,7 +218,7 @@ static void debug_objects_oom(void) > unsigned long flags; > int i; > > - printk(KERN_WARNING "ODEBUG: Out of memory. ODEBUG disabled\n"); > + pr_warn("ODEBUG: Out of memory. ODEBUG disabled\n"); > > for (i = 0; i < ODEBUG_HASH_SIZE; i++, db++) { > raw_spin_lock_irqsave(&db->lock, flags); > @@ -292,11 +292,9 @@ static void debug_object_is_on_stack(void *addr, int onstack) > > limit++; > if (is_on_stack) > - printk(KERN_WARNING > - "ODEBUG: object is on stack, but not annotated\n"); > + pr_warn("ODEBUG: object is on stack, but not annotated\n"); > else > - printk(KERN_WARNING > - "ODEBUG: object is not on stack, but annotated\n"); > + pr_warn("ODEBUG: object is not on stack, but annotated\n"); > WARN_ON(1); > } > > @@ -985,7 +983,7 @@ static void __init debug_objects_selftest(void) > if (check_results(&obj, ODEBUG_STATE_NONE, ++fixups, ++warnings)) > goto out; > #endif > - printk(KERN_INFO "ODEBUG: selftest passed\n"); > + pr_info("ODEBUG: selftest passed\n"); > > out: > debug_objects_fixups = oldfixups; > @@ -1090,7 +1088,7 @@ void __init debug_objects_mem_init(void) > debug_objects_enabled = 0; > if (obj_cache) > kmem_cache_destroy(obj_cache); > - printk(KERN_WARNING "ODEBUG: out of memory.\n"); > + pr_warn("ODEBUG: out of memory.\n"); > } else > debug_objects_selftest(); > } > -- > 1.9.1