[12] RFR: 8213825: assert(false) failed: Non-balanced monitor enter/exit! Likely JNI locking

[dean.long at oracle.com]

Looks good to me too.  Nice job tracking this down, Jamsheed!

dl

On 1/23/19 9:11 AM, Vladimir Kozlov wrote:
> Hi Jamsheed,
>
> Fix is good. I approved it for JDK 12 push.
>
> Thanks,
> Vladimir
>
> On 1/23/19 6:08 AM, Jamsheed wrote:
>> Hi,
>>
>> Request for review
>>
>> bug: https://bugs.openjdk.java.net/browse/JDK-8213825
>>
>> webrev: http://cr.openjdk.java.net/~jcm/8213825/webrev.00/index.html
>>
>> Bug & Fix Desc:
>>
>> if markword load has sfpt as control i/p(i.e synchronizations near a 
>> safepoint), it skips sfpt assuming sfptOp wouldn't write to markword 
>> memory
>> fix: not to skip sfpt for markword loads.
>>
>> tests: hs-tier1-5,  hs-precheckin-comp
>>
>> Best regards,
>>
>> Jamsheed
>>