Fix weird uses of num_online_cpus().

This may be OK in archs with contiguous CPU numbers and without
hotplug CPUs, but it sets a terrible example.

And open-coding it like drivers/scsi/hpsa.c is just weird.

BTRFS has a weird comparison with num_online_cpus() too, but since
BTRFS just screwed up my test machines' root partition, I'm not
touching it :)

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Reported-by: Oleg Drokin <green@linuxhacker.ru>

1 files changed, 3 insertions, 3 deletions

diff --git a/arch/sh/kernel/smp.c b/arch/sh/kernel/smp.c
index fc5acfc93c92..de6be008fc01 100644
--- a/arch/sh/kernel/smp.c
+++ b/arch/sh/kernel/smp.c
@@ -363,7 +363,7 @@ void flush_tlb_mm(struct mm_struct *mm)
                 smp_call_function(flush_tlb_mm_ipi, (void *)mm, 1);
         } else {
                 int i;
-                for (i = 0; i < num_online_cpus(); i++)
+                for_each_online_cpu(i)
                         if (smp_processor_id() != i)
                                 cpu_context(i, mm) = 0;
         }
@@ -400,7 +400,7 @@ void flush_tlb_range(struct vm_area_struct *vma,
                 smp_call_function(flush_tlb_range_ipi, (void *)&fd, 1);
         } else {
                 int i;
-                for (i = 0; i < num_online_cpus(); i++)
+                for_each_online_cpu(i)
                         if (smp_processor_id() != i)
                                 cpu_context(i, mm) = 0;
         }
@@ -443,7 +443,7 @@ void flush_tlb_page(struct vm_area_struct *vma, unsigned long page)
                 smp_call_function(flush_tlb_page_ipi, (void *)&fd, 1);
         } else {
                 int i;
-                for (i = 0; i < num_online_cpus(); i++)
+                for_each_online_cpu(i)
                         if (smp_processor_id() != i)
                                 cpu_context(i, vma->vm_mm) = 0;
         }