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/