8211387: [Zero] atomic_copy64: Use ldrexd for atomic reads on ARMv7
authorsgehwolf
Tue, 02 Oct 2018 17:17:43 +0200
changeset 52051 6ee9500fe653
parent 52050 a42c378b6f01
child 52052 e25291a90cba
8211387: [Zero] atomic_copy64: Use ldrexd for atomic reads on ARMv7 Reviewed-by: dholmes, aph Contributed-by: Andrew Haley <aph@redhat.com>
src/hotspot/os_cpu/linux_zero/os_linux_zero.hpp
--- a/src/hotspot/os_cpu/linux_zero/os_linux_zero.hpp	Tue Oct 09 07:06:32 2018 +0100
+++ b/src/hotspot/os_cpu/linux_zero/os_linux_zero.hpp	Tue Oct 02 17:17:43 2018 +0200
@@ -54,6 +54,16 @@
                   "std %0, %1\n"
                   : "=&f"(tmp), "=Q"(*(volatile double*)dst)
                   : "Q"(*(volatile double*)src));
+#elif defined(__ARM_ARCH_7A__)
+    // Note that a ldrexd + clrex combination is only needed for
+    // correctness on the OS level (context-switches). In this
+    // case, clrex *may* be beneficial for performance. For now
+    // don't bother with clrex as this is Zero.
+    jlong tmp;
+    asm volatile ("ldrexd  %0, [%1]\n"
+                  : "=r"(tmp)
+                  : "r"(src), "m"(src));
+    *(jlong *) dst = tmp;
 #else
     *(jlong *) dst = *(const jlong *) src;
 #endif