RFR: JDK-8060017: Report heap sizing time

Eric Caspole eric.caspole at oracle.com
Wed Mar 18 22:00:53 UTC 2015


Thanks Kim,
I fixed the copyrights and a jchecky thing with trailing spaces I did 
not notice earlier. New one:
  http://cr.openjdk.java.net/~ecaspole/JDK-8060017/03/webrev/

Eric

On 3/17/2015 5:30 PM, Kim Barrett wrote:
> On Mar 16, 2015, at 6:06 PM, Eric Caspole <eric.caspole at oracle.com> wrote:
>> Hi,
>> Please review this fix for showing the details of heap expansion time in the PrintGCDetails.
>>
>> https://bugs.openjdk.java.net/browse/JDK-8060017
>> http://cr.openjdk.java.net/~ecaspole/JDK-8060017/02/webrev/
>>
>> The time to expand the heap is shown in the "Other" section at the end of each GC cycle info.
>>
>> I tested this with a smallish app with a high allocation rate which does a lot of heap expands depending on the command line options used. Also, passes JPRT. The output shows up and graphs correctly in the gc log app.
>> Thanks,
>> Eric
> Looks good.
>
> A few copyrights need to be updated.
>




More information about the hotspot-gc-dev mailing list