tile: use %*pb[l] to print bitmaps including cpumasks and nodemasks

printk and friends can now format bitmaps using '%*pb[l]'.  cpumask
and nodemask also provide cpumask_pr_args() and nodemask_pr_args()
respectively which can be used to generate the two printf arguments
necessary to format the specified cpu/nodemask.

Signed-off-by: Tejun Heo <tj@kernel.org>
Cc: Chris Metcalf <cmetcalf@tilera.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

5 files changed, 20 insertions, 33 deletions

diff --git a/arch/tile/kernel/hardwall.c b/arch/tile/kernel/hardwall.c
index c4646bb99342..2fd1694ac1d0 100644
--- a/arch/tile/kernel/hardwall.c
+++ b/arch/tile/kernel/hardwall.c
@@ -909,11 +909,8 @@ static void hardwall_destroy(struct hardwall_info *info)
 static int hardwall_proc_show(struct seq_file *sf, void *v)
 {
         struct hardwall_info *info = sf->private;
-        char buf[256];
 
-        int rc = cpulist_scnprintf(buf, sizeof(buf), &info->cpumask);
-        buf[rc++] = '\n';
-        seq_write(sf, buf, rc);
+        seq_printf(sf, "%*pbl\n", cpumask_pr_args(&info->cpumask));
         return 0;
 }
 
