RFR: 8144145: G1GCPhaseTimes should allow externally accounted time

Thomas Schatzl thomas.schatzl at oracle.com
Thu Nov 26 16:11:39 UTC 2015


Hi,

On Thu, 2015-11-26 at 16:31 +0100, Erik Helin wrote:
> Hi all,
> 
> this small patch allows code to report accounted time to
> G1GCPhaseTimes, for example if something takes time that isn't tracked
> by G1GCPhaseTimes (code in the runtime, shared code, etc. that
> might be logged some other way).
> 
> Enhancement:
> https://bugs.openjdk.java.net/browse/JDK-8144145
> 
> Webrev:
> http://cr.openjdk.java.net/~ehelin/8144145/webrev.00/
> 
> Testing:
> - GC Basher with full verification.

  looks okay. Can you change the name of the accessor to that field to
correspond to the field name, i.e. without the "ly" suffix?

I prefer these accessors to correspond exactly to the member variable,
even ignoring proper English sometimes.

I do not need to see a changeset for such a change

Thomas




More information about the hotspot-gc-dev mailing list