[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    Re: jmx-dev Codereview request: 8025206 IIntermittent test failure: javax/management/monitor/NullAtt
From:       Daniel Fuchs <daniel.fuchs () oracle ! com>
Date:       2013-09-30 10:09:10
Message-ID: 52494DC6.7030908 () oracle ! com
[Download RAW message or body]


Looks good!

-- daniel

On 9/30/13 11:47 AM, shanliang wrote:
> Daniel Fuchs wrote:
>> Hi Shanliang,
>>
>> Shouldn't 'messageReceived' be at least declared as volatile?
>> It looks as if this test is a multi-thread test which
>> is not MT-safe.
> OK I am convinced:
>      http://cr.openjdk.java.net/~sjiang/JDK-8025206/02/
>
> Thanks,
> Shanliang
>
>>
>> cheers,
>>
>> -- daniel
>>
>> On 9/30/13 8:55 AM, shanliang wrote:
>>> Hi,
>>>
>>> Please review this test fix, I set a much long waiting time to receive a
>>> notification, but not simply wait the test timeout, in order to have
>>> less modification.
>>>
>>> webrev: http://cr.openjdk.java.net/~sjiang/JDK-8025206/00/
>>> bug: https://bugs.openjdk.java.net/browse/JDK-8025206
>>>
>>> Thanks,
>>> Shanliang
>>
>

[prev in list] [next in list] [prev in thread] [next in thread]