diff options
author | hawkes@sgi.com <hawkes@sgi.com> | 2005-10-13 15:01:18 -0400 |
---|---|---|
committer | Tony Luck <tony.luck@intel.com> | 2005-10-25 18:12:05 -0400 |
commit | ddf6d0a00cbb4ad6d4fb4200cc911bb9389174c1 (patch) | |
tree | b9b426b6d618685fe69a09f62a081fa210204abc | |
parent | dc565b525d4b7091a3abb6616d210c8a896a11d7 (diff) |
[IA64] another place to use for_each_cpu_mask() in arch/ia64
In arch/ia64 change the explicit use of a for-loop using NR_CPUS into the
general for_each_online_cpu() construct. This widens the scope of potential
future optimizations of the general constructs, as well as takes advantage
of the existing optimizations of first_cpu() and next_cpu(), which is
advantageous when the true CPU count is much smaller than NR_CPUS.
Signed-off-by: John Hawkes <hawkes@sgi.com>
Signed-off-by: Tony Luck <tony.luck@intel.com>
-rw-r--r-- | arch/ia64/kernel/mca.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/arch/ia64/kernel/mca.c b/arch/ia64/kernel/mca.c index d0a5106fba24..52c47da17246 100644 --- a/arch/ia64/kernel/mca.c +++ b/arch/ia64/kernel/mca.c | |||
@@ -508,9 +508,7 @@ ia64_mca_wakeup_all(void) | |||
508 | int cpu; | 508 | int cpu; |
509 | 509 | ||
510 | /* Clear the Rendez checkin flag for all cpus */ | 510 | /* Clear the Rendez checkin flag for all cpus */ |
511 | for(cpu = 0; cpu < NR_CPUS; cpu++) { | 511 | for_each_online_cpu(cpu) { |
512 | if (!cpu_online(cpu)) | ||
513 | continue; | ||
514 | if (ia64_mc_info.imi_rendez_checkin[cpu] == IA64_MCA_RENDEZ_CHECKIN_DONE) | 512 | if (ia64_mc_info.imi_rendez_checkin[cpu] == IA64_MCA_RENDEZ_CHECKIN_DONE) |
515 | ia64_mca_wakeup(cpu); | 513 | ia64_mca_wakeup(cpu); |
516 | } | 514 | } |