RFR (XL): 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data

Thomas Schatzl thomas.schatzl at oracle.com
Tue Aug 12 15:09:04 UTC 2014


Hi all,

  can I have reviews for this somewhat large refactoring change? It is
about refactoring the HeapRegionSeq class to manage heap region and
auxiliary data.

I.e. currently HeapRegionSeq only manages the HeapRegion instances
corresponding to the heap's region. The change gives HeapRegionSeq more
responsibilities, encapsulating functionality related to heap memory
management. This decreases the amount of responsibilities (and
complexity) for the G1CollectedHeap class: decisions about how heap
related memory is allocated/freed/iterated (i.e. how the heap regions
are actually allocated in the heap) is removed from G1CollectedHeap.

This change only changes the interface to this functionality. It is a
preparatory change for JDK-8038423 "G1: Decommit memory within the
heap", so the change might be slightly more extensive than really
required.

The RFR for JDK-8038423 will follow to look at it in context.

There is another CR that renames HeapRegionSeq to HeapRegionmanager too.

CR:
https://bugs.openjdk.java.net/browse/JDK-8054818

Webrev:
http://cr.openjdk.java.net/~tschatzl/8054818/webrev/

Testing:
jprt, nightly and bigapps (kitchensink, ...) with -XX:+UseG1GC

Thanks,
  Thomas






More information about the hotspot-gc-dev mailing list