Messages in this thread Patch in this message |  | | | Date | Fri, 3 Jul 2009 18:31:25 GMT | | From | tip-bot for Ingo Molnar <> | | Subject | [tip:perfcounters/urgent] x86: atomic64: Optimize CMPXCHG8B sequences to not use the LOCK prefix |
Commit-ID: 07aa81b2169a2cb5e62ad239fd8055ef484e4f79 Gitweb: http://git.kernel.org/tip/07aa81b2169a2cb5e62ad239fd8055ef484e4f79 Author: Ingo Molnar <mingo@elte.hu> AuthorDate: Fri, 3 Jul 2009 20:15:58 +0200 Committer: Ingo Molnar <mingo@elte.hu> CommitDate: Fri, 3 Jul 2009 20:24:00 +0200
x86: atomic64: Optimize CMPXCHG8B sequences to not use the LOCK prefix
Arjan noted that the CMPXCHG8B instruction has an implicit lock prefix, so there's no need to add the prefix explicitly.
This gives us smaller code and smaller alternative-instructions tables.
Cc: Linus Torvalds <torvalds@linux-foundation.org> Cc: Eric Dumazet <eric.dumazet@gmail.com> Cc: Peter Zijlstra <a.p.zijlstra@chello.nl> Cc: Mike Galbraith <efault@gmx.de> Cc: Paul Mackerras <paulus@samba.org> 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: <20090703111033.6e9627de@infradead.org> Signed-off-by: Ingo Molnar <mingo@elte.hu>
--- arch/x86/include/asm/atomic_32.h | 2 +- arch/x86/lib/atomic64_32.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/x86/include/asm/atomic_32.h b/arch/x86/include/asm/atomic_32.h index de89959..130f4ab 100644 --- a/arch/x86/include/asm/atomic_32.h +++ b/arch/x86/include/asm/atomic_32.h @@ -302,7 +302,7 @@ static inline u64 atomic64_read(atomic64_t *ptr) asm volatile( "mov %%ebx, %%eax\n\t" "mov %%ecx, %%edx\n\t" - LOCK_PREFIX "cmpxchg8b %1\n" + "cmpxchg8b %1\n" : "=&A" (res) : "m" (*ptr) ); diff --git a/arch/x86/lib/atomic64_32.c b/arch/x86/lib/atomic64_32.c index d79f4ac..e4a6671 100644 --- a/arch/x86/lib/atomic64_32.c +++ b/arch/x86/lib/atomic64_32.c @@ -12,7 +12,7 @@ static noinline u64 cmpxchg8b(u64 *ptr, u64 old, u64 new) u32 high = new >> 32; asm volatile( - LOCK_PREFIX "cmpxchg8b %1\n" + "cmpxchg8b %1\n" : "+A" (old), "+m" (*ptr) : "b" (low), "c" (high) );
|  |