diff options
author | Pranith Kumar <bobby.prani@gmail.com> | 2014-09-23 10:29:50 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2014-10-03 00:06:23 -0400 |
commit | 2291059c852706c6f5ffb400366042b7625066cd (patch) | |
tree | 7827b5e7992bc0d3533d5832773c6be2f4221782 /arch/arm | |
parent | da4c54457e6d8262423aded4bcbccc4103049506 (diff) |
locking,arch: Use ACCESS_ONCE() instead of cast to volatile in atomic_read()
Use the much more reader friendly ACCESS_ONCE() instead of the cast to volatile.
This is purely a stylistic change.
Signed-off-by: Pranith Kumar <bobby.prani@gmail.com>
Acked-by: Jesper Nilsson <jesper.nilsson@axis.com>
Acked-by: Hans-Christian Egtvedt <egtvedt@samfundet.no>
Acked-by: Max Filippov <jcmvbkbc@gmail.com>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: linux-arch@vger.kernel.org
Link: http://lkml.kernel.org/r/1411482607-20948-1-git-send-email-bobby.prani@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Diffstat (limited to 'arch/arm')
-rw-r--r-- | arch/arm/include/asm/atomic.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/arm/include/asm/atomic.h b/arch/arm/include/asm/atomic.h index 832f1cdfcd6a..e22c11970b7b 100644 --- a/arch/arm/include/asm/atomic.h +++ b/arch/arm/include/asm/atomic.h | |||
@@ -27,7 +27,7 @@ | |||
27 | * strex/ldrex monitor on some implementations. The reason we can use it for | 27 | * strex/ldrex monitor on some implementations. The reason we can use it for |
28 | * atomic_set() is the clrex or dummy strex done on every exception return. | 28 | * atomic_set() is the clrex or dummy strex done on every exception return. |
29 | */ | 29 | */ |
30 | #define atomic_read(v) (*(volatile int *)&(v)->counter) | 30 | #define atomic_read(v) ACCESS_ONCE((v)->counter) |
31 | #define atomic_set(v,i) (((v)->counter) = (i)) | 31 | #define atomic_set(v,i) (((v)->counter) = (i)) |
32 | 32 | ||
33 | #if __LINUX_ARM_ARCH__ >= 6 | 33 | #if __LINUX_ARM_ARCH__ >= 6 |