RFR: 8058093: Test nsk/stress/jck60/jck60014: assert in src/share/vm/oops/constantPool.cpp: should not be resolved otherwise

Coleen Phillimore coleen.phillimore at oracle.com
Fri Oct 24 13:49:35 UTC 2014


Also:  8044209: nsk/split_verifier/security/coglio06 fails with exit 
code 97 - missing 'prohibited package name'

Summary: Fix resolution error saving.

These bugs are together because code changes fix both.   For jck60014, 
constant pool resolution is not deterministic.  You can have some 
threads install security managers and then remove them (or install new 
ones).  This test isn't supposed to pass but it isn't supposed to crash 
either.  So the code change restores the concurrent update of constant 
pool tag so that if it's resolved, the resolution wins the race.  It 
restores the comment also.

For coglio06, the call from deoptimization ignores the error so if we 
don't save the resolution error in the constant pool, it is reresolved 
by the interpreter and the error is reported with the cause.  
Technically, we should save the cause too, but I don't have a test case 
that relies on that with this fix and it's expensive for footprint and 
code changes.

Testing:  hotspot/jtreg, vm.quick.testlist, jck's (which test saving 
resolution error for jvmti modifying bootclasspath), and my own test 
which I can't add because it relies on adding sleeps in the jvm and test 
case.

open webrev at http://cr.openjdk.java.net/~coleenp/8058093/
bug link https://bugs.openjdk.java.net/browse/JDK-8058093

Thanks,
Coleen


More information about the hotspot-runtime-dev mailing list