RFR(s): 8212108: SafepointSynchronizer never ending counter (big enough)

[Robbin Ehn]

Thanks David!

/Robbin

On 11/22/18 10:58 AM, David Holmes wrote:
> On 22/11/2018 7:35 pm, Robbin Ehn wrote:
>> Hi David,
>>
>>> Okay.
>>
>> Okay as in reviewed ? :)
> 
> :) Yes.
> 
> David
> 
>> /Robbin
>>
>>>
>>> Thanks,
>>> David
>>>
>>>>>
>>>>> I'm not familiar with JFR types - is "ulong" guaranteed to be 64-bit?
>>>>
>>>> It translate to an u8, so yes.
>>>>
>>>>>
>>>>>> Passes t1-3 and the two updated FR tests.
>>>>>
>>>>> Does any of that actually run long enough to overflow a 32-bit counter?
>>>>
>>>> We have no tests that run long enough to rollover.
>>>>
>>>> Thanks, Robbin
>>>>
>>>>>
>>>>> Thanks,
>>>>> David
>>>>>
>>>>>>
>>>>>> Thanks Robbin