RFR: JDK-8145303: Clean up the units for log_gc_footer
Thomas Schatzl
thomas.schatzl at oracle.com
Mon Dec 14 10:51:19 UTC 2015
Hi Bengt,
On Mon, 2015-12-14 at 10:42 +0100, Bengt Rutisson wrote:
> Hi everyone,
>
> Could I have a couple of reviews for this small patch?
>
> http://cr.openjdk.java.net/~brutisso/8145303/webrev.00/
> https://bugs.openjdk.java.net/browse/JDK-8145303
>
> The log_gc_footer() method takes one parameter of type double. But it
> should be jlong. It also converts to seconds but farther down the call
> chain we convert to milliseconds.
do you mind removing the line "double pause_start_sec =
os::elapsedTime()" in g1CollectedHeap.cpp:3717 just above that change in
3718. It's dead code.
Do not need another review for that.
Thanks,
Thomas
More information about the hotspot-gc-dev
mailing list