[9] RFR(S): 8152861: VM crash on assert: locked methods shouldn't be flushed

[Tobias Hartmann]

Thanks, Vladimir!

Best regards,
Tobias

On 29.03.2016 19:09, Vladimir Kozlov wrote:
> Looks good.
> 
> Thanks,
> Vladimir
> 
> On 3/29/16 10:02 AM, Tobias Hartmann wrote:
>> Hi,
>>
>> please review the following patch:
>>
>> https://bugs.openjdk.java.net/browse/JDK-8152861
>> http://cr.openjdk.java.net/~thartmann/8152861/webrev.00/
>>
>> JDK-8023191 made it possible for OSR nmethods to transition from not-entrant to flushed in one sweeper iteration. After making an OSR nmethod zombie, we need to check (again) if the nmethod is locked because JVMTI may claim the nmethod in JvmtiDeferredEvent::compiled_method_unload_event() which is called from nmethod::make_not_entrant_or_zombie() -> nmethod::post_compiled_method_unload(). In this case, we cannot flush the nmethod immediately but have to wait for another iteration of sweeper to finally flush it.
>>
>> Tested with failing JVMTI tests, JPRT and RBT (running). This was not caught by the hs-comp nightlies because the problem is only triggered by two JVMTI tests...
>>
>> Thanks,
>> Tobias
>>