RFR (XS): 8073654: Marking statistics should use size_t

Sangheon Kim sangheon.kim at oracle.com
Mon Mar 9 16:36:20 UTC 2015


Hi Thomas,

Thank you for reviewing and sponsoring this!
I'll send the full changeset after removing extra space.

Thanks,
Sangheon


On 03/09/2015 02:28 AM, Thomas Schatzl wrote:
> Hi Sangheon,
>
>
> On Thu, 2015-03-05 at 17:23 -0800, Sangheon Kim wrote:
>> Hi all,
>>
>> Can I have reviews for this simple fix for changing variables for
>> statistics from 'int' to 'size_t'.
>> This is to avoid overflows on large heaps.
>>
>> CR:
>> https://bugs.openjdk.java.net/browse/JDK-8073654
>> Webrev:
>> http://cr.openjdk.java.net/~sangheki/8073654/webrev.00/
>> Testing:
>> JPRT
>>
> In concurrentMark.cpp, there is now an extra space at the end of line
> 292. I do not need a new webrev for fixing that.
>
> Looks good.
>
> I can sponsor it if you still need a sponsor.
>
> Thanks,
>    Thomas
>
>
>




More information about the hotspot-gc-dev mailing list