summaryrefslogtreecommitdiffstats
path: root/sys/amd64/include
diff options
context:
space:
mode:
authorattilio <attilio@FreeBSD.org>2012-10-09 14:32:30 +0000
committerattilio <attilio@FreeBSD.org>2012-10-09 14:32:30 +0000
commit6997194551ffae05d71ea650af49c4551d753a1a (patch)
tree18e9f31a3018a88b46e78d76a45d0953dea5276b /sys/amd64/include
parent6a02d06d1d81ffda0499310925ef854fd237f382 (diff)
downloadFreeBSD-src-6997194551ffae05d71ea650af49c4551d753a1a.zip
FreeBSD-src-6997194551ffae05d71ea650af49c4551d753a1a.tar.gz
Add an unified macro to deny ability from the compiler to reorder
instruction loads/stores at its will. The macro __compiler_membar() is currently supported for both gcc and clang, but kernel compilation will fail otherwise. Reviewed by: bde, kib Discussed with: dim, theraven MFC after: 2 weeks
Diffstat (limited to 'sys/amd64/include')
-rw-r--r--sys/amd64/include/atomic.h4
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/amd64/include/atomic.h b/sys/amd64/include/atomic.h
index 99a94b7..91c33e6 100644
--- a/sys/amd64/include/atomic.h
+++ b/sys/amd64/include/atomic.h
@@ -226,7 +226,7 @@ atomic_fetchadd_long(volatile u_long *p, u_long v)
static __inline void \
atomic_store_rel_##TYPE(volatile u_##TYPE *p, u_##TYPE v)\
{ \
- __asm __volatile("" : : : "memory"); \
+ __compiler_membar(); \
*p = v; \
} \
struct __hack
@@ -240,7 +240,7 @@ atomic_load_acq_##TYPE(volatile u_##TYPE *p) \
u_##TYPE tmp; \
\
tmp = *p; \
- __asm __volatile("" : : : "memory"); \
+ __compiler_membar(); \
return (tmp); \
} \
struct __hack
OpenPOWER on IntegriCloud