RFR: JDK-8058944: Unify the time reporting strings in G1 remark

Bengt Rutisson bengt.rutisson at oracle.com
Tue Mar 1 10:19:14 UTC 2016


Hi Thomas,

On 2016-03-01 11:17, Thomas Schatzl wrote:
> Hi Bengt,
>
> On Tue, 2016-03-01 at 11:03 +0100, Bengt Rutisson wrote:
>> Hi Stefa,
> [...]
>> Here's an updated webrev:
>> http://cr.openjdk.java.net/~brutisso/8058944/webrev.01/
>>
>> If you find this ok I will change the title of the bug report to be
>> "Unify the reporting strings for the GC debug level logging in G1"
>> since
>> it no longer concerns just the remark phase.
> Looks good except for the "Refeference Processing" typo... ;)

Ooop! Thanks for catching that!

Fixed it this way:

diff --git a/src/share/vm/gc/g1/g1ConcurrentMark.cpp 
b/src/share/vm/gc/g1/g1ConcurrentMark.cpp
--- a/src/share/vm/gc/g1/g1ConcurrentMark.cpp
+++ b/src/share/vm/gc/g1/g1ConcurrentMark.cpp
@@ -2020,3 +2020,3 @@
    {
-    GCTraceTime(Debug, gc) trace("Refeference Processing", 
g1h->gc_timer_cm());
+    GCTraceTime(Debug, gc) trace("Reference Processing", 
g1h->gc_timer_cm());


Thanks,
Bengt


>
> Thanks,
>    Thomas




More information about the hotspot-gc-dev mailing list