8205509: assert(_name_and_id != 0LL) failed: encountered a class loader null name and id
authorlfoltan
Mon, 25 Jun 2018 10:34:46 -0400
changeset 50759 00c4edaf2017
parent 50758 afca3c78ea0f
child 50760 8e296de294e7
8205509: assert(_name_and_id != 0LL) failed: encountered a class loader null name and id Summary: The method loader_name_and_id should return the external name of the class loader if _name_and_id is null. Reviewed-by: coleenp, stuefe
src/hotspot/share/classfile/classLoaderData.cpp
src/hotspot/share/classfile/classLoaderData.hpp
--- a/src/hotspot/share/classfile/classLoaderData.cpp	Mon Jun 25 16:32:23 2018 +0300
+++ b/src/hotspot/share/classfile/classLoaderData.cpp	Mon Jun 25 10:34:46 2018 -0400
@@ -109,10 +109,9 @@
 // it will be available for error messages, logging, JFR, etc.  The name
 // and klass are available after the class_loader oop is no longer alive,
 // during unloading.
-void ClassLoaderData::initialize_name_and_klass(Handle class_loader) {
+void ClassLoaderData::initialize_name(Handle class_loader) {
   Thread* THREAD = Thread::current();
   ResourceMark rm(THREAD);
-  _class_loader_klass = class_loader->klass();
 
   // Obtain the class loader's name.  If the class loader's name was not
   // explicitly set during construction, the CLD's _name field will be null.
@@ -159,6 +158,7 @@
 
   if (!h_class_loader.is_null()) {
     _class_loader = _handles.add(h_class_loader());
+    _class_loader_klass = h_class_loader->klass();
   }
 
   if (!is_anonymous) {
@@ -951,9 +951,11 @@
 const char* ClassLoaderData::loader_name_and_id() const {
   if (_class_loader_klass == NULL) {
     return "'" BOOTSTRAP_LOADER_NAME "'";
+  } else if (_name_and_id != NULL) {
+    return _name_and_id->as_C_string();
   } else {
-    assert(_name_and_id != NULL, "encountered a class loader null name and id");
-    return _name_and_id->as_C_string();
+    // May be called in a race before _name_and_id is initialized.
+    return _class_loader_klass->external_name();
   }
 }
 
@@ -1069,10 +1071,10 @@
 
 ClassLoaderData* ClassLoaderDataGraph::add(Handle loader, bool is_anonymous) {
   ClassLoaderData* loader_data = add_to_graph(loader, is_anonymous);
-  // Initialize name and class after the loader data is added to the CLDG
-  // because adding the Symbol for the name might safepoint.
+  // Initialize _name and _name_and_id after the loader data is added to the
+  // CLDG because adding the Symbol for _name and _name_and_id might safepoint.
   if (loader.not_null()) {
-    loader_data->initialize_name_and_klass(loader);
+    loader_data->initialize_name(loader);
   }
   return loader_data;
 }
--- a/src/hotspot/share/classfile/classLoaderData.hpp	Mon Jun 25 16:32:23 2018 +0300
+++ b/src/hotspot/share/classfile/classLoaderData.hpp	Mon Jun 25 10:34:46 2018 -0400
@@ -305,7 +305,7 @@
 
   Dictionary* create_dictionary();
 
-  void initialize_name_and_klass(Handle class_loader);
+  void initialize_name(Handle class_loader);
  public:
   // GC interface.
   void clear_claimed() { _claimed = 0; }