RFR (L): 8073013: Add detailed information about PLAB memory usage

Thomas Schatzl thomas.schatzl at oracle.com
Thu Aug 6 16:01:19 UTC 2015


Hi all,

  just one minor addition: this change is based on all recent RFRs
including 8003237 (the evacuation failure speedup change) and 8133042
(remove some friends in g1collectedheap).

For the others a push job is on its way.

Thanks,
  Thomas

On Thu, 2015-08-06 at 17:49 +0200, Thomas Schatzl wrote:
> Hi all,
> 
>   can I have reviews for this change that prepares for a lot of changes
> related to 8030849 "Investigate high fragmentation/waste in some
> situations during allocation during GC in G1"?
> 
> Before investigating and fixing fragmentation during PLAB, we need to
> track that information.
> 
> For this reason, PLABStats has been extended in a G1EvacStats class with
> the necessary information that is needed to do better PLAB sizing, in
> particular G1EvacStats::_region_end_waste that tracks the waste at the
> end of regions during refill, G1EvacStats::_regions_filled that tracks
> the number of region refills, G1EvacStats::_inline_allocated that tracks
> the amount of inline allocation (excluding PLAB allocation),
> G1EvacStats::_failure_used and G1EvacStats::_failure_waste that track
> memory usage in regions where evacuation failure occurred.
> 
> There had to be somewhat extensive changes particularly to get the
> _region_end_waste in the allocation classes, which is what makes this
> change a bit large.
> 
> Overall there is no change in the sizing policy except for accounting
> _region_end_waste.
> 
> As you may notice, there is a new JFR event, but no real additional
> logging for the log file (the -XX:+PrintPLAB message has been extended
> though). That will be added in an extra CR, as soon as the new logging
> framework has been added.
> 
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8073013
> 
> Webrev:
> http://cr.openjdk.java.net/~tschatzl/8073013/webrev
> 
> Testing:
> jprt, a few aurora runs, local testing, lots of benchmarks mainly in
> conjunction with the following changes.
> 
> Thanks,
>   Thomas
> 
> 




More information about the hotspot-gc-dev mailing list