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

Thomas Schatzl thomas.schatzl at oracle.com
Mon Mar 9 09:28:30 UTC 2015


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