RFR (S) 8202669: Intermittent crash in ClassLoadingService::compute_class_size()

David Holmes david.holmes at oracle.com
Mon May 21 23:28:03 UTC 2018

Hi Coleen,

This looks good.

On 22/05/2018 9:01 AM, coleen.phillimore at oracle.com wrote:
> Summary: Don't report classes that failed to load when reporting class 
> unloading.  Remove the classes first.
> Also add a NULL check for transitive_interfaces for safety, but the new 
> tests pass without it and fail without the fix.
> open webrev at http://cr.openjdk.java.net/~coleenp/8202669.01/webrev
> bug link https://bugs.openjdk.java.net/browse/JDK-8202669


590   // after erroreous classes are released.

Typo: erroreous -> erroneous



56       byte[] buf = InMemoryJavaCompiler.compile("java.lang.Object", 
  57                                         "--patch-module=java.base");

Indent is wrong on line 57.

I don't see why you need the two test programs as 
TestUnloadClassError.java tests both conditions.

No need to see a new webrev.


> Tested with hs-tier1-5.
> Thanks,
> Coleen

More information about the hotspot-runtime-dev mailing list