RFR: JDK-8060017: Report heap sizing time

[Eric Caspole]

Thanks Kim I simplified this part.
http://cr.openjdk.java.net/~ecaspole/JDK-8060017/04/webrev/
Eric

On 3/18/2015 6:51 PM, Kim Barrett wrote:
> On Mar 18, 2015, at 6:00 PM, Eric Caspole <eric.caspole at oracle.com> wrote:
>> 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/
> ------------------------------------------------------------------------------
> src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp
> 3979           size_t expand_bytes = g1_policy()->expansion_amount();
> 3980           double expand_time_ms = 0.0;
> 3981           if (expand_bytes > 0) {
> 3982             double expand_heap_start_time_sec = os::elapsedTime();
> 3983             size_t bytes_before = capacity();
> 3984             // No need for an ergo verbose message here,
> 3985             // expansion_amount() does this when it returns a value > 0.
> 3986             if (!expand(expand_bytes)) {
> 3987               // We failed to expand the heap. Cannot do anything about it.
> 3988             }
> 3989             double expand_heap_end_time_sec = os::elapsedTime();
> 3990             expand_time_ms = (expand_heap_end_time_sec - expand_heap_start_time_sec) * MILLIUNITS;
> 3991           }
> 3992           g1_policy()->phase_times()->record_expand_heap_time(expand_time_ms);
>
> [I meant to say something about this in the first round, but forgot.
> Sorry about that.]
>
> Seems like a lot more complicated than it really needs to be, in order
> to avoid measuring and reporting the time for expand_bytes == 0.
> ------------------------------------------------------------------------------
>