]> git.karo-electronics.de Git - karo-tx-linux.git/commitdiff
[S390] avoid warning in show_cpuinfo
authorMartin Schwidefsky <schwidefsky@de.ibm.com>
Mon, 24 Oct 2011 09:46:17 +0000 (11:46 +0200)
committerMartin Schwidefsky <schwidefsky@de.ibm.com>
Mon, 24 Oct 2011 09:46:25 +0000 (11:46 +0200)
The .start function and indirectly the .next function of the show_cpuinfo
sequential operation uses NR_CPUS as limit instead of nr_cpu_ids.
This can cause warnings like this:

WARNING: at /usr/src/linux/include/linux/cpumask.h:107
Process lscpu (pid: 575, task: 000000007deb4338, ksp: 000000007794f588)
Krnl PSW : 0704000180000000 0000000000106db4 (show_cpuinfo+0x108/0x234)
           R:0 T:1 IO:1 EX:1 Key:0 M:1 W:0 P:0 AS:0 CC:0 PM:0 EA:3
Krnl GPRS: 0000000000000003 0000000000791988 000000000071b478 0000000000000004
           0000000000000001 0000000000000000 000000007d139500 0000000000000400
           0000000000000000 000000000070e24c 000000007d48d600 0000000000000005
           000000007d48d600 00000000004dfa10 0000000000106cf8 000000007794fcc0
Krnl Code: 0000000000106da895001000           cli     0(%r1),0
           0000000000106daca774ffac           brc     7,106d04
           0000000000106db0a7f40001           brc     15,106db2
          >0000000000106db492011000           mvi     0(%r1),1
           0000000000106db8a7f4ffa6           brc     15,106d04
           0000000000106dbcc0e5000065b4       brasl   %r14,113924
           0000000000106dc2c09000303a45       larl    %r9,70e24c
           0000000000106dc8c020001eefd4       larl    %r2,4e4d70

Replacing NR_CPUS with nr_cpu_ids fixes it.

Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
arch/s390/kernel/processor.c

index 311e9d7128884795733f2915448f98b62f60dc2e..6e0073e43f5448e8910fae02e973aecfdf1594f8 100644 (file)
@@ -74,7 +74,7 @@ static int show_cpuinfo(struct seq_file *m, void *v)
 
 static void *c_start(struct seq_file *m, loff_t *pos)
 {
-       return *pos < NR_CPUS ? (void *)((unsigned long) *pos + 1) : NULL;
+       return *pos < nr_cpu_ids ? (void *)((unsigned long) *pos + 1) : NULL;
 }
 
 static void *c_next(struct seq_file *m, void *v, loff_t *pos)