Code Review for hotspot

Prepared by:mgerdin mgerdin on Thu Apr 14 13:14:56 CEST 2016
Workspace:/home/mgerdin/work/repos/hg/jdk9/hs-rt-work/hotspot
Compare against: http://hg.openjdk.java.net/jdk9/hs-rt/hotspot
Compare against version:10864
Summary of changes: 1850 lines changed: 37 ins; 1678 del; 135 mod; 19285 unchg
Changeset: hotspot.changeset
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/collectionSetChooser.cpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
1 line changed: 0 ins; 1 del; 0 mod; 274 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/concurrentG1Refine.cpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
2 lines changed: 1 ins; 0 del; 1 mod; 200 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/concurrentG1RefineThread.cpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
1 line changed: 0 ins; 1 del; 0 mod; 174 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/concurrentMarkThread.cpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
4 lines changed: 1 ins; 1 del; 2 mod; 314 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/concurrentMarkThread.hpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
2 lines changed: 0 ins; 0 del; 2 mod; 86 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/g1Allocator.cpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
1 line changed: 0 ins; 1 del; 0 mod; 484 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/g1CardCounts.cpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
2 lines changed: 0 ins; 2 del; 0 mod; 157 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/g1CollectedHeap.cpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
7 lines changed: 2 ins; 0 del; 5 mod; 5538 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/g1CollectedHeap.hpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
6 lines changed: 4 ins; 0 del; 2 mod; 1529 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/g1CollectedHeap.inline.hpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
1 line changed: 0 ins; 1 del; 0 mod; 306 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/g1CollectedHeap_ext.cpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
4 lines changed: 4 ins; 0 del; 0 mod; 40 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/g1CollectionSet.cpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
4 lines changed: 1 ins; 1 del; 2 mod; 423 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/g1CollectionSet.hpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
9 lines changed: 1 ins; 6 del; 2 mod; 199 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/g1CollectorPolicy.cpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
1166 lines changed: 0 ins; 1163 del; 3 mod; 106 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/g1CollectorPolicy.hpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
397 lines changed: 0 ins; 392 del; 5 mod; 46 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/g1ConcurrentMark.cpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
6 lines changed: 1 ins; 1 del; 4 mod; 3071 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/g1HeapTransition.cpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
2 lines changed: 1 ins; 1 del; 0 mod; 121 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/g1HeapVerifier.cpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
1 line changed: 1 ins; 0 del; 0 mod; 731 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/g1MonitoringSupport.cpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
2 lines changed: 1 ins; 1 del; 0 mod; 273 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/g1ParScanThreadState.hpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
2 lines changed: 1 ins; 1 del; 0 mod; 242 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/g1RemSet.cpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
1 line changed: 0 ins; 1 del; 0 mod; 609 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/g1RemSet.hpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
3 lines changed: 1 ins; 1 del; 1 mod; 236 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/g1RootProcessor.cpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
2 lines changed: 1 ins; 1 del; 0 mod; 301 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/g1YoungGenSizer.cpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
7 lines changed: 5 ins; 0 del; 2 mod; 120 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/g1YoungGenSizer.hpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
8 lines changed: 4 ins; 0 del; 4 mod; 97 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/g1YoungRemSetSamplingThread.cpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
3 lines changed: 1 ins; 1 del; 1 mod; 109 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/g1YoungRemSetSamplingThread.hpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
1 line changed: 0 ins; 0 del; 1 mod; 59 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/heapRegion.cpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
1 line changed: 0 ins; 0 del; 1 mod; 1095 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/vm_operations_g1.cpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
1 line changed: 0 ins; 0 del; 1 mod; 240 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/youngList.cpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
1 line changed: 0 ins; 0 del; 1 mod; 201 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/shared/vmGCOperations.cpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
1 line changed: 1 ins; 0 del; 0 mod; 294 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/services/g1MemoryPool.cpp

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
2 lines changed: 0 ins; 2 del; 0 mod; 89 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/g1Policy.cpp (was src/share/vm/gc/g1/g1CollectorPolicy.cpp)

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
178 lines changed: 4 ins; 89 del; 85 mod; 1098 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc/g1/g1Policy.hpp (was src/share/vm/gc/g1/g1CollectorPolicy.hpp)

rev 10865 : 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
Summary: Move G1 runtime policy code to new class G1Policy which is unrelated to CollectorPolicy
Reviewed-by:
21 lines changed: 1 ins; 10 del; 10 mod; 423 unchg

This code review page was prepared using /localhome/hg/webrev/webrev.ksh (vers 25.16-hg+openjdk.java.net).