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).