powerpc/xmon: Fix printing of set of CPUs in xmon
authorPaul Mackerras <paulus@samba.org>
Tue, 3 Sep 2013 10:16:23 +0000 (20:16 +1000)
committerBenjamin Herrenschmidt <benh@kernel.crashing.org>
Thu, 5 Sep 2013 07:29:19 +0000 (17:29 +1000)
Commit 24ec2125f3 ("powerpc/xmon: Use cpumask iterator to avoid warning")
replaced a loop from 0 to NR_CPUS-1 with a for_each_possible_cpu() loop,
which means that if the last possible cpu is in xmon, we print the
wrong value for the end of the range.  For example, if 4 cpus are
possible, NR_CPUS is 128, and all cpus are in xmon, we print "0-7f"
rather than "0-3".  The code also assumes that the set of possible
cpus is contiguous, which may not necessarily be true.

This fixes the code to check explicitly for contiguity, and to print
the ending value correctly.

Signed-off-by: Paul Mackerras <paulus@samba.org>
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
arch/powerpc/xmon/xmon.c

index 9f3655be695d3c3f57eadced550fcf89ae9e6266..af9d3469fb9933e4f3d424f01e3230ddeb580c67 100644 (file)
@@ -972,27 +972,27 @@ static void bootcmds(void)
 static int cpu_cmd(void)
 {
 #ifdef CONFIG_SMP
-       unsigned long cpu;
+       unsigned long cpu, first_cpu, last_cpu;
        int timeout;
-       int count;
 
        if (!scanhex(&cpu)) {
                /* print cpus waiting or in xmon */
                printf("cpus stopped:");
-               count = 0;
+               last_cpu = first_cpu = NR_CPUS;
                for_each_possible_cpu(cpu) {
                        if (cpumask_test_cpu(cpu, &cpus_in_xmon)) {
-                               if (count == 0)
-                                       printf(" %x", cpu);
-                               ++count;
-                       } else {
-                               if (count > 1)
-                                       printf("-%x", cpu - 1);
-                               count = 0;
+                               if (cpu == last_cpu + 1) {
+                                       last_cpu = cpu;
+                               } else {
+                                       if (last_cpu != first_cpu)
+                                               printf("-%lx", last_cpu);
+                                       last_cpu = first_cpu = cpu;
+                                       printf(" %lx", cpu);
+                               }
                        }
                }
-               if (count > 1)
-                       printf("-%x", NR_CPUS - 1);
+               if (last_cpu != first_cpu)
+                       printf("-%lx", last_cpu);
                printf("\n");
                return 0;
        }