RFR (M): 8008918: Reference statistics events for the tracing framework

Thomas Schatzl thomas.schatzl at oracle.com
Mon Mar 4 08:24:37 UTC 2013


Hi,

On Fri, 2013-03-01 at 15:09 +0100, Erik Helin wrote:
> Hi all,
> 
> this change adds the vm/gc/reference/statistics event to all collectors.
> 
> Please note that this change depends on "8009232: Improve stats 
> gathering code for reference processor" which is also out for review on
> hotspot-gc-dev at openjdk.java.net.
> 
> Webrev:
> http://cr.openjdk.java.net/~ehelin/8008918/webrev.00/
> 
> Bug:
> https://jbs.oracle.com/bugs/browse/JDK-8008918

Minor: Maybe, in concurrentMarkSweepGeneration.cpp the declaration of
the stats record could be put inside the block.

Do you think if there is an easy way to put the sequence of allocating
the stats record, calling rp->process_discovered_reference and the
report_gc_reference_processing into a single method that is called
everywhere?

This code sequence, is the same in all collectors, and seems to be a
good target for refactoring.

I.e. what the change basically did is to replace the call to
process_discovered_reference by above sequence of three statements.

Hth,
Thomas





More information about the hotspot-gc-dev mailing list