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

[Coleen Phillimore]

Thank you for the code review, Lois!
Coleen

On 10/24/2014 05:00 PM, Lois Foltan wrote:
> Hi Coleen,
>
> I liked how you fixed this and good additional comments. Reviewed.
>
> Thanks,
> Lois
>
> On 10/24/2014 9:49 AM, Coleen Phillimore wrote:
>> 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
>