Discussion:
[PATCH] atomic_read: Use ACCESS_ONCE() instead of cast to volatile
Peter Zijlstra
2014-09-23 15:15:09 UTC
Permalink
Use the much reader friendly ACCESS_ONCE() instead of the cast to volatile. This
is purely a style change.
Looks good to me; I can route it through the locking tree if there's no
objections.
Jesper Nilsson
2014-09-23 15:40:16 UTC
Permalink
Use the much reader friendly ACCESS_ONCE() instead of the cast to volatile. This
is purely a style change.
---
arch/alpha/include/asm/atomic.h | 4 ++--
arch/arm/include/asm/atomic.h | 2 +-
arch/arm64/include/asm/atomic.h | 4 ++--
arch/avr32/include/asm/atomic.h | 2 +-
arch/cris/include/asm/atomic.h | 2 +-
For the CRIS parts:

Acked-by: Jesper Nilsson <***@axis.com>

/^JN - Jesper Nilsson
--
Jesper Nilsson -- ***@axis.com
Hans-Christian Egtvedt
2014-09-23 19:43:27 UTC
Permalink
Use the much reader friendly ACCESS_ONCE() instead of the cast to volatile. This
is purely a style change.
---
arch/alpha/include/asm/atomic.h | 4 ++--
arch/arm/include/asm/atomic.h | 2 +-
arch/arm64/include/asm/atomic.h | 4 ++--
arch/avr32/include/asm/atomic.h | 2 +-
For the AVR32 related part.
arch/cris/include/asm/atomic.h | 2 +-
arch/frv/include/asm/atomic.h | 2 +-
arch/ia64/include/asm/atomic.h | 4 ++--
arch/m32r/include/asm/atomic.h | 2 +-
arch/m68k/include/asm/atomic.h | 2 +-
arch/mips/include/asm/atomic.h | 4 ++--
arch/parisc/include/asm/atomic.h | 4 ++--
arch/sh/include/asm/atomic.h | 2 +-
arch/sparc/include/asm/atomic_32.h | 2 +-
arch/sparc/include/asm/atomic_64.h | 4 ++--
arch/x86/include/asm/atomic.h | 2 +-
arch/x86/include/asm/atomic64_64.h | 2 +-
arch/xtensa/include/asm/atomic.h | 2 +-
include/asm-generic/atomic.h | 2 +-
18 files changed, 24 insertions(+), 24 deletions(-)
<snipp diff>
--
BR, HcE
Max Filippov
2014-09-23 23:56:39 UTC
Permalink
Use the much reader friendly ACCESS_ONCE() instead of the cast to volatile. This
is purely a style change.
---
arch/alpha/include/asm/atomic.h | 4 ++--
arch/arm/include/asm/atomic.h | 2 +-
arch/arm64/include/asm/atomic.h | 4 ++--
arch/avr32/include/asm/atomic.h | 2 +-
arch/cris/include/asm/atomic.h | 2 +-
arch/frv/include/asm/atomic.h | 2 +-
arch/ia64/include/asm/atomic.h | 4 ++--
arch/m32r/include/asm/atomic.h | 2 +-
arch/m68k/include/asm/atomic.h | 2 +-
arch/mips/include/asm/atomic.h | 4 ++--
arch/parisc/include/asm/atomic.h | 4 ++--
arch/sh/include/asm/atomic.h | 2 +-
arch/sparc/include/asm/atomic_32.h | 2 +-
arch/sparc/include/asm/atomic_64.h | 4 ++--
arch/x86/include/asm/atomic.h | 2 +-
arch/x86/include/asm/atomic64_64.h | 2 +-
arch/xtensa/include/asm/atomic.h | 2 +-
include/asm-generic/atomic.h | 2 +-
18 files changed, 24 insertions(+), 24 deletions(-)
For xtensa:
Acked-by: Max Filippov <***@gmail.com>
--
Thanks.
-- Max
Loading...