RFR (S) 8225681: vmTestbase/nsk/jvmti/RedefineClasses/StressRedefine fails due a) MT-unsafe modification of inline cache

[coleen.phillimore at oracle.com]

Thanks Serguei!
Coleen

On 9/25/19 5:56 PM, serguei.spitsyn at oracle.com wrote:
> Hi Coleen,
>
> This looks fine to me.
> Nice unification for all platforms.
> Thank you for taking care about this issue!
>
> Thanks,
> Serguei
>
>
> On 9/25/19 2:28 PM, coleen.phillimore at oracle.com wrote:
>> Adding serviceability-dev.
>> Coleen
>>
>> On 9/25/19 5:22 PM, coleen.phillimore at oracle.com wrote:
>>> Summary: allow old methods in 
>>> CompiledStaticDirectCall::set_to_interpreted
>>>
>>> This is the comment in the bug that describes this race and this fix:
>>> https://bugs.openjdk.java.net/browse/JDK-8225681?focusedCommentId=14278441&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-14278441 
>>>
>>>
>>> The rest of the bug and sightings are actually caused by 
>>> https://bugs.openjdk.java.net/browse/JDK-8226690,
>>> and this one might have been caused by it also, but the race that 
>>> Erik describes is possible as well.
>>>
>>> The s390 code had an exception for callee->is_compiled_lambda_form() 
>>> which should probably apply to all the platforms, so the code is the 
>>> same on all the platforms with this change.
>>>
>>> Tested with tier1-6.
>>>
>>> open webrev at 
>>> http://cr.openjdk.java.net/~coleenp/2019/8225681.01/webrev
>>> bug link https://bugs.openjdk.java.net/browse/JDK-8225681
>>>
>>> Thanks,
>>> Coleen
>>
>