diff --git a/arch/tile/kernel/proc.c b/arch/tile/kernel/proc.c
index 6829a9508649..7983e9868df6 100644
--- a/arch/tile/kernel/proc.c
+++ b/arch/tile/kernel/proc.c
@@ -45,10 +45,9 @@ static int show_cpuinfo(struct seq_file *m, void *v)
         int n = ptr_to_cpu(v);
 
         if (n == 0) {
-                char buf[NR_CPUS*5];
-                cpulist_scnprintf(buf, sizeof(buf), cpu_online_mask);
                 seq_printf(m, "cpu count\t: %d\n", num_online_cpus());
-                seq_printf(m, "cpu list\t: %s\n", buf);
+                seq_printf(m, "cpu list\t: %*pbl\n",
+                           cpumask_pr_args(cpu_online_mask));
                 seq_printf(m, "model name\t: %s\n", chip_model);
                 seq_printf(m, "flags\t\t:\n");  /* nothing for now */
                 seq_printf(m, "cpu MHz\t\t: %llu.%06llu\n",
diff --git a/arch/tile/kernel/setup.c b/arch/tile/kernel/setup.c
index 864eea69556d..f1f579914952 100644
--- a/arch/tile/kernel/setup.c
+++ b/arch/tile/kernel/setup.c
@@ -215,12 +215,11 @@ early_param("mem", setup_mem);  /* compatibility with x86 */
 
 static int __init setup_isolnodes(char *str)
 {
-        char buf[MAX_NUMNODES * 5];
         if (str == NULL || nodelist_parse(str, isolnodes) != 0)
                 return -EINVAL;
 
-        nodelist_scnprintf(buf, sizeof(buf), isolnodes);
-        pr_info("Set isolnodes value to '%s'\n", buf);
+        pr_info("Set isolnodes value to '%*pbl'\n",
+                nodemask_pr_args(&isolnodes));
         return 0;
 }
 early_param("isolnodes", setup_isolnodes);
@@ -1315,11 +1314,9 @@ early_param("disabled_cpus", disabled_cpus);
 
 void __init print_disabled_cpus(void)
 {
-        if (!cpumask_empty(&disabled_map)) {
-                char buf[100];
-                cpulist_scnprintf(buf, sizeof(buf), &disabled_map);
-                pr_info("CPUs not available for Linux: %s\n", buf);
-        }
+        if (!cpumask_empty(&disabled_map))
+                pr_info("CPUs not available for Linux: %*pbl\n",
+                        cpumask_pr_args(&disabled_map));
 }
 
 static void __init setup_cpu_maps(void)
diff --git a/arch/tile/mm/homecache.c b/arch/tile/mm/homecache.c
index 0029b3fb651b..40ca30a9fee3 100644
--- a/arch/tile/mm/homecache.c
+++ b/arch/tile/mm/homecache.c
@@ -115,7 +115,6 @@ void flush_remote(unsigned long cache_pfn, unsigned long cache_control,
         struct cpumask cache_cpumask_copy, tlb_cpumask_copy;
         struct cpumask *cache_cpumask, *tlb_cpumask;
         HV_PhysAddr cache_pa;
-        char cache_buf[NR_CPUS*5], tlb_buf[NR_CPUS*5];
 
         mb();   /* provided just to simplify "magic hypervisor" mode */
 
@@ -149,13 +148,12 @@ void flush_remote(unsigned long cache_pfn, unsigned long cache_control,
                              asids, asidcount);
         if (rc == 0)
                 return;
-        cpumask_scnprintf(cache_buf, sizeof(cache_buf), &cache_cpumask_copy);
-        cpumask_scnprintf(tlb_buf, sizeof(tlb_buf), &tlb_cpumask_copy);
 
-        pr_err("hv_flush_remote(%#llx, %#lx, %p [%s], %#lx, %#lx, %#lx, %p [%s], %p, %d) = %d\n",
-               cache_pa, cache_control, cache_cpumask, cache_buf,
-               (unsigned long)tlb_va, tlb_length, tlb_pgsize,
-               tlb_cpumask, tlb_buf, asids, asidcount, rc);
+        pr_err("hv_flush_remote(%#llx, %#lx, %p [%*pb], %#lx, %#lx, %#lx, %p [%*pb], %p, %d) = %d\n",
+               cache_pa, cache_control, cache_cpumask,
+               cpumask_pr_args(&cache_cpumask_copy),
+               (unsigned long)tlb_va, tlb_length, tlb_pgsize, tlb_cpumask,
+               cpumask_pr_args(&tlb_cpumask_copy), asids, asidcount, rc);
         panic("Unsafe to continue.");
 }
 
diff --git a/arch/tile/mm/init.c b/arch/tile/mm/init.c
index be240cc4978d..ace32d7d3864 100644
--- a/arch/tile/mm/init.c
+++ b/arch/tile/mm/init.c
@@ -353,15 +353,13 @@ static int __init setup_ktext(char *str)
 
         /* Neighborhood ktext pages on specified mask */
         else if (cpulist_parse(str, &ktext_mask) == 0) {
-                char buf[NR_CPUS * 5];
-                cpulist_scnprintf(buf, sizeof(buf), &ktext_mask);
                 if (cpumask_weight(&ktext_mask) > 1) {
                         ktext_small = 1;
-                        pr_info("ktext: using caching neighborhood %s with small pages\n",
-                                buf);
+                        pr_info("ktext: using caching neighborhood %*pbl with small pages\n",
+                                cpumask_pr_args(&ktext_mask));
                 } else {
-                        pr_info("ktext: caching on cpu %s with one huge page\n",
-                                buf);
+                        pr_info("ktext: caching on cpu %*pbl with one huge page\n",
+                                cpumask_pr_args(&ktext_mask));
                 }
         }
 
@@ -492,11 +490,9 @@ static void __init kernel_physical_mapping_init(pgd_t *pgd_base)
                 struct cpumask bad;
                 cpumask_andnot(&bad, &ktext_mask, cpu_possible_mask);
                 cpumask_and(&ktext_mask, &ktext_mask, cpu_possible_mask);
-                if (!cpumask_empty(&bad)) {
-                        char buf[NR_CPUS * 5];
-                        cpulist_scnprintf(buf, sizeof(buf), &bad);
-                        pr_info("ktext: not using unavailable cpus %s\n", buf);
-                }
+                if (!cpumask_empty(&bad))
+                        pr_info("ktext: not using unavailable cpus %*pbl\n",
+                                cpumask_pr_args(&bad));
                 if (cpumask_empty(&ktext_mask)) {
                         pr_warn("ktext: no valid cpus; caching on %d\n",
                                 smp_processor_id());