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

[coleen.phillimore at oracle.com]

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.

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
>>
>