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

[Jamsheed]

Thanks a lot for the review, Dean.

Best regards,

Jamsheed

On 1/24/19 1:54 AM, dean.long at oracle.com wrote:
> 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
>>>
>