* Re: [PATCH] atomic_read: Use ACCESS_ONCE() instead of cast to volatile
[not found] <1411482607-20948-1-git-send-email-bobby.prani@gmail.com>
@ 2014-09-23 15:15 ` Peter Zijlstra
2014-09-23 15:40 ` Jesper Nilsson
` (2 subsequent siblings)
3 siblings, 0 replies; 4+ messages in thread
From: Peter Zijlstra @ 2014-09-23 15:15 UTC (permalink / raw)
To: Pranith Kumar
Cc: Richard Henderson, Ivan Kokshaysky, Matt Turner, Russell King,
Catalin Marinas, Will Deacon, Haavard Skinnemoen,
Hans-Christian Egtvedt, Mikael Starvik, Jesper Nilsson,
David Howells, Tony Luck, Fenghua Yu, Hirokazu Takata,
Geert Uytterhoeven, Ralf Baechle, James E.J. Bottomley,
Helge Deller, David S. Miller
On Tue, Sep 23, 2014 at 10:29:50AM -0400, Pranith Kumar wrote:
> Use the much reader friendly ACCESS_ONCE() instead of the cast to volatile. This
> is purely a style change.
>
> Signed-off-by: Pranith Kumar <bobby.prani@gmail.com>
Looks good to me; I can route it through the locking tree if there's no
objections.
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] atomic_read: Use ACCESS_ONCE() instead of cast to volatile
[not found] <1411482607-20948-1-git-send-email-bobby.prani@gmail.com>
2014-09-23 15:15 ` [PATCH] atomic_read: Use ACCESS_ONCE() instead of cast to volatile Peter Zijlstra
@ 2014-09-23 15:40 ` Jesper Nilsson
2014-09-23 19:43 ` Hans-Christian Egtvedt
2014-09-23 23:56 ` Max Filippov
3 siblings, 0 replies; 4+ messages in thread
From: Jesper Nilsson @ 2014-09-23 15:40 UTC (permalink / raw)
To: Pranith Kumar
Cc: Richard Henderson, Ivan Kokshaysky, Matt Turner, Russell King,
Catalin Marinas, Will Deacon, Haavard Skinnemoen,
Hans-Christian Egtvedt, Mikael Starvik, Jesper Nilsson,
David Howells, Tony Luck, Fenghua Yu, Hirokazu Takata,
Geert Uytterhoeven, Ralf Baechle, James E.J. Bottomley,
Helge Deller, David S. Miller, Thomas
On Tue, Sep 23, 2014 at 04:29:50PM +0200, Pranith Kumar wrote:
> Use the much reader friendly ACCESS_ONCE() instead of the cast to volatile. This
> is purely a style change.
>
> Signed-off-by: Pranith Kumar <bobby.prani@gmail.com>
> ---
> 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 <jesper.nilsson@axis.com>
/^JN - Jesper Nilsson
--
Jesper Nilsson -- jesper.nilsson@axis.com
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] atomic_read: Use ACCESS_ONCE() instead of cast to volatile
[not found] <1411482607-20948-1-git-send-email-bobby.prani@gmail.com>
2014-09-23 15:15 ` [PATCH] atomic_read: Use ACCESS_ONCE() instead of cast to volatile Peter Zijlstra
2014-09-23 15:40 ` Jesper Nilsson
@ 2014-09-23 19:43 ` Hans-Christian Egtvedt
2014-09-23 23:56 ` Max Filippov
3 siblings, 0 replies; 4+ messages in thread
From: Hans-Christian Egtvedt @ 2014-09-23 19:43 UTC (permalink / raw)
To: Pranith Kumar
Cc: Richard Henderson, Ivan Kokshaysky, Matt Turner, Russell King,
Catalin Marinas, Will Deacon, Haavard Skinnemoen, Mikael Starvik,
Jesper Nilsson, David Howells, Tony Luck, Fenghua Yu,
Hirokazu Takata, Geert Uytterhoeven, Ralf Baechle,
James E.J. Bottomley, Helge Deller, David S. Miller,
Thomas Gleixner, Ingo
Around Tue 23 Sep 2014 10:29:50 -0400 or thereabout, Pranith Kumar wrote:
> Use the much reader friendly ACCESS_ONCE() instead of the cast to volatile. This
> is purely a style change.
>
> Signed-off-by: Pranith Kumar <bobby.prani@gmail.com>
> ---
> 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.
Acked-by: Hans-Christian Egtvedt <egtvedt@samfundet.no>
> 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
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] atomic_read: Use ACCESS_ONCE() instead of cast to volatile
[not found] <1411482607-20948-1-git-send-email-bobby.prani@gmail.com>
` (2 preceding siblings ...)
2014-09-23 19:43 ` Hans-Christian Egtvedt
@ 2014-09-23 23:56 ` Max Filippov
3 siblings, 0 replies; 4+ messages in thread
From: Max Filippov @ 2014-09-23 23:56 UTC (permalink / raw)
To: Pranith Kumar
Cc: Richard Henderson, Ivan Kokshaysky, Matt Turner, Russell King,
Catalin Marinas, Will Deacon, Haavard Skinnemoen,
Hans-Christian Egtvedt, Mikael Starvik, Jesper Nilsson,
David Howells, Tony Luck, Fenghua Yu, Hirokazu Takata,
Geert Uytterhoeven, Ralf Baechle, James E.J. Bottomley,
Helge Deller, David S. Miller
On Tue, Sep 23, 2014 at 6:29 PM, Pranith Kumar <bobby.prani@gmail.com> wrote:
> Use the much reader friendly ACCESS_ONCE() instead of the cast to volatile. This
> is purely a style change.
>
> Signed-off-by: Pranith Kumar <bobby.prani@gmail.com>
> ---
> 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 <jcmvbkbc@gmail.com>
--
Thanks.
-- Max
^ permalink raw reply [flat|nested] 4+ messages in thread