Code Review for hotspot

Prepared by:ehelin ehelin on Fri Nov 14 13:32:32 CET 2014
Workspace:/home/ehelin/code/jdk9/hs-gc/hotspot
Compare against: ssh://hg.openjdk.java.net/jdk9/hs-gc/hotspot
Compare against version:7317
Summary of changes: 81 lines changed: 6 ins; 46 del; 29 mod; 16992 unchg
Changeset: hotspot.changeset
Legend: Modified file
Deleted file
New file

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

rev 7318 : 8064721: The card tables only ever need two covering regions
1 line changed: 0 ins; 0 del; 1 mod; 6761 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc_implementation/g1/g1SATBCardTableModRefBS.cpp

rev 7318 : 8064721: The card tables only ever need two covering regions
6 lines changed: 0 ins; 2 del; 4 mod; 252 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc_implementation/g1/g1SATBCardTableModRefBS.hpp

rev 7318 : 8064721: The card tables only ever need two covering regions
4 lines changed: 0 ins; 2 del; 2 mod; 177 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc_implementation/parallelScavenge/cardTableExtension.hpp

rev 7318 : 8064721: The card tables only ever need two covering regions
2 lines changed: 0 ins; 0 del; 2 mod; 111 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc_implementation/parallelScavenge/parallelScavengeHeap.cpp

rev 7318 : 8064721: The card tables only ever need two covering regions
1 line changed: 0 ins; 0 del; 1 mod; 736 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/memory/barrierSet.hpp

rev 7318 : 8064721: The card tables only ever need two covering regions
18 lines changed: 5 ins; 12 del; 1 mod; 174 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/memory/cardTableModRefBS.cpp

rev 7318 : 8064721: The card tables only ever need two covering regions
3 lines changed: 0 ins; 1 del; 2 mod; 712 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/memory/cardTableModRefBS.hpp

rev 7318 : 8064721: The card tables only ever need two covering regions
4 lines changed: 0 ins; 1 del; 3 mod; 489 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/memory/cardTableRS.cpp

rev 7318 : 8064721: The card tables only ever need two covering regions
8 lines changed: 0 ins; 3 del; 5 mod; 624 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/memory/cardTableRS.hpp

rev 7318 : 8064721: The card tables only ever need two covering regions
rev 7319 : 8064721: The card tables only ever need two covering regions (01)
3 lines changed: 1 ins; 0 del; 2 mod; 188 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/memory/collectorPolicy.cpp

rev 7318 : 8064721: The card tables only ever need two covering regions
3 lines changed: 0 ins; 1 del; 2 mod; 1101 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/memory/collectorPolicy.hpp

rev 7318 : 8064721: The card tables only ever need two covering regions
4 lines changed: 0 ins; 3 del; 1 mod; 314 unchg

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

rev 7318 : 8064721: The card tables only ever need two covering regions
12 lines changed: 0 ins; 10 del; 2 mod; 1295 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/memory/genCollectedHeap.hpp

rev 7318 : 8064721: The card tables only ever need two covering regions
3 lines changed: 0 ins; 2 del; 1 mod; 526 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/memory/generationSpec.hpp

rev 7318 : 8064721: The card tables only ever need two covering regions
4 lines changed: 0 ins; 4 del; 0 mod; 66 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/memory/modRefBarrierSet.hpp

rev 7318 : 8064721: The card tables only ever need two covering regions
rev 7319 : 8064721: The card tables only ever need two covering regions (01)
4 lines changed: 0 ins; 4 del; 0 mod; 100 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/runtime/vmStructs.cpp

rev 7318 : 8064721: The card tables only ever need two covering regions
1 line changed: 0 ins; 1 del; 0 mod; 3366 unchg

This code review page was prepared using /home/ehelin/code/code-tools/webrev/webrev.ksh (vers 25.6-hg+openjdk.java.net).