diff options
| author | avg <avg@FreeBSD.org> | 2010-12-17 18:22:50 +0000 |
|---|---|---|
| committer | avg <avg@FreeBSD.org> | 2010-12-17 18:22:50 +0000 |
| commit | 34b3952bddf23043b6c6c67cc355eaa6ca2e50ee (patch) | |
| tree | fa7e04fa5d8f504aa1b5050820fbd8f031d1730e /sys/cddl | |
| parent | 1b88c874083589a9b9de4a78a18bb8586b69f3d8 (diff) | |
| download | FreeBSD-src-34b3952bddf23043b6c6c67cc355eaa6ca2e50ee.zip FreeBSD-src-34b3952bddf23043b6c6c67cc355eaa6ca2e50ee.tar.gz | |
cyclic xcall: use smp_no_rendevous_barrier as setup function parameter
In this case we call target function only on a single CPU and do not
need any synchronization at the setup stage.
It's a bit non-obvious but setup function of NULL means that
smp_rendezvous_cpus waits for all CPUs to arrive at the rendezvous
point, but without doing any actual setup. While using
smp_no_rendevous_barrier means that each CPU proceeds on its own
schedule without any synchronization whatsoever.
MFC after: 3 weeks
Diffstat (limited to 'sys/cddl')
| -rw-r--r-- | sys/cddl/dev/cyclic/i386/cyclic_machdep.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/cddl/dev/cyclic/i386/cyclic_machdep.c b/sys/cddl/dev/cyclic/i386/cyclic_machdep.c index 1d0864a..1c18fc1 100644 --- a/sys/cddl/dev/cyclic/i386/cyclic_machdep.c +++ b/sys/cddl/dev/cyclic/i386/cyclic_machdep.c @@ -122,6 +122,6 @@ static void reprogram(cyb_arg_t arg, hrtime_t exp) static void xcall(cyb_arg_t arg, cpu_t *c, cyc_func_t func, void *param) { - smp_rendezvous_cpus((cpumask_t) (1 << c->cpuid), NULL, - func, smp_no_rendevous_barrier, param); + smp_rendezvous_cpus((cpumask_t) (1 << c->cpuid), + smp_no_rendevous_barrier, func, smp_no_rendevous_barrier, param); } |
