RFR: 8144145: G1GCPhaseTimes should allow externally accounted time
erik.helin at oracle.com
Thu Nov 26 16:20:59 UTC 2015
On 2015-11-26, Thomas Schatzl wrote:
thanks for reviewing!
> 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
Sure, I'm fine with that. I will update the name of the method to
"inc_external_accounted_time_ms()" before I push.
More information about the hotspot-gc-dev