RFR (S) 8175249: VMThread::run fails in VerifyBeforeExit : Universe::verify

[Jiangli Zhou]

Hi Coleen,

The approach looks very clean. 

I have a question for the following in ClassLoaderDataGraph::do_unloading(), which may not be an issue. The ‘if' check is for anonymous class loader, which does not have dictionary. So do_unloading(is_alive_closure) is not happening during unloading of an anonymous class loader. Could a ‘pd’ for anonymous class being added to any other loaders’ pd_set? I’ll try to construct a test case.

1294       if (data->dictionary() != NULL) {
 <>1295         data->dictionary()->do_unloading(is_alive_closure);
1296       }
Thanks,
Jiangli

> On Jan 22, 2018, at 3:52 PM, coleen.phillimore at oracle.com wrote:
> 
> Summary: protection domain package access cache needs to be walked in unloading
> 
> Tested with mach5 tier1-5 on linux and windows, and with tests in the bug report.
> 
> open webrev at http://cr.openjdk.java.net/~coleenp/8175249.01/webrev
> bug link https://bugs.openjdk.java.net/browse/JDK-8175249
> 
> Thanks,
> Coleen