Code Review for g1-mark-stack-size

Prepared by:johnc on Tue Nov 27 15:19:38 PST 2012
Workspace:/export/workspaces/g1-mark-stack-size
Compare against: http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot
Compare against version:1356be4c000b
Summary of changes: 61 lines changed: 48 ins; 10 del; 3 mod; 6047 unchg
Patch of changes: g1-mark-stack-size.patch
Legend: Modified file
Deleted file
New file

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

rev 3708 : 8000244: G1: Ergonomically set MarkStackSize and use virtual space for global marking stack
Summary: Set the value of MarkStackSize to a value based on the number of parallel marking threads with a reasonable minimum. Expand the marking stack if we have to restart marking due to an overflow up to a reasonable maximum. Allocate the underlying space for the marking stack from virtual memory.
Reviewed-by: jmasa
rev 3709 : imported patch reuse-old-marking-stack
55 lines changed: 43 ins; 10 del; 2 mod; 4558 unchg

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

rev 3708 : 8000244: G1: Ergonomically set MarkStackSize and use virtual space for global marking stack
Summary: Set the value of MarkStackSize to a value based on the number of parallel marking threads with a reasonable minimum. Expand the marking stack if we have to restart marking due to an overflow up to a reasonable maximum. Allocate the underlying space for the marking stack from virtual memory.
Reviewed-by: jmasa
rev 3709 : imported patch reuse-old-marking-stack
2 lines changed: 1 ins; 0 del; 1 mod; 1259 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw src/share/vm/runtime/virtualspace.hpp

rev 3709 : imported patch reuse-old-marking-stack
4 lines changed: 4 ins; 0 del; 0 mod; 230 unchg

This code review page was prepared using /java/j2se/bin/webrev (vers 23.18-hg).