Code Review for src.9

Prepared by:tschatzl on Thu Jul 10 11:11:06 CEST 2014
Workspace:/home/thomas/Downloads/vmshare/cmm-decommit/src.9
Compare against: http://hg.openjdk.java.net/jdk9/hs-gc/hotspot
Compare against version:-2
Summary of changes: 17 lines changed: 0 ins; 6 del; 11 mod; 11541 unchg
Patch of changes: src.9.patch
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp

rev 6673 : 8048112: G1 Full GC needs to support the case when the very first region is not available
Summary: To allow uncommit of regions within the heap, G1 Full GC should correctly handle the case when the very first region is not available (uncommitted). Provide support for that by lazily initializing the compaction point during iteration of the list of heap regions. Further refactor the code to let the G1CollectedHeap handle finding the next region to compact into.
Reviewed-by:
rev 6674 : imported patch fixes-kim
5 lines changed: 0 ins; 3 del; 2 mod; 7021 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/gc_implementation/g1/g1CollectedHeap.hpp

rev 6673 : 8048112: G1 Full GC needs to support the case when the very first region is not available
Summary: To allow uncommit of regions within the heap, G1 Full GC should correctly handle the case when the very first region is not available (uncommitted). Provide support for that by lazily initializing the compaction point during iteration of the list of heap regions. Further refactor the code to let the G1CollectedHeap handle finding the next region to compact into.
Reviewed-by:
rev 6674 : imported patch fixes-kim
4 lines changed: 0 ins; 0 del; 4 mod; 1716 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/gc_implementation/g1/g1MarkSweep.cpp

rev 6673 : 8048112: G1 Full GC needs to support the case when the very first region is not available
Summary: To allow uncommit of regions within the heap, G1 Full GC should correctly handle the case when the very first region is not available (uncommitted). Provide support for that by lazily initializing the compaction point during iteration of the list of heap regions. Further refactor the code to let the G1CollectedHeap handle finding the next region to compact into.
Reviewed-by:
rev 6674 : imported patch fixes-kim
1 line changed: 0 ins; 0 del; 1 mod; 376 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/gc_implementation/g1/heapRegionSet.hpp

rev 6674 : imported patch fixes-kim
1 line changed: 0 ins; 0 del; 1 mod; 330 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/memory/genCollectedHeap.cpp

rev 6674 : imported patch fixes-kim
1 line changed: 0 ins; 0 del; 1 mod; 1311 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/memory/space.hpp

rev 6673 : 8048112: G1 Full GC needs to support the case when the very first region is not available
Summary: To allow uncommit of regions within the heap, G1 Full GC should correctly handle the case when the very first region is not available (uncommitted). Provide support for that by lazily initializing the compaction point during iteration of the list of heap regions. Further refactor the code to let the G1CollectedHeap handle finding the next region to compact into.
Reviewed-by:
rev 6674 : imported patch fixes-kim
5 lines changed: 0 ins; 3 del; 2 mod; 787 unchg

This code review page was prepared using ../../tools/webrev.sh (vers 23.18-hg).