diff options
author | Mark Rutland <mark.rutland@arm.com> | 2017-11-27 05:38:21 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2017-12-12 07:22:09 -0500 |
commit | f971e511cb7d6f1b3730248cf2967d3ccdd8874c (patch) | |
tree | a887659c8ae5dc3da4842516f6e7eecf5bf0bc4c | |
parent | e966eaeeb623f09975ef362c2866fae6f86844f9 (diff) |
tools/perf: Convert ACCESS_ONCE() to READ_ONCE()
Recently there was a treewide conversion of ACCESS_ONCE() to
{READ,WRITE}_ONCE(), but a new use was introduced concurrently by
commit:
1695849735752d2a ("perf mmap: Move perf_mmap and methods to separate mmap.[ch] files")
Let's convert this over to READ_ONCE() so that we can remove the
ACCESS_ONCE() definitions in subsequent patches.
Tested-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Signed-off-by: Mark Rutland <mark.rutland@arm.com>
Reviewed-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Joe Perches <joe@perches.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: apw@canonical.com
Link: http://lkml.kernel.org/r/20171127103824.36526-2-mark.rutland@arm.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
-rw-r--r-- | tools/perf/util/mmap.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/perf/util/mmap.h b/tools/perf/util/mmap.h index efd78b827b05..3a5cb5a6e94a 100644 --- a/tools/perf/util/mmap.h +++ b/tools/perf/util/mmap.h | |||
@@ -70,7 +70,7 @@ void perf_mmap__read_catchup(struct perf_mmap *md); | |||
70 | static inline u64 perf_mmap__read_head(struct perf_mmap *mm) | 70 | static inline u64 perf_mmap__read_head(struct perf_mmap *mm) |
71 | { | 71 | { |
72 | struct perf_event_mmap_page *pc = mm->base; | 72 | struct perf_event_mmap_page *pc = mm->base; |
73 | u64 head = ACCESS_ONCE(pc->data_head); | 73 | u64 head = READ_ONCE(pc->data_head); |
74 | rmb(); | 74 | rmb(); |
75 | return head; | 75 | return head; |
76 | } | 76 | } |