Code Review for 6458402

Prepared by:kamg on Thu Dec 30 16:52:11 EST 2010
Workspace:/java/east/u3/kamg/ws/svc/6458402
Compare against: http://hg.openjdk.java.net/jdk7/hotspot-rt/hotspot
Compare against version:1951
Summary of changes: 504 lines changed: 53 ins; 336 del; 115 mod; 23906 unchg
Patch of changes: 6458402.patch
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/gc_implementation/concurrentMarkSweep/concurrentMarkSweepGeneration.cpp

rev 1952 : 6458402: 3 jvmti tests fail with CMS and +ExplicitGCInvokesConcurrent
Summary: Make JvmtiGCMark safe to run non-safepoint and instrument CMS
Reviewed-by:
5 lines changed: 0 ins; 5 del; 0 mod; 9281 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/gc_implementation/concurrentMarkSweep/vmCMSOperations.cpp

rev 1952 : 6458402: 3 jvmti tests fail with CMS and +ExplicitGCInvokesConcurrent
Summary: Make JvmtiGCMark safe to run non-safepoint and instrument CMS
Reviewed-by:
6 lines changed: 6 ins; 0 del; 0 mod; 264 unchg

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

rev 1952 : 6458402: 3 jvmti tests fail with CMS and +ExplicitGCInvokesConcurrent
Summary: Make JvmtiGCMark safe to run non-safepoint and instrument CMS
Reviewed-by:
6 lines changed: 3 ins; 0 del; 3 mod; 179 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/gc_implementation/parallelScavenge/vmPSOperations.cpp

rev 1952 : 6458402: 3 jvmti tests fail with CMS and +ExplicitGCInvokesConcurrent
Summary: Make JvmtiGCMark safe to run non-safepoint and instrument CMS
Reviewed-by:
10 lines changed: 0 ins; 7 del; 3 mod; 100 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/gc_implementation/shared/vmGCOperations.cpp

rev 1952 : 6458402: 3 jvmti tests fail with CMS and +ExplicitGCInvokesConcurrent
Summary: Make JvmtiGCMark safe to run non-safepoint and instrument CMS
Reviewed-by:
11 lines changed: 1 ins; 6 del; 4 mod; 203 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/gc_implementation/shared/vmGCOperations.hpp

rev 1952 : 6458402: 3 jvmti tests fail with CMS and +ExplicitGCInvokesConcurrent
Summary: Make JvmtiGCMark safe to run non-safepoint and instrument CMS
Reviewed-by:
16 lines changed: 16 ins; 0 del; 0 mod; 212 unchg

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

rev 1952 : 6458402: 3 jvmti tests fail with CMS and +ExplicitGCInvokesConcurrent
Summary: Make JvmtiGCMark safe to run non-safepoint and instrument CMS
Reviewed-by:
15 lines changed: 8 ins; 4 del; 3 mod; 1488 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/prims/jvmtiExport.cpp

rev 1952 : 6458402: 3 jvmti tests fail with CMS and +ExplicitGCInvokesConcurrent
Summary: Make JvmtiGCMark safe to run non-safepoint and instrument CMS
Reviewed-by:
45 lines changed: 0 ins; 41 del; 4 mod; 2553 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/prims/jvmtiExport.hpp

rev 1952 : 6458402: 3 jvmti tests fail with CMS and +ExplicitGCInvokesConcurrent
Summary: Make JvmtiGCMark safe to run non-safepoint and instrument CMS
Reviewed-by:
49 lines changed: 0 ins; 47 del; 2 mod; 520 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/prims/jvmtiImpl.cpp

rev 1952 : 6458402: 3 jvmti tests fail with CMS and +ExplicitGCInvokesConcurrent
Summary: Make JvmtiGCMark safe to run non-safepoint and instrument CMS
Reviewed-by:
20 lines changed: 1 ins; 19 del; 0 mod; 895 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/prims/jvmtiImpl.hpp

rev 1952 : 6458402: 3 jvmti tests fail with CMS and +ExplicitGCInvokesConcurrent
Summary: Make JvmtiGCMark safe to run non-safepoint and instrument CMS
Reviewed-by:
4 lines changed: 0 ins; 4 del; 0 mod; 434 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/prims/jvmtiTagMap.cpp

rev 1952 : 6458402: 3 jvmti tests fail with CMS and +ExplicitGCInvokesConcurrent
Summary: Make JvmtiGCMark safe to run non-safepoint and instrument CMS
Reviewed-by:
293 lines changed: 16 ins; 187 del; 90 mod; 3302 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/prims/jvmtiTagMap.hpp

rev 1952 : 6458402: 3 jvmti tests fail with CMS and +ExplicitGCInvokesConcurrent
Summary: Make JvmtiGCMark safe to run non-safepoint and instrument CMS
Reviewed-by:
19 lines changed: 0 ins; 13 del; 6 mod; 125 unchg

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

rev 1952 : 6458402: 3 jvmti tests fail with CMS and +ExplicitGCInvokesConcurrent
Summary: Make JvmtiGCMark safe to run non-safepoint and instrument CMS
Reviewed-by:
3 lines changed: 0 ins; 3 del; 0 mod; 3758 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/runtime/jniHandles.cpp

rev 1952 : 6458402: 3 jvmti tests fail with CMS and +ExplicitGCInvokesConcurrent
Summary: Make JvmtiGCMark safe to run non-safepoint and instrument CMS
Reviewed-by:
2 lines changed: 2 ins; 0 del; 0 mod; 592 unchg

This code review page was prepared using /home/kamg/bin/hg_webrev (vers 23.18-hg-never).