diff options
| author | Andrew Morton <akpm@osdl.org> | 2004-01-18 18:30:55 -0800 |
|---|---|---|
| committer | Linus Torvalds <torvalds@home.osdl.org> | 2004-01-18 18:30:55 -0800 |
| commit | 012061cc70e9c7e0d794a3e609fd79fdf9f5267d (patch) | |
| tree | a1a1cee6d1e7b4576a7ee148e297c04936388402 /kernel/timer.c | |
| parent | 90c7f71962cc5aa0bf128ce68b9a8607176558c4 (diff) | |
[PATCH] Use for_each_cpu() Where It's Meant To Be
From: Rusty Russell <rusty@rustcorp.com.au>
Some places use cpu_online() where they should be using cpu_possible, most
commonly for tallying statistics. This makes no difference without hotplug
CPU.
Use the for_each_cpu() macro in those places, providing good examples (and
making the external hotplug CPU patch smaller).
Some places use cpu_online() where they should be using cpu_possible, most
commonly for tallying statistics. This makes no difference without hotplug
CPU.
Use the for_each_cpu() macro in those places, providing good examples (and
making the external hotplug CPU patch smaller).
Diffstat (limited to 'kernel/timer.c')
| -rw-r--r-- | kernel/timer.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/kernel/timer.c b/kernel/timer.c index a9c8c1958b6a..51df7364331c 100644 --- a/kernel/timer.c +++ b/kernel/timer.c @@ -332,10 +332,7 @@ int del_timer_sync(struct timer_list *timer) del_again: ret += del_timer(timer); - for (i = 0; i < NR_CPUS; i++) { - if (!cpu_online(i)) - continue; - + for_each_cpu(i) { base = &per_cpu(tvec_bases, i); if (base->running_timer == timer) { while (base->running_timer == timer) { |