sh: define __smp_xxx, fix smp_store_mb for !SMP
authorMichael S. Tsirkin <mst@redhat.com>
Sun, 27 Dec 2015 13:04:42 +0000 (15:04 +0200)
committerMichael S. Tsirkin <mst@redhat.com>
Tue, 12 Jan 2016 18:46:57 +0000 (20:46 +0200)
sh variant of smp_store_mb() calls xchg() on !SMP which is stronger than
implied by both the name and the documentation.

define __smp_store_mb instead: code in asm-generic/barrier.h
will then define smp_store_mb correctly depending on
CONFIG_SMP.

Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Acked-by: Arnd Bergmann <arnd@arndb.de>
Acked-by: Peter Zijlstra (Intel) <peterz@infradead.org>
arch/sh/include/asm/barrier.h

index bf91037db4e01f66dcf6a2e84536279dcbf028c1..f887c6465a821b7b96a2a300268f077afd70ae5a 100644 (file)
@@ -32,7 +32,8 @@
 #define ctrl_barrier() __asm__ __volatile__ ("nop;nop;nop;nop;nop;nop;nop;nop")
 #endif
 
-#define smp_store_mb(var, value) do { (void)xchg(&var, value); } while (0)
+#define __smp_store_mb(var, value) do { (void)xchg(&var, value); } while (0)
+#define smp_store_mb(var, value) __smp_store_mb(var, value)
 
 #include <asm-generic/barrier.h>