Merge
authorsspitsyn
Mon, 19 Dec 2016 04:04:48 +0000
changeset 42867 5448e1bbe72a
parent 42865 b9f19a575020 (current diff)
parent 42866 97de8ed6c6b3 (diff)
child 42869 32e49f36d795
Merge
--- a/hotspot/src/share/vm/classfile/moduleEntry.cpp	Sun Dec 18 22:07:31 2016 -0500
+++ b/hotspot/src/share/vm/classfile/moduleEntry.cpp	Mon Dec 19 04:04:48 2016 +0000
@@ -108,6 +108,18 @@
   }
 
   MutexLocker m1(Module_lock);
+  // This is a guard against possible race between agent threads that redefine
+  // or retransform classes in this module. Only one of them is adding the
+  // default read edges to the unnamed modules of the boot and app class loaders
+  // with an upcall to jdk.internal.module.Modules.transformedByAgent.
+  // At the same time, another thread can instrument the module classes by
+  // injecting dependencies that require the default read edges for resolution.
+  if (this->has_default_read_edges() && !m->is_named()) {
+    ClassLoaderData* cld = m->loader_data();
+    if (cld->is_the_null_class_loader_data() || cld->is_system_class_loader_data()) {
+      return true; // default read edge
+    }
+  }
   if (!has_reads()) {
     return false;
   } else {