RFR: JDK-8136991: [REDO] Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125

Bengt Rutisson bengt.rutisson at oracle.com
Mon Sep 28 07:02:33 UTC 2015


Thanks for reviewing this, Thomas!

Bengt

On 2015-09-25 21:20, Thomas Schatzl wrote:
> Hi,
>
> On Fri, 2015-09-25 at 10:27 +0200, Bengt Rutisson wrote:
>> Hi Kim,
> [...]
>>> Better would be to separate out the present implementation of
>>> GCTraceTime into a helper class (let's call it GCTraceTimeImpl), which
>>> is a VALUE_OBJ_CLASS_SPEC class. Then GCTraceTime, GCCMTraceTime, and
>>> GCRefTraceTime all derive from StackObj and have a GCTraceTimeImpl
>>> member.
>> Good suggestion.
>>
>> Here's an updated webrev:
>> http://cr.openjdk.java.net/~brutisso/8136991/webrev.05
>>
>> (Note that referenceType.hpp is unchanged even if webrev decides to
>> include it.)
>>
>> And here's a diff compared to the last version:
>> http://cr.openjdk.java.net/~brutisso/8136991/webrev.04-05.diff
>    ship it.
>
> Thomas
>
>




More information about the hotspot-gc-dev mailing list