Code Review for src.9

Prepared by:tschatzl on Thu Jul 10 11:11:39 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:-3
Summary of changes: 70 lines changed: 25 ins; 26 del; 19 mod; 12643 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
11 lines changed: 8 ins; 1 del; 2 mod; 7013 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
7 lines changed: 0 ins; 1 del; 6 mod; 1714 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
35 lines changed: 17 ins; 13 del; 5 mod; 355 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/gc_implementation/g1/heapRegion.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:
12 lines changed: 0 ins; 11 del; 1 mod; 1134 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
3 lines changed: 0 ins; 0 del; 3 mod; 786 unchg

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