diff mbox

[2/2] Fail compiles if no platform specific memory barriers exist

Message ID 8dde3b93ddc1251c3487438331590acb099c6360.1465310573.git.root@r9.asicdesigners.com (mailing list archive)
State Accepted
Headers show

Commit Message

Steve Wise June 7, 2016, 2:42 p.m. UTC
Currently there is a "generic" implementation for the memory barrier
macros in arch.h.  These turned out to be insuffient for ARM64 causing
memory corruption problems when doing RDMA operations.  So going forward,
fail a compile on a platform w/o platform-specific memory barrier macros.

Signed-off-by: Steve Wise <swise@opengridcomputing.com>
---
 include/infiniband/arch.h |    7 +------
 1 files changed, 1 insertions(+), 6 deletions(-)
diff mbox

Patch

diff --git a/include/infiniband/arch.h b/include/infiniband/arch.h
index c31dd0a..e35ecf0 100644
--- a/include/infiniband/arch.h
+++ b/include/infiniband/arch.h
@@ -132,12 +132,7 @@  static inline uint64_t ntohll(uint64_t x) { return x; }
 
 #else
 
-#warning No architecture specific defines found.  Using generic implementation.
-
-#define mb()	 asm volatile("" ::: "memory")
-#define rmb()	 mb()
-#define wmb()	 mb()
-#define wc_wmb() wmb()
+#error No architecture specific memory barrier defines found!
 
 #endif