diff options
author | marius <marius@FreeBSD.org> | 2011-05-14 23:20:14 +0000 |
---|---|---|
committer | marius <marius@FreeBSD.org> | 2011-05-14 23:20:14 +0000 |
commit | e0bdfaa331e0646b286138c0a1ebde578634f499 (patch) | |
tree | 1db9dfcfab38270506ed7a45e53652b2cb131025 /sys/sparc64 | |
parent | c5a5c48e70de18e3c28cb0005fc2d9ab6a83a8f7 (diff) | |
download | FreeBSD-src-e0bdfaa331e0646b286138c0a1ebde578634f499.zip FreeBSD-src-e0bdfaa331e0646b286138c0a1ebde578634f499.tar.gz |
Fix yet another inversion in the logic by applying the x86 version of this,
which avoids CPU_EMPTY() in the first place.
Do I get a beer or something for every inversion I find?
Diffstat (limited to 'sys/sparc64')
-rw-r--r-- | sys/sparc64/sparc64/mp_machdep.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/sparc64/sparc64/mp_machdep.c b/sys/sparc64/sparc64/mp_machdep.c index a314eb0..7edc3a3 100644 --- a/sys/sparc64/sparc64/mp_machdep.c +++ b/sys/sparc64/sparc64/mp_machdep.c @@ -573,8 +573,8 @@ spitfire_ipi_selected(cpuset_t cpus, u_long d0, u_long d1, u_long d2) { u_int cpu; - while (CPU_EMPTY(&cpus)) { - cpu = cpusetobj_ffs(&cpus) - 1; + while ((cpu = cpusetobj_ffs(&cpus)) != 0) { + cpu--; CPU_CLR(cpu, &cpus); spitfire_ipi_single(cpu, d0, d1, d2); } |