lib/percpu-refcount.c: use a more common logging style
authorJoe Perches <joe@perches.com>
Thu, 4 Jun 2020 23:50:20 +0000 (16:50 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 5 Jun 2020 02:06:25 +0000 (19:06 -0700)
Remove the trailing newline from the used-once pr_fmt and add it to the
single use of pr_<level> in this code to use a more common logging style.

Miscellanea:

o Use %lu in the pr_debug format and remove the unnecessary cast

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Cc: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Link: http://lkml.kernel.org/r/47372467902a047c03b0fd29aab56e0c38d3f848.camel@perches.com
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
lib/percpu-refcount.c

index 8d09260..0ba686b 100644 (file)
@@ -1,5 +1,5 @@
 // SPDX-License-Identifier: GPL-2.0-only
-#define pr_fmt(fmt) "%s: " fmt "\n", __func__
+#define pr_fmt(fmt) "%s: " fmt, __func__
 
 #include <linux/kernel.h>
 #include <linux/sched.h>
@@ -141,8 +141,8 @@ static void percpu_ref_switch_to_atomic_rcu(struct rcu_head *rcu)
        for_each_possible_cpu(cpu)
                count += *per_cpu_ptr(percpu_count, cpu);
 
-       pr_debug("global %ld percpu %ld",
-                atomic_long_read(&ref->count), (long)count);
+       pr_debug("global %lu percpu %lu\n",
+                atomic_long_read(&ref->count), count);
 
        /*
         * It's crucial that we sum the percpu counters _before_ adding the sum