RFR 8210422: runtime/modules/ModuleStress/ExportModuleStressTest.java - assertion failed: address not aligned: 0x00000008baadbabe

[Kim Barrett]

> On Sep 10, 2018, at 9:37 AM, coleen.phillimore at oracle.com wrote:
> 
> Summary: CLDG_lock caused safepoint in inconsistent state
> 
> See bug comments for more details.
> 
> open webrev at http://cr.openjdk.java.net/~coleenp/8210422.01/webrev
> bug link https://bugs.openjdk.java.net/browse/JDK-8210422
> 
> Tested with mach5 hs-tier1-7.  There are good parallel class loading tests that test this code.  Unfortunately these tests aren't open yet.
> 
> Thanks,
> Coleen

src/hotspot/share/classfile/classLoaderData.cpp
1126   // Initialize _name and _name_and_id after the loader data is added to the
1127   // CLDG.

In this revised implementation, why is the loader_data's name
initialized only after it is added to the graph?  You removed the
comment explaining why it used to be that way, since that no longer
applies.  So now the comment just says it's done after, but without
any rationale given for the ordering.

Other than that, looks good.