RFR (M): 8073052: Rename and clean up the allocation manager hierarchy in g1Allocator.?pp
thomas.schatzl at oracle.com
Wed Jul 29 07:51:21 UTC 2015
thanks for the review.
On Tue, 2015-07-28 at 14:42 +0200, Mikael Gerdin wrote:
> Hi Thomas,
> On 2015-07-28 10:46, Thomas Schatzl wrote:
> > Hi all,
> > Finally, the change aligns the naming of G1ParGCAllocator with other
> > classes that use "PLAB" in their names instead of "ParGC".
> > CR:
> > https://bugs.openjdk.java.net/browse/JDK-8073052
> > Webrev:
> > http://cr.openjdk.java.net/~tschatzl/8073052/webrev.3/
> Overall a nice cleanup!
> In unsafe_max_tlab_alloc you get the heap via G1H::heap() instead of _g1h.
> G1PLABAllocator seems to look up the G1H::heap() fairly often,
> did you consider passing on the G1H* to the G1PLABAllocator to reduce
> the clutter from accessing through the static getter in multiple
> locations? Another option could be to expose the G1H* from G1Allocator,
> yielding _allocator->heap()->...
> instead of _g1h->...
More information about the hotspot-gc-dev