RFR (S) 8222231: Clean up interfaceSupport.inline.hpp duplicated code

[Patricio Chilano]

Hi Coleen,

On 4/10/19 3:27 PM, coleen.phillimore at oracle.com wrote:
>
> On 4/10/19 12:01 PM, Patricio Chilano wrote:
>> Hi Coleen!
>>
>> Change looks good to me! Just a small comment. Since we now will not 
>> clear unhandled oops in the TBIVMWDC jacket anymore, I think we 
>> should add that check in Monitor::wait() like we do in Monitor::lock().
>
> Patricio,  Thank you for reviewing this.  Good find!
>
> http://cr.openjdk.java.net/~coleenp/2019/8222231.02/webrev/
>
> I added it and retested runThese jck tests with -XX:+CheckUnhandledOops.
Looks good to me!


Thanks,
Patricio
> Thanks,
> Coleen
>
>>
>> Thanks!
>> Patricio
>>
>> On 4/10/19 10:36 AM, coleen.phillimore at oracle.com wrote:
>>>
>>> Thank you David!
>>> Coleen
>>>
>>> On 4/9/19 10:04 PM, David Holmes wrote:
>>>> Hi Coleen,
>>>>
>>>> Thanks for doing this! It is all a lot simpler now. Reviewed with 
>>>> enthusiasm. :)
>>>>
>>>> David
>>>>
>>>> On 10/04/2019 9:58 am, coleen.phillimore at oracle.com wrote:
>>>>> Some code was left from removing code UseMembar and further 
>>>>> improvements encouraged by dholmes.  I also removed a couple 
>>>>> redundant and unneeded clear_unhandled_oops calls.  The one in 
>>>>> ThreadBlockInVMWithDeadlockCheck is in the calling code 
>>>>> Monitor::lock, and the thread in vm from native makes no sense.
>>>>>
>>>>> Tested with runtime jtreg tests, and mach5 tier1-3 in progress.
>>>>>
>>>>> open webrev at 
>>>>> http://cr.openjdk.java.net/~coleenp/2019/8222231.01/webrev
>>>>> bug link https://bugs.openjdk.java.net/browse/JDK-8222231
>>>>>
>>>>> Thanks,
>>>>> Coleen
>>>
>>
>