RFR (XXS): 8038829: G1: More useful information in a few assert messages

Jon Masamitsu jon.masamitsu at oracle.com
Mon Mar 31 22:41:33 UTC 2014


Looks good.

Reviewed.

Jon

On 3/31/14 8:14 AM, Bengt Rutisson wrote:
>
> Hi everyone,
>
> Could I have a couple of reviews of this very small fix to some assert 
> messages:
>
> http://cr.openjdk.java.net/~brutisso/8038829/webrev.00/
>
> https://bugs.openjdk.java.net/browse/JDK-8038829
>
> In VerifyRegionListsClosure::doHeapRegion() the assert messages print 
> the region_num() value of the heap region. This value is the the 
> number of regions that a humongous object covers. It is only relevant 
> for starts humongous regions. For all other regions this value is 1.
>
> Most likely the intended value was hrs_index(), which is a sequential 
> number enumerating all heap regions.
>
> Thanks,
> Bengt

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/hotspot-gc-dev/attachments/20140331/bc8c4e8d/attachment.htm>


More information about the hotspot-gc-dev mailing list