RFR(S): 8173335: Improve logging for j.l.ref.reference processing

Aleksey Shipilev shade at redhat.com
Mon Aug 7 11:53:21 UTC 2017


On 08/04/2017 08:46 AM, sangheon wrote:
> Webrev:
> http://cr.openjdk.java.net/~sangheki/8173335/webrev.4 (full)
> http://cr.openjdk.java.net/~sangheki/8173335/webrev.4_to_3b (inc)

Looks good to me. It takes quite a bit of G1 in its sweep, but I guess G1 folks are fine with that?

Minor nit:

 *) In referenceProcessor.cpp, these line should be separated by a newline, because indenting is
confusing:

 195   ReferenceProcessorPhaseTimes* phase_times) {
 196   double start_time = os::elapsedTime();

Thanks,
-Aleksey

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <https://mail.openjdk.java.net/pipermail/hotspot-gc-dev/attachments/20170807/761150eb/signature.asc>


More information about the hotspot-gc-dev mailing list