diff options
| author | Steven Rostedt (Red Hat) <rostedt@goodmis.org> | 2015-11-03 16:19:02 -0500 |
|---|---|---|
| committer | Steven Rostedt <rostedt@goodmis.org> | 2015-11-03 16:19:02 -0500 |
| commit | 54ed1444052467044e9e01334ac8123dd6345211 (patch) | |
| tree | e435736db73b7c7fede3ffa953641aaa8c576615 /kernel/trace | |
| parent | 919cd9799936843d0af4f0904a3e39e70294c4d8 (diff) | |
| download | linux-54ed1444052467044e9e01334ac8123dd6345211.tar.gz linux-54ed1444052467044e9e01334ac8123dd6345211.tar.bz2 linux-54ed1444052467044e9e01334ac8123dd6345211.zip | |
ring_buffer: Remove unneeded smp_wmb() before wakeup of reader benchmark
wake_up_process() has a memory barrier before doing anything, thus adding a
memory barrier before calling it is redundant.
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'kernel/trace')
| -rw-r--r-- | kernel/trace/ring_buffer_benchmark.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/kernel/trace/ring_buffer_benchmark.c b/kernel/trace/ring_buffer_benchmark.c index 9e00fd178226..6df9a83e20d7 100644 --- a/kernel/trace/ring_buffer_benchmark.c +++ b/kernel/trace/ring_buffer_benchmark.c @@ -288,8 +288,6 @@ static void ring_buffer_producer(void) /* the completions must be visible before the finish var */ smp_wmb(); reader_finish = 1; - /* finish var visible before waking up the consumer */ - smp_wmb(); wake_up_process(consumer); wait_for_completion(&read_done); } |