Mirror the same in bsd_zero.hpp

Author: shipilev

src/hotspot/os_cpu/bsd_zero/orderAccess_bsd_zero.hpp

@@ -28,8 +28,6 @@
 
 // Included in orderAccess.hpp header file.
 
-#ifdef ARM
-
 /*
  * ARM Kernel helper for memory barrier.
  * Using __asm __volatile ("":::"memory") does not work reliable on ARM
@@ -39,28 +37,32 @@
 typedef void (__kernel_dmb_t) (void);
 #define __kernel_dmb (*(__kernel_dmb_t *) 0xffff0fa0)
 
-#define FULL_MEM_BARRIER __kernel_dmb()
 #define LIGHT_MEM_BARRIER __kernel_dmb()
+#define FULL_MEM_BARRIER  __kernel_dmb()
 
-#else // ARM
-
-#define FULL_MEM_BARRIER __sync_synchronize()
-
-#ifdef PPC
+#elif defined(PPC) // ----------------------------------------------------
 
 #ifdef __NO_LWSYNC__
 #define LIGHT_MEM_BARRIER __asm __volatile ("sync":::"memory")
 #else
 #define LIGHT_MEM_BARRIER __asm __volatile ("lwsync":::"memory")
 #endif
 
-#else // PPC
+#define FULL_MEM_BARRIER  __sync_synchronize()
+
+#elif defined(X86) // ----------------------------------------------------
 
 #define LIGHT_MEM_BARRIER __asm __volatile ("":::"memory")
+#define FULL_MEM_BARRIER  __sync_synchronize()
+
+#else              // ----------------------------------------------------
+
+// Default to strongest barriers for correctness.
 
-#endif // PPC
+#define LIGHT_MEM_BARRIER __sync_synchronize()
+#define FULL_MEM_BARRIER  __sync_synchronize()
 
-#endif // ARM
+#endif             // ----------------------------------------------------
 
 // Note: What is meant by LIGHT_MEM_BARRIER is a barrier which is sufficient
 // to provide TSO semantics, i.e. StoreStore | LoadLoad | LoadStore.