review request (S) - 8057818: collect allocation context stats
John Coomes
John.Coomes at oracle.com
Wed Sep 10 17:15:58 UTC 2014
Jon Masamitsu (jon.masamitsu at oracle.com) wrote:
> John,
>
> http://cr.openjdk.java.net/~jcoomes/8u/8u40/8057818-stats-collect/src/share/vm/gc_implementation/g1/g1CollectedHeap.hpp.udiff.html
>
> For now or a later clean up.
>
> I'm used to having the accessor have a name more
> similar to the field. I would suggest changing _alloc_ctx_stats
> to _allocation_context_stats.
>
> + // Statistics for each allocation context
> + AllocationContextStats _alloc_ctx_stats;
> +
>
> + inline AllocationContextStats & allocation_context_stats();
Sure, I changed the field name.
> Rest looks good modulo Mikael's comments.
>
> Reviewed.
Thanks.
-John
> On 09/09/2014 12:00 AM, John Coomes wrote:
> > Hi,
> >
> > Please review an interface for collecting allocation context
> > statistics at g1 gc pauses:
> >
> > 8057818: collect allocation context statistics at gc pauses
> >
> > http://cr.openjdk.java.net/~jcoomes/8u/8u40/8057818-stats-collect/
> >
> > -John
>
More information about the hotspot-gc-dev
mailing list