# HG changeset patch
# User mdoerr
# Date 1473159687 -7200
#      Tue Sep 06 13:01:27 2016 +0200
# Node ID cb2076b580866864747798a6110c7a640010bdd5
# Parent  6ed5d1d6f24cc160f1628dc5fa42e3bf645ca806
8165489: Missing G1 barrier in Unsafe_GetObjectVolatile
Reviewed-by:

diff --git a/src/share/vm/prims/unsafe.cpp b/src/share/vm/prims/unsafe.cpp
--- a/src/share/vm/prims/unsafe.cpp
+++ b/src/share/vm/prims/unsafe.cpp
@@ -263,6 +263,21 @@
 
 // Get/PutObject must be special-cased, since it works with handles.
 
+#if INCLUDE_ALL_GCS
+// We could be accessing the referent field in a reference
+// object. If G1 is enabled then we need to register non-null
+// referent with the SATB barrier.
+inline void G1SATB_registerReference(oop o, jlong offset, oop v) {
+  if (UseG1GC && offset == java_lang_ref_Reference::referent_offset && o != NULL && v != NULL) {
+    Klass* k = o->klass();
+    if (InstanceKlass::cast(k)->reference_type() != REF_NONE) {
+      assert(InstanceKlass::cast(k)->is_subclass_of(SystemDictionary::Reference_klass()), "sanity");
+      G1SATBCardTableModRefBS::enqueue(v);
+    }
+  }
+}
+#endif
+
 // These functions allow a null base pointer with an arbitrary address.
 // But if the base pointer is non-null, the offset should make some sense.
 // That is, it should be in the range [0, MAX_OBJECT_SIZE].
@@ -277,34 +292,11 @@
     v = *(oop*)index_oop_from_field_offset_long(p, offset);
   }
 
-  jobject ret = JNIHandles::make_local(env, v);
+#if INCLUDE_ALL_GCS
+  G1SATB_registerReference(p, offset, v);
+#endif
 
-#if INCLUDE_ALL_GCS
-  // We could be accessing the referent field in a reference
-  // object. If G1 is enabled then we need to register non-null
-  // referent with the SATB barrier.
-  if (UseG1GC) {
-    bool needs_barrier = false;
-
-    if (ret != NULL) {
-      if (offset == java_lang_ref_Reference::referent_offset && obj != NULL) {
-        oop o = JNIHandles::resolve(obj);
-        Klass* k = o->klass();
-        if (InstanceKlass::cast(k)->reference_type() != REF_NONE) {
-          assert(InstanceKlass::cast(k)->is_subclass_of(SystemDictionary::Reference_klass()), "sanity");
-          needs_barrier = true;
-        }
-      }
-    }
-
-    if (needs_barrier) {
-      oop referent = JNIHandles::resolve(ret);
-      G1SATBCardTableModRefBS::enqueue(referent);
-    }
-  }
-#endif // INCLUDE_ALL_GCS
-
-  return ret;
+  return JNIHandles::make_local(env, v);
 } UNSAFE_END
 
 UNSAFE_ENTRY(void, Unsafe_PutObject(JNIEnv *env, jobject unsafe, jobject obj, jlong offset, jobject x_h)) {
@@ -335,6 +327,10 @@
     (void)const_cast<oop&>(v = *(volatile oop*) addr);
   }
 
+#if INCLUDE_ALL_GCS
+  G1SATB_registerReference(p, offset, v);
+#endif
+
   OrderAccess::acquire();
   return JNIHandles::make_local(env, v);
 } UNSAFE_END