x86: atomic64: Code atomic(64)_read and atomic(64)_set in C not CPP
authorPaul Mackerras <paulus@samba.org>
Wed, 1 Jul 2009 22:57:12 +0000 (08:57 +1000)
committerIngo Molnar <mingo@elte.hu>
Fri, 3 Jul 2009 12:42:39 +0000 (14:42 +0200)
commit8e049ef054f1cc765f05f13e1396bb9a17c19e66
tree99a8879e1415bf4c6ceaa0eb6a3372fa8842b908
parent199e23780a7e75c63a9e3d1108804e3af450ea3e
x86: atomic64: Code atomic(64)_read and atomic(64)_set in C not CPP

Occasionally we get bugs where atomic_read or atomic_set are
used on atomic64_t variables or vice versa.  These bugs don't
generate warnings on x86 because atomic_read and atomic_set are
coded as macros rather than C functions, so we don't get any
type-checking on their arguments; similarly for atomic64_read
and atomic64_set in 64-bit kernels.

This converts them to C functions so that the arguments are
type-checked and bugs like this will get caught more easily. It
also converts atomic_cmpxchg and atomic_xchg, and
atomic64_cmpxchg and atomic64_xchg on 64-bit, so we get
type-checking on their arguments too.

Compiling a typical 64-bit x86 config, this generates no new
warnings, and the vmlinux text is 86 bytes smaller.

Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: David Howells <dhowells@redhat.com>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Arnd Bergmann <arnd@arndb.de>
LKML-Reference: <alpine.LFD.2.01.0907021653030.3210@localhost.localdomain>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
arch/x86/include/asm/atomic_32.h
arch/x86/include/asm/atomic_64.h