pc_offset() >= pc_offset) failed: sanity 8008382: Remove redundant use of Atomic::add(jlong, jlong *) in create_new_gc_id() 8008546: Wrong G1ConfidencePercent results in GUARANTEE(VARIANCE() > -1.0) FAILED">

hs24-b34-jdk7u14-b15 (original) (raw)

Code Review for hs24-b34-jdk7u14-b15

Prepared by: amurillo on Thu Feb 28 11:29:12 PST 2013
Workspace: hs24-b34-jdk7u14-b15/hotspot
Compare against: jdk7u/jdk7u/hotspot
Compare against version: 4211
Summary of changes: 6074 lines changed: 6041 ins; 10 del; 23 mod; 45533 unchg
Patch of changes: <hotspot.patch>
Author comments: 8004867: VM crashing with assert "share/vm/opto/node.hpp:357 - assert(i < _max) failed: oob"8005772: Stubs report compile id -1 in phase events8006938: Change os::print_location to be more descriptive when a location is pointing into an object8007085: EnableTracing prints garbage for Compilation: [Java Method8007147: Trace event ExecuteVMOperation may get dangling pointer8007312: null check signal semaphore in os::signal_notify windows8007779: os::die() on solaris should generate core file8007804: Need to be able to access Performance counter by name from JVM8008071: Crashed in promote_malloc_records() with Kitchensink after 19 days8008102: SA on OS X does not stop the attached process8008284: new hotspot build - hs24-b348008340: [sampling] assert(upper->pc_offset() >= pc_offset) failed: sanity8008382: Remove redundant use of Atomic::add(jlong, jlong *) in create_new_gc_id()8008546: Wrong G1ConfidencePercent results in GUARANTEE(VARIANCE() > -1.0) FAILED
Legend: Modified file Deleted file New file

[Cdiffs](.hgtags.cdiff.html) [Udiffs](.hgtags.udiff.html) [Wdiffs](.hgtags.wdiff.html) [Sdiffs](.hgtags.sdiff.html) [Frames](.hgtags.frames.html) [Old](.hgtags-.html) [New](.hgtags.html) [Patch](.hgtags.patch) [Raw](raw%5Ffiles/new/.hgtags) .hgtags

rev 4252 : Added tag hs24-b34 for changeset 860ae068f4df

10 lines changed: 10 ins; 0 del; 0 mod; 436 unchg

[Cdiffs](agent/src/os/bsd/MacosxDebuggerLocal.m.cdiff.html) [Udiffs](agent/src/os/bsd/MacosxDebuggerLocal.m.udiff.html) [Wdiffs](agent/src/os/bsd/MacosxDebuggerLocal.m.wdiff.html) [Sdiffs](agent/src/os/bsd/MacosxDebuggerLocal.m.sdiff.html) [Frames](agent/src/os/bsd/MacosxDebuggerLocal.m.frames.html) [Old](agent/src/os/bsd/MacosxDebuggerLocal.m-.html) [New](agent/src/os/bsd/MacosxDebuggerLocal.m.html) [Patch](agent/src/os/bsd/MacosxDebuggerLocal.m.patch) [Raw](raw%5Ffiles/new/agent/src/os/bsd/MacosxDebuggerLocal.m) agent/src/os/bsd/MacosxDebuggerLocal.m

rev 4233 : 8008102: SA on OS X does not stop the attached process Reviewed-by: dholmes, rbackman

93 lines changed: 92 ins; 0 del; 1 mod; 405 unchg

[Cdiffs](make/hotspot%5Fversion.cdiff.html) [Udiffs](make/hotspot%5Fversion.udiff.html) [Wdiffs](make/hotspot%5Fversion.wdiff.html) [Sdiffs](make/hotspot%5Fversion.sdiff.html) [Frames](make/hotspot%5Fversion.frames.html) [Old](make/hotspot%5Fversion-.html) [New](make/hotspot%5Fversion.html) [Patch](make/hotspot%5Fversion.patch) [Raw](raw%5Ffiles/new/make/hotspot%5Fversion) make/hotspot_version

rev 4232 : 8008284: new hotspot build - hs24-b34 Reviewed-by: jcoomes rev 4251 : Merge

1 line changed: 0 ins; 0 del; 1 mod; 44 unchg

[Cdiffs](src/cpu/sparc/vm/frame%5Fsparc.cpp.cdiff.html) [Udiffs](src/cpu/sparc/vm/frame%5Fsparc.cpp.udiff.html) [Wdiffs](src/cpu/sparc/vm/frame%5Fsparc.cpp.wdiff.html) [Sdiffs](src/cpu/sparc/vm/frame%5Fsparc.cpp.sdiff.html) [Frames](src/cpu/sparc/vm/frame%5Fsparc.cpp.frames.html) [Old](src/cpu/sparc/vm/frame%5Fsparc.cpp-.html) [New](src/cpu/sparc/vm/frame%5Fsparc.cpp.html) [Patch](src/cpu/sparc/vm/frame%5Fsparc.cpp.patch) [Raw](raw%5Ffiles/new/src/cpu/sparc/vm/frame%5Fsparc.cpp) src/cpu/sparc/vm/frame_sparc.cpp

rev 4248 : 8008340: [sampling] assert(upper->pc_offset() >= pc_offset) failed: sanity Reviewed-by: kvn, sla

5 lines changed: 5 ins; 0 del; 0 mod; 852 unchg

[Cdiffs](src/cpu/x86/vm/frame%5Fx86.cpp.cdiff.html) [Udiffs](src/cpu/x86/vm/frame%5Fx86.cpp.udiff.html) [Wdiffs](src/cpu/x86/vm/frame%5Fx86.cpp.wdiff.html) [Sdiffs](src/cpu/x86/vm/frame%5Fx86.cpp.sdiff.html) [Frames](src/cpu/x86/vm/frame%5Fx86.cpp.frames.html) [Old](src/cpu/x86/vm/frame%5Fx86.cpp-.html) [New](src/cpu/x86/vm/frame%5Fx86.cpp.html) [Patch](src/cpu/x86/vm/frame%5Fx86.cpp.patch) [Raw](raw%5Ffiles/new/src/cpu/x86/vm/frame%5Fx86.cpp) src/cpu/x86/vm/frame_x86.cpp

rev 4248 : 8008340: [sampling] assert(upper->pc_offset() >= pc_offset) failed: sanity Reviewed-by: kvn, sla

11 lines changed: 6 ins; 5 del; 0 mod; 721 unchg

[Cdiffs](src/os/solaris/vm/os%5Fsolaris.cpp.cdiff.html) [Udiffs](src/os/solaris/vm/os%5Fsolaris.cpp.udiff.html) [Wdiffs](src/os/solaris/vm/os%5Fsolaris.cpp.wdiff.html) [Sdiffs](src/os/solaris/vm/os%5Fsolaris.cpp.sdiff.html) [Frames](src/os/solaris/vm/os%5Fsolaris.cpp.frames.html) [Old](src/os/solaris/vm/os%5Fsolaris.cpp-.html) [New](src/os/solaris/vm/os%5Fsolaris.cpp.html) [Patch](src/os/solaris/vm/os%5Fsolaris.cpp.patch) [Raw](raw%5Ffiles/new/src/os/solaris/vm/os%5Fsolaris.cpp) src/os/solaris/vm/os_solaris.cpp

rev 4234 : 8007779: os::die() on solaris should generate core file Reviewed-by: dholmes, rbackman

1 line changed: 0 ins; 0 del; 1 mod; 6653 unchg

[Cdiffs](src/os/windows/vm/os%5Fwindows.cpp.cdiff.html) [Udiffs](src/os/windows/vm/os%5Fwindows.cpp.udiff.html) [Wdiffs](src/os/windows/vm/os%5Fwindows.cpp.wdiff.html) [Sdiffs](src/os/windows/vm/os%5Fwindows.cpp.sdiff.html) [Frames](src/os/windows/vm/os%5Fwindows.cpp.frames.html) [Old](src/os/windows/vm/os%5Fwindows.cpp-.html) [New](src/os/windows/vm/os%5Fwindows.cpp.html) [Patch](src/os/windows/vm/os%5Fwindows.cpp.patch) [Raw](raw%5Ffiles/new/src/os/windows/vm/os%5Fwindows.cpp) src/os/windows/vm/os_windows.cpp

rev 4245 : 8007312: null check signal semaphore in os::signal_notify windows Reviewed-by: dholmes, sla

6 lines changed: 1 ins; 0 del; 5 mod; 5465 unchg

[Cdiffs](src/share/vm/compiler/compileBroker.cpp.cdiff.html) [Udiffs](src/share/vm/compiler/compileBroker.cpp.udiff.html) [Wdiffs](src/share/vm/compiler/compileBroker.cpp.wdiff.html) [Sdiffs](src/share/vm/compiler/compileBroker.cpp.sdiff.html) [Frames](src/share/vm/compiler/compileBroker.cpp.frames.html) [Old](src/share/vm/compiler/compileBroker.cpp-.html) [New](src/share/vm/compiler/compileBroker.cpp.html) [Patch](src/share/vm/compiler/compileBroker.cpp.patch) [Raw](raw%5Ffiles/new/src/share/vm/compiler/compileBroker.cpp) src/share/vm/compiler/compileBroker.cpp

rev 4237 : 8005772: Stubs report compile id -1 in phase events Summary: Use 0 to indicate id is NA, -1 for error or uninitalized Reviewed-by: kvn, twisti

1 line changed: 0 ins; 0 del; 1 mod; 2165 unchg

[Cdiffs](src/share/vm/gc%5Fimplementation/concurrentMarkSweep/concurrentMarkSweepGeneration.cpp.cdiff.html) [Udiffs](src/share/vm/gc%5Fimplementation/concurrentMarkSweep/concurrentMarkSweepGeneration.cpp.udiff.html) [Wdiffs](src/share/vm/gc%5Fimplementation/concurrentMarkSweep/concurrentMarkSweepGeneration.cpp.wdiff.html) [Sdiffs](src/share/vm/gc%5Fimplementation/concurrentMarkSweep/concurrentMarkSweepGeneration.cpp.sdiff.html) [Frames](src/share/vm/gc%5Fimplementation/concurrentMarkSweep/concurrentMarkSweepGeneration.cpp.frames.html) [Old](src/share/vm/gc%5Fimplementation/concurrentMarkSweep/concurrentMarkSweepGeneration.cpp-.html) [New](src/share/vm/gc%5Fimplementation/concurrentMarkSweep/concurrentMarkSweepGeneration.cpp.html) [Patch](src/share/vm/gc%5Fimplementation/concurrentMarkSweep/concurrentMarkSweepGeneration.cpp.patch) [Raw](raw%5Ffiles/new/src/share/vm/gc%5Fimplementation/concurrentMarkSweep/concurrentMarkSweepGeneration.cpp) src/share/vm/gc_implementation/concurrentMarkSweep/concurrentMarkSweepGeneration.cpp

rev 4235 : 8008382: Remove redundant use of Atomic::add(jlong, jlong *) in create_new_gc_id() Summary: Moving register_gc_start() in to the CMS VM operation makes sure that create_new_gc_id() is not called by multiple threads in parallel. This removes the need for atomics in create_new_gc_d(). Also, Atomic::add(jlong, jlong *) is unimplemented for ARM. Reviewed-by: stefank, dholmes, ehelin

3 lines changed: 0 ins; 3 del; 0 mod; 9392 unchg

[Cdiffs](src/share/vm/gc%5Fimplementation/concurrentMarkSweep/vmCMSOperations.cpp.cdiff.html) [Udiffs](src/share/vm/gc%5Fimplementation/concurrentMarkSweep/vmCMSOperations.cpp.udiff.html) [Wdiffs](src/share/vm/gc%5Fimplementation/concurrentMarkSweep/vmCMSOperations.cpp.wdiff.html) [Sdiffs](src/share/vm/gc%5Fimplementation/concurrentMarkSweep/vmCMSOperations.cpp.sdiff.html) [Frames](src/share/vm/gc%5Fimplementation/concurrentMarkSweep/vmCMSOperations.cpp.frames.html) [Old](src/share/vm/gc%5Fimplementation/concurrentMarkSweep/vmCMSOperations.cpp-.html) [New](src/share/vm/gc%5Fimplementation/concurrentMarkSweep/vmCMSOperations.cpp.html) [Patch](src/share/vm/gc%5Fimplementation/concurrentMarkSweep/vmCMSOperations.cpp.patch) [Raw](raw%5Ffiles/new/src/share/vm/gc%5Fimplementation/concurrentMarkSweep/vmCMSOperations.cpp) src/share/vm/gc_implementation/concurrentMarkSweep/vmCMSOperations.cpp

rev 4235 : 8008382: Remove redundant use of Atomic::add(jlong, jlong *) in create_new_gc_id() Summary: Moving register_gc_start() in to the CMS VM operation makes sure that create_new_gc_id() is not called by multiple threads in parallel. This removes the need for atomics in create_new_gc_d(). Also, Atomic::add(jlong, jlong *) is unimplemented for ARM. Reviewed-by: stefank, dholmes, ehelin

1 line changed: 1 ins; 0 del; 0 mod; 309 unchg

[Cdiffs](src/share/vm/gc%5Fimplementation/g1/g1CollectorPolicy.cpp.cdiff.html) [Udiffs](src/share/vm/gc%5Fimplementation/g1/g1CollectorPolicy.cpp.udiff.html) [Wdiffs](src/share/vm/gc%5Fimplementation/g1/g1CollectorPolicy.cpp.wdiff.html) [Sdiffs](src/share/vm/gc%5Fimplementation/g1/g1CollectorPolicy.cpp.sdiff.html) [Frames](src/share/vm/gc%5Fimplementation/g1/g1CollectorPolicy.cpp.frames.html) [Old](src/share/vm/gc%5Fimplementation/g1/g1CollectorPolicy.cpp-.html) [New](src/share/vm/gc%5Fimplementation/g1/g1CollectorPolicy.cpp.html) [Patch](src/share/vm/gc%5Fimplementation/g1/g1CollectorPolicy.cpp.patch) [Raw](raw%5Ffiles/new/src/share/vm/gc%5Fimplementation/g1/g1CollectorPolicy.cpp) src/share/vm/gc_implementation/g1/g1CollectorPolicy.cpp

rev 4249 : 8008546: Wrong G1ConfidencePercent results in GUARANTEE(VARIANCE() > -1.0) FAILED Reviewed-by: brutisso, johnc Contributed-by: vladimir.kempik@oracle.com

10 lines changed: 9 ins; 1 del; 0 mod; 2142 unchg

[Cdiffs](src/share/vm/gc%5Fimplementation/g1/g1%5Fglobals.hpp.cdiff.html) [Udiffs](src/share/vm/gc%5Fimplementation/g1/g1%5Fglobals.hpp.udiff.html) [Wdiffs](src/share/vm/gc%5Fimplementation/g1/g1%5Fglobals.hpp.wdiff.html) [Sdiffs](src/share/vm/gc%5Fimplementation/g1/g1%5Fglobals.hpp.sdiff.html) [Frames](src/share/vm/gc%5Fimplementation/g1/g1%5Fglobals.hpp.frames.html) [Old](src/share/vm/gc%5Fimplementation/g1/g1%5Fglobals.hpp-.html) [New](src/share/vm/gc%5Fimplementation/g1/g1%5Fglobals.hpp.html) [Patch](src/share/vm/gc%5Fimplementation/g1/g1%5Fglobals.hpp.patch) [Raw](raw%5Ffiles/new/src/share/vm/gc%5Fimplementation/g1/g1%5Fglobals.hpp) src/share/vm/gc_implementation/g1/g1_globals.hpp

rev 4249 : 8008546: Wrong G1ConfidencePercent results in GUARANTEE(VARIANCE() > -1.0) FAILED Reviewed-by: brutisso, johnc Contributed-by: vladimir.kempik@oracle.com

1 line changed: 0 ins; 0 del; 1 mod; 346 unchg

[Cdiffs](src/share/vm/gc%5Fimplementation/shared/gcTrace.cpp.cdiff.html) [Udiffs](src/share/vm/gc%5Fimplementation/shared/gcTrace.cpp.udiff.html) [Wdiffs](src/share/vm/gc%5Fimplementation/shared/gcTrace.cpp.wdiff.html) [Sdiffs](src/share/vm/gc%5Fimplementation/shared/gcTrace.cpp.sdiff.html) [Frames](src/share/vm/gc%5Fimplementation/shared/gcTrace.cpp.frames.html) [Old](src/share/vm/gc%5Fimplementation/shared/gcTrace.cpp-.html) [New](src/share/vm/gc%5Fimplementation/shared/gcTrace.cpp.html) [Patch](src/share/vm/gc%5Fimplementation/shared/gcTrace.cpp.patch) [Raw](raw%5Ffiles/new/src/share/vm/gc%5Fimplementation/shared/gcTrace.cpp) src/share/vm/gc_implementation/shared/gcTrace.cpp

rev 4235 : 8008382: Remove redundant use of Atomic::add(jlong, jlong *) in create_new_gc_id() Summary: Moving register_gc_start() in to the CMS VM operation makes sure that create_new_gc_id() is not called by multiple threads in parallel. This removes the need for atomics in create_new_gc_d(). Also, Atomic::add(jlong, jlong *) is unimplemented for ARM. Reviewed-by: stefank, dholmes, ehelin

3 lines changed: 0 ins; 1 del; 2 mod; 140 unchg

[Cdiffs](src/share/vm/interpreter/linkResolver.cpp.cdiff.html) [Udiffs](src/share/vm/interpreter/linkResolver.cpp.udiff.html) [Wdiffs](src/share/vm/interpreter/linkResolver.cpp.wdiff.html) [Sdiffs](src/share/vm/interpreter/linkResolver.cpp.sdiff.html) [Frames](src/share/vm/interpreter/linkResolver.cpp.frames.html) [Old](src/share/vm/interpreter/linkResolver.cpp-.html) [New](src/share/vm/interpreter/linkResolver.cpp.html) [Patch](src/share/vm/interpreter/linkResolver.cpp.patch) [Raw](raw%5Ffiles/new/src/share/vm/interpreter/linkResolver.cpp) src/share/vm/interpreter/linkResolver.cpp

1 line changed: 0 ins; 0 del; 1 mod; 1306 unchg

[Cdiffs](src/share/vm/opto/compile.cpp.cdiff.html) [Udiffs](src/share/vm/opto/compile.cpp.udiff.html) [Wdiffs](src/share/vm/opto/compile.cpp.wdiff.html) [Sdiffs](src/share/vm/opto/compile.cpp.sdiff.html) [Frames](src/share/vm/opto/compile.cpp.frames.html) [Old](src/share/vm/opto/compile.cpp-.html) [New](src/share/vm/opto/compile.cpp.html) [Patch](src/share/vm/opto/compile.cpp.patch) [Raw](raw%5Ffiles/new/src/share/vm/opto/compile.cpp) src/share/vm/opto/compile.cpp

rev 4237 : 8005772: Stubs report compile id -1 in phase events Summary: Use 0 to indicate id is NA, -1 for error or uninitalized Reviewed-by: kvn, twisti

1 line changed: 0 ins; 0 del; 1 mod; 3511 unchg

[Cdiffs](src/share/vm/opto/superword.cpp.cdiff.html) [Udiffs](src/share/vm/opto/superword.cpp.udiff.html) [Wdiffs](src/share/vm/opto/superword.cpp.wdiff.html) [Sdiffs](src/share/vm/opto/superword.cpp.sdiff.html) [Frames](src/share/vm/opto/superword.cpp.frames.html) [Old](src/share/vm/opto/superword.cpp-.html) [New](src/share/vm/opto/superword.cpp.html) [Patch](src/share/vm/opto/superword.cpp.patch) [Raw](raw%5Ffiles/new/src/share/vm/opto/superword.cpp) src/share/vm/opto/superword.cpp

rev 4242 : 8004867: VM crashing with assert "share/vm/opto/node.hpp:357 - assert(i < _max) failed: oob" Summary: Added few checks and early bailout from Superword optimization to avoid such cases in a future. Reviewed-by: roland, twisti

15 lines changed: 13 ins; 0 del; 2 mod; 2531 unchg

[Cdiffs](src/share/vm/opto/superword.hpp.cdiff.html) [Udiffs](src/share/vm/opto/superword.hpp.udiff.html) [Wdiffs](src/share/vm/opto/superword.hpp.wdiff.html) [Sdiffs](src/share/vm/opto/superword.hpp.sdiff.html) [Frames](src/share/vm/opto/superword.hpp.frames.html) [Old](src/share/vm/opto/superword.hpp-.html) [New](src/share/vm/opto/superword.hpp.html) [Patch](src/share/vm/opto/superword.hpp.patch) [Raw](raw%5Ffiles/new/src/share/vm/opto/superword.hpp) src/share/vm/opto/superword.hpp

rev 4242 : 8004867: VM crashing with assert "share/vm/opto/node.hpp:357 - assert(i < _max) failed: oob" Summary: Added few checks and early bailout from Superword optimization to avoid such cases in a future. Reviewed-by: roland, twisti

1 line changed: 0 ins; 0 del; 1 mod; 531 unchg

[Cdiffs](src/share/vm/runtime/globals.hpp.cdiff.html) [Udiffs](src/share/vm/runtime/globals.hpp.udiff.html) [Wdiffs](src/share/vm/runtime/globals.hpp.wdiff.html) [Sdiffs](src/share/vm/runtime/globals.hpp.sdiff.html) [Frames](src/share/vm/runtime/globals.hpp.frames.html) [Old](src/share/vm/runtime/globals.hpp-.html) [New](src/share/vm/runtime/globals.hpp.html) [Patch](src/share/vm/runtime/globals.hpp.patch) [Raw](raw%5Ffiles/new/src/share/vm/runtime/globals.hpp) src/share/vm/runtime/globals.hpp

3 lines changed: 3 ins; 0 del; 0 mod; 3690 unchg

[Cdiffs](src/share/vm/runtime/os.cpp.cdiff.html) [Udiffs](src/share/vm/runtime/os.cpp.udiff.html) [Wdiffs](src/share/vm/runtime/os.cpp.wdiff.html) [Sdiffs](src/share/vm/runtime/os.cpp.sdiff.html) [Frames](src/share/vm/runtime/os.cpp.frames.html) [Old](src/share/vm/runtime/os.cpp-.html) [New](src/share/vm/runtime/os.cpp.html) [Patch](src/share/vm/runtime/os.cpp.patch) [Raw](raw%5Ffiles/new/src/share/vm/runtime/os.cpp) src/share/vm/runtime/os.cpp

rev 4236 : 8006938: Change os::print_location to be more descriptive when a location is pointing into an object Reviewed-by: stefank, twisti, poonam Contributed-by: sergey.gabdurakhmanov@oracle.com

5 lines changed: 4 ins; 0 del; 1 mod; 1482 unchg

[Cdiffs](src/share/vm/runtime/perfData.cpp.cdiff.html) [Udiffs](src/share/vm/runtime/perfData.cpp.udiff.html) [Wdiffs](src/share/vm/runtime/perfData.cpp.wdiff.html) [Sdiffs](src/share/vm/runtime/perfData.cpp.sdiff.html) [Frames](src/share/vm/runtime/perfData.cpp.frames.html) [Old](src/share/vm/runtime/perfData.cpp-.html) [New](src/share/vm/runtime/perfData.cpp.html) [Patch](src/share/vm/runtime/perfData.cpp.patch) [Raw](raw%5Ffiles/new/src/share/vm/runtime/perfData.cpp) src/share/vm/runtime/perfData.cpp

rev 4238 : 8007804: Need to be able to access Performance counter by name from JVM Reviewed-by: dholmes, sla, sspitsyn

4 lines changed: 4 ins; 0 del; 0 mod; 603 unchg

[Cdiffs](src/share/vm/runtime/perfData.hpp.cdiff.html) [Udiffs](src/share/vm/runtime/perfData.hpp.udiff.html) [Wdiffs](src/share/vm/runtime/perfData.hpp.wdiff.html) [Sdiffs](src/share/vm/runtime/perfData.hpp.sdiff.html) [Frames](src/share/vm/runtime/perfData.hpp.frames.html) [Old](src/share/vm/runtime/perfData.hpp-.html) [New](src/share/vm/runtime/perfData.hpp.html) [Patch](src/share/vm/runtime/perfData.hpp.patch) [Raw](raw%5Ffiles/new/src/share/vm/runtime/perfData.hpp) src/share/vm/runtime/perfData.hpp

rev 4238 : 8007804: Need to be able to access Performance counter by name from JVM Reviewed-by: dholmes, sla, sspitsyn

3 lines changed: 3 ins; 0 del; 0 mod; 969 unchg

[Cdiffs](src/share/vm/runtime/vmThread.cpp.cdiff.html) [Udiffs](src/share/vm/runtime/vmThread.cpp.udiff.html) [Wdiffs](src/share/vm/runtime/vmThread.cpp.wdiff.html) [Sdiffs](src/share/vm/runtime/vmThread.cpp.sdiff.html) [Frames](src/share/vm/runtime/vmThread.cpp.frames.html) [Old](src/share/vm/runtime/vmThread.cpp-.html) [New](src/share/vm/runtime/vmThread.cpp.html) [Patch](src/share/vm/runtime/vmThread.cpp.patch) [Raw](raw%5Ffiles/new/src/share/vm/runtime/vmThread.cpp) src/share/vm/runtime/vmThread.cpp

rev 4244 : 8007147: Trace event ExecuteVMOperation may get dangling pointer Reviewed-by: dholmes, sla

6 lines changed: 4 ins; 0 del; 2 mod; 716 unchg

[Cdiffs](src/share/vm/services/memSnapshot.cpp.cdiff.html) [Udiffs](src/share/vm/services/memSnapshot.cpp.udiff.html) [Wdiffs](src/share/vm/services/memSnapshot.cpp.wdiff.html) [Sdiffs](src/share/vm/services/memSnapshot.cpp.sdiff.html) [Frames](src/share/vm/services/memSnapshot.cpp.frames.html) [Old](src/share/vm/services/memSnapshot.cpp-.html) [New](src/share/vm/services/memSnapshot.cpp.html) [Patch](src/share/vm/services/memSnapshot.cpp.patch) [Raw](raw%5Ffiles/new/src/share/vm/services/memSnapshot.cpp) src/share/vm/services/memSnapshot.cpp

rev 4243 : 8008071: Crashed in promote_malloc_records() with Kitchensink after 19 days Summary: Added NULL pointer check for arena size record Reviewed-by: sspitsyn, dholmes

2 lines changed: 1 ins; 0 del; 1 mod; 729 unchg

[Cdiffs](src/share/vm/trace/trace.xml.cdiff.html) [Udiffs](src/share/vm/trace/trace.xml.udiff.html) [Wdiffs](src/share/vm/trace/trace.xml.wdiff.html) [Sdiffs](src/share/vm/trace/trace.xml.sdiff.html) [Frames](src/share/vm/trace/trace.xml.frames.html) [Old](src/share/vm/trace/trace.xml-.html) [New](src/share/vm/trace/trace.xml.html) [Patch](src/share/vm/trace/trace.xml.patch) [Raw](raw%5Ffiles/new/src/share/vm/trace/trace.xml) src/share/vm/trace/trace.xml

rev 4244 : 8007147: Trace event ExecuteVMOperation may get dangling pointer Reviewed-by: dholmes, sla

1 line changed: 0 ins; 0 del; 1 mod; 297 unchg

[Cdiffs](src/share/vm/trace/traceStream.hpp.cdiff.html) [Udiffs](src/share/vm/trace/traceStream.hpp.udiff.html) [Wdiffs](src/share/vm/trace/traceStream.hpp.wdiff.html) [Sdiffs](src/share/vm/trace/traceStream.hpp.sdiff.html) [Frames](src/share/vm/trace/traceStream.hpp.frames.html) [Old](src/share/vm/trace/traceStream.hpp-.html) [New](src/share/vm/trace/traceStream.hpp.html) [Patch](src/share/vm/trace/traceStream.hpp.patch) [Raw](raw%5Ffiles/new/src/share/vm/trace/traceStream.hpp) src/share/vm/trace/traceStream.hpp

rev 4246 : 8007085: EnableTracing prints garbage for Compilation: [Java Method Reviewed-by: coleenp, sla Contributed-by: markus.gronlund@oracle.com

1 line changed: 0 ins; 0 del; 1 mod; 98 unchg

------ ------ ------ ------ ------ ---[New](test/compiler/8004867/TestIntAtomicCAS.java.html) [Patch](test/compiler/8004867/TestIntAtomicCAS.java.patch) [Raw](raw%5Ffiles/new/test/compiler/8004867/TestIntAtomicCAS.java) test/compiler/8004867/TestIntAtomicCAS.java

rev 4242 : 8004867: VM crashing with assert "share/vm/opto/node.hpp:357 - assert(i < _max) failed: oob" Summary: Added few checks and early bailout from Superword optimization to avoid such cases in a future. Reviewed-by: roland, twisti

969 lines changed: 969 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ ---[New](test/compiler/8004867/TestIntAtomicOrdered.java.html) [Patch](test/compiler/8004867/TestIntAtomicOrdered.java.patch) [Raw](raw%5Ffiles/new/test/compiler/8004867/TestIntAtomicOrdered.java) test/compiler/8004867/TestIntAtomicOrdered.java

rev 4242 : 8004867: VM crashing with assert "share/vm/opto/node.hpp:357 - assert(i < _max) failed: oob" Summary: Added few checks and early bailout from Superword optimization to avoid such cases in a future. Reviewed-by: roland, twisti

969 lines changed: 969 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ ---[New](test/compiler/8004867/TestIntAtomicVolatile.java.html) [Patch](test/compiler/8004867/TestIntAtomicVolatile.java.patch) [Raw](raw%5Ffiles/new/test/compiler/8004867/TestIntAtomicVolatile.java) test/compiler/8004867/TestIntAtomicVolatile.java

rev 4242 : 8004867: VM crashing with assert "share/vm/opto/node.hpp:357 - assert(i < _max) failed: oob" Summary: Added few checks and early bailout from Superword optimization to avoid such cases in a future. Reviewed-by: roland, twisti

969 lines changed: 969 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ ---[New](test/compiler/8004867/TestIntUnsafeCAS.java.html) [Patch](test/compiler/8004867/TestIntUnsafeCAS.java.patch) [Raw](raw%5Ffiles/new/test/compiler/8004867/TestIntUnsafeCAS.java) test/compiler/8004867/TestIntUnsafeCAS.java

rev 4242 : 8004867: VM crashing with assert "share/vm/opto/node.hpp:357 - assert(i < _max) failed: oob" Summary: Added few checks and early bailout from Superword optimization to avoid such cases in a future. Reviewed-by: roland, twisti

998 lines changed: 998 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ ---[New](test/compiler/8004867/TestIntUnsafeOrdered.java.html) [Patch](test/compiler/8004867/TestIntUnsafeOrdered.java.patch) [Raw](raw%5Ffiles/new/test/compiler/8004867/TestIntUnsafeOrdered.java) test/compiler/8004867/TestIntUnsafeOrdered.java

rev 4242 : 8004867: VM crashing with assert "share/vm/opto/node.hpp:357 - assert(i < _max) failed: oob" Summary: Added few checks and early bailout from Superword optimization to avoid such cases in a future. Reviewed-by: roland, twisti

990 lines changed: 990 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ ---[New](test/compiler/8004867/TestIntUnsafeVolatile.java.html) [Patch](test/compiler/8004867/TestIntUnsafeVolatile.java.patch) [Raw](raw%5Ffiles/new/test/compiler/8004867/TestIntUnsafeVolatile.java) test/compiler/8004867/TestIntUnsafeVolatile.java

rev 4242 : 8004867: VM crashing with assert "share/vm/opto/node.hpp:357 - assert(i < _max) failed: oob" Summary: Added few checks and early bailout from Superword optimization to avoid such cases in a future. Reviewed-by: roland, twisti

990 lines changed: 990 ins; 0 del; 0 mod; 0 unchg


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