RFR 8215050: [TESTBUG] serviceability/tmtools/jstack/WaitNotifyThreadTest.java fails when run with flag -Xcomp

[coleen.phillimore at oracle.com]

Thank you.  I think this looks good.
Coleen

On 12/10/18 8:32 AM, Patricio Chilano wrote:
> Hi Coleen,
>
> On 12/10/18 8:02 AM, coleen.phillimore at oracle.com wrote:
>>
>> Can you put the jstack output in the bug, so I can see what it's 
>> trying to compare?
> Done.
>
> Thanks,
> Patricio
>> Thanks,
>> Coleen
>>
>> On 12/8/18 7:42 PM, Patricio Chilano wrote:
>>> Hi David,
>>>
>>> On 12/8/18 6:21 PM, David Holmes wrote:
>>>> Hi Patricio,
>>>>
>>>> On 9/12/2018 9:04 am, Patricio Chilano wrote:
>>>>> Hi,
>>>>>
>>>>> Could you review this small fix for test 
>>>>> serviceability/tmtools/jstack/WaitNotifyThreadTest.java ?
>>>>> After change 8214148 the test fails if the flag -Xcomp is used as 
>>>>> explained in the bug details. The proposed change is to identified 
>>>>> this special case and set the monitor address to match the one 
>>>>> expected.
>>>>>
>>>>> http://cr.openjdk.java.net/~pchilanomate/8215050.02/webrev/
>>>>> https://bugs.openjdk.java.net/browse/JDK-8215050
>>>>>
>>>>> Run tier-4 where the test was failing and passed (the test was 
>>>>> failing on tier6 also because it uses -Xcomp too). Currently 
>>>>> running tiers1-3.
>>>>
>>>> Change seems fine. The lack of address must be something relatively 
>>>> new as I don't see it with Xcomp in JDK 9.
>>> Thanks for the review!
>>>
>>>
>>> Thanks,
>>> Patricio
>>>> Thanks,
>>>> David
>>>>
>>>>> Thanks,
>>>>> Patricio
>>>
>>
>