On Tue, 2008-01-29 at 22:15 -0500, Steven Rostedt wrote:

> +static void notrace __wakeup_reset(struct tracing_trace *tr)
> +{
> +     struct tracing_trace_cpu *data;
> +     int cpu;
> +
> +     assert_spin_locked(&wakeup_lock);
> +
> +     for_each_possible_cpu(cpu) {
> +             data = tr->data[cpu];
> +             tracing_reset(data);
> +     }
> +
> +     wakeup_cpu = -1;
> +     wakeup_prio = -1;
> +     if (wakeup_task) {
> +             put_task_struct(wakeup_task);
> +             tracing_stop_function_trace();
> +     }
> +
> +     wakeup_task = NULL;
> +
> +     /*
> +      * Don't let the trace_enabled = 1 show up before
> +      * the wakeup_task is reset.
> +      */
> +     smp_wmb();
> +}

Another un-balanced barrier.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to