RFR (XS): 8u40 Backport of 8038423: G1: Decommit memory within the heap

Bengt Rutisson bengt.rutisson at oracle.com
Wed Aug 20 07:01:52 UTC 2014


Hi Thomas,

Merge looks good.

The four new files (g1PageBasedVirtualSpace.cpp/hpp and 
g1RegionToSpaceMapper.cpp/hpp) are missing from the backport webrev. 
Don't forget to 'hg add' them before you push ;) No need for a new review.

Thanks,
Bengt


On 2014-08-19 16:21, Thomas Schatzl wrote:
> Hi all,
>
>    can I have reviews for backport merge conflicts for 8038423?
>
> Again, it's two minor issues:
>
>   - typo in g1CardCounts.cpp (expects "... will be an OOB access...", but
> is "... will be an OOB accesss...").
>
>   - missing change in heapRegionRemSet.cpp, the format string for the
> initializer of the OtherRegionTable mutex is expected to read
> "HeapRegionRemSet lock #%u", but is "HeapRegionRemSet lock
> #"UINT32_FORMAT".
> [The missing change seems to be 8039244, which cannot be easily
> backported.]
>
> Webrev:
> http://cr.openjdk.java.net/~tschatzl/8038423/webrev.8u40/
>
> Original webrev:
> http://cr.openjdk.java.net/~tschatzl/8038423/webrev.1/
>
> Thanks,
>    Thomas
>




More information about the hotspot-gc-dev mailing list