JDK-8145303: Clean up the units for log_gc_footer
Lindenmaier, Goetz
goetz.lindenmaier at sap.com
Mon Dec 14 10:57:12 UTC 2015
Hi Bengt,
thanks for doing a real fix for this. Looks good (and works with gcc 4.1.2).
I'll remove my patch for g1CollectedHeap.cpp from my
webrev for 8143215: gcc 4.1.2: fix three issues breaking the build.
Thanks a lot!
Goetz.
> -----Original Message-----
> From: hotspot-gc-dev [mailto:hotspot-gc-dev-bounces at openjdk.java.net]
> On Behalf Of Bengt Rutisson
> Sent: Montag, 14. Dezember 2015 10:42
> To: hotspot-gc-dev at openjdk.java.net
> Subject: RFR: JDK-8145303: Clean up the units for log_gc_footer
>
>
> 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.
>
> Thanks,
> Bengt
More information about the hotspot-gc-dev
mailing list