RFR: 8295278: Add parallel class loading tests

Calvin Cheung ccheung at openjdk.org
Fri Oct 14 03:39:15 UTC 2022


On Wed, 5 Oct 2022 22:53:48 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

> I moved the ParallelLoad test to a subdirectory, and created a new one called CallLoadClass that calls loadClass directly and then waits on the ClassLoader object, while another thread calls forName() to go through the SystemDictionary resolution.
> 
> I also made dedicated tests out of the bugs reported in
> [JDK-4699981](https://bugs.openjdk.org/browse/JDK-4699981) ClassCircularityError thrown without reason during class loading and
> [JDK-4670071](https://bugs.openjdk.org/browse/JDK-4670071) java.lang.ClassLoader.loadClassInternal(String) is too restrictive
> 
> Tested with tier1 on Oracle platforms. These tests are run in tier1.

I looked at mostly the tests under CallLoadClass and have one comment. Thanks.

test/hotspot/jtreg/runtime/ParallelLoad/CallLoadClass/ClassLoadingThread.java line 68:

> 66:             } else {
> 67:                 success = true;
> 68:             }

Should `success` be set to `false` in this case?

-------------

PR: https://git.openjdk.org/jdk/pull/10587


More information about the hotspot-runtime-dev mailing list