Code Review for hs25.40-b20-jdk8u40-b16

Prepared by:amurillo on Fri Nov 21 11:59:48 PST 2014
Workspace:hs25.40-b20/hs25.40-b20-jdk8u40-b16
Compare against: jdk8u/jdk8u
Summary of changes: 2891 lines changed: 1181 ins; 1102 del; 608 mod; 114786 unchg
Patch of changes: hs25.40-b20-jdk8u40-b16.patch
Author comments:
8033602: wrong stabs data in libjvm.debuginfo on JDK 8 - SPARC
8034005: cannot debug in synchronizer.o or objectMonitor.o on Solaris X86
8048169: Change 8037816 breaks HS build on PPC64 and CPP-Interpreter platforms
8054008: Using -XX:-LazyBootClassLoader crashes with ACCESS_VIOLATION on Win 64bit.
8056049: getProcessCpuLoad() stops working in one process when a different process exits
8056071: compiler/whitebox/IsMethodCompilableTest.java fails with 'method() is not compilable after 3 iterations'
8057622: java/util/stream/test/org/openjdk/tests/java/util/stream/InfiniteStreamWithLimitOpTest: SEGV inside compiled code (sparc)
8058209: Race in G1 card scanning could allow scanning of memory covered by PLABs
8058251: assert(_count > 0) failed: Negative counter when running runtime/NMT/MallocTrackingVerify.java
8059131: sawindbg.dll is not compiled with /SAFESEH
8059803: Update use of GetVersionEx to get correct Windows version in hs_err files
8060467: CMS: small OldPLABSize and -XX:-ResizePLAB cause assert(ResizePLAB || n_blks == OldPLABSize) failed: Error
8061969: [TESTBUG] MallocSiteHashOverflow.java should be enabled for 32-bit platforms
8062036: ConcurrentMarkThread::slt may be invoked before ConcurrentMarkThread::makeSurrogateLockerThread causing intermittent crashes
8062247: [TESTBUG] Allow WhiteBox test to access JVM offsets
8062870: src/share/vm/services/mallocTracker.hpp:64 assert(_count > 0) failed: Negative counter
8062950: Bug in locking code when UseOptoBiasInlining is disabled: assert(dmw->is_neutral()) failed: invariant
8064361: new hotspot build - hs25.40-b19
8064375: Change certain errors to warnings in CDS output.
8064556: G1: ParallelGCThreads=0 may cause assert(!MetadataOnStackMark::has_buffer_for_thread(Thread::current())) failed: Should be empty
8064667: Add -XX:+CheckEndorsedAndExtDirs flag to JDK 8
8064854: new hotspot build - hs25.40-b20
8065227: Report allocation context stats at end of cleanup
8065305: Make it possible to extend the G1CollectorPolicy
8065361: Fixup headers and definitions for INCLUDE_TRACE
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw jdk/src/windows/native/sun/management/OperatingSystemImpl.c

rev 10261 : 8056049: getProcessCpuLoad() stops working in one process when a different process exits
Reviewed-by: ctornqvi
1206 lines changed: 629 ins; 207 del; 370 mod; 364 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/.hgtags

rev 6894 : Added tag hs25.40-b19 for changeset 3a8a0fd171c5
rev 6913 : Merge
rev 6914 : Added tag hs25.40-b20 for changeset aa2442f89230
2 lines changed: 2 ins; 0 del; 0 mod; 556 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/make/hotspot_version

rev 6882 : 8064361: new hotspot build - hs25.40-b19
Reviewed-by: jcoomes
rev 6895 : 8064854: new hotspot build - hs25.40-b20
Reviewed-by: jcoomes
1 line changed: 0 ins; 0 del; 1 mod; 44 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/make/solaris/makefiles/defs.make

rev 6896 : 8033602: wrong stabs data in libjvm.debuginfo on JDK 8 - SPARC
8034005: cannot debug in synchronizer.o or objectMonitor.o on Solaris X86
Summary: Solaris needs objcopy version of 2.21.1 or newer is needed to create valid .debuginfo files.
Reviewed-by: dsamersoff, sspitsyn, dholmes, ihse
50 lines changed: 49 ins; 0 del; 1 mod; 256 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/make/solaris/makefiles/dtrace.make

rev 6896 : 8033602: wrong stabs data in libjvm.debuginfo on JDK 8 - SPARC
8034005: cannot debug in synchronizer.o or objectMonitor.o on Solaris X86
Summary: Solaris needs objcopy version of 2.21.1 or newer is needed to create valid .debuginfo files.
Reviewed-by: dsamersoff, sspitsyn, dholmes, ihse
30 lines changed: 0 ins; 21 del; 9 mod; 353 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/make/solaris/makefiles/jsig.make

rev 6896 : 8033602: wrong stabs data in libjvm.debuginfo on JDK 8 - SPARC
8034005: cannot debug in synchronizer.o or objectMonitor.o on Solaris X86
Summary: Solaris needs objcopy version of 2.21.1 or newer is needed to create valid .debuginfo files.
Reviewed-by: dsamersoff, sspitsyn, dholmes, ihse
12 lines changed: 0 ins; 9 del; 3 mod; 76 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/make/solaris/makefiles/saproc.make

rev 6896 : 8033602: wrong stabs data in libjvm.debuginfo on JDK 8 - SPARC
8034005: cannot debug in synchronizer.o or objectMonitor.o on Solaris X86
Summary: Solaris needs objcopy version of 2.21.1 or newer is needed to create valid .debuginfo files.
Reviewed-by: dsamersoff, sspitsyn, dholmes, ihse
12 lines changed: 0 ins; 9 del; 3 mod; 147 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/make/solaris/makefiles/vm.make

rev 6896 : 8033602: wrong stabs data in libjvm.debuginfo on JDK 8 - SPARC
8034005: cannot debug in synchronizer.o or objectMonitor.o on Solaris X86
Summary: Solaris needs objcopy version of 2.21.1 or newer is needed to create valid .debuginfo files.
Reviewed-by: dsamersoff, sspitsyn, dholmes, ihse
20 lines changed: 0 ins; 17 del; 3 mod; 345 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/make/windows/makefiles/compile.make

rev 6885 : 8059803: Update use of GetVersionEx to get correct Windows version in hs_err files
Summary: Update use of GetVersionEx to get correct Windows version in hs_err files
Reviewed-by: dcubed, gtriantafill
1 line changed: 0 ins; 0 del; 1 mod; 301 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/make/windows/makefiles/sa.make

rev 6897 : 8059131: sawindbg.dll is not compiled with /SAFESEH
Summary: Make variable SAFESEH_FLAG replaced with /SAFESEH link option.
Reviewed-by: mgronlun, sla
1 line changed: 0 ins; 0 del; 1 mod; 148 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/cpu/ppc/vm/assembler_ppc.cpp

rev 6900 : 8048169: Change 8037816 breaks HS build on PPC64 and CPP-Interpreter platforms
Summary: Fix the matching of format string parameter types to the actual argument types for the PPC64 and CPP-Interpreter files in the same way as 8037816 already did it for all the other files
Reviewed-by: stefank, coleenp, dholmes
2 lines changed: 0 ins; 0 del; 2 mod; 698 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/cpu/ppc/vm/compiledIC_ppc.cpp

rev 6900 : 8048169: Change 8037816 breaks HS build on PPC64 and CPP-Interpreter platforms
Summary: Fix the matching of format string parameter types to the actual argument types for the PPC64 and CPP-Interpreter files in the same way as 8037816 already did it for all the other files
Reviewed-by: stefank, coleenp, dholmes
2 lines changed: 0 ins; 0 del; 2 mod; 231 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/cpu/ppc/vm/frame_ppc.cpp

rev 6900 : 8048169: Change 8037816 breaks HS build on PPC64 and CPP-Interpreter platforms
Summary: Fix the matching of format string parameter types to the actual argument types for the PPC64 and CPP-Interpreter files in the same way as 8037816 already did it for all the other files
Reviewed-by: stefank, coleenp, dholmes
2 lines changed: 0 ins; 0 del; 2 mod; 318 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/cpu/ppc/vm/macroAssembler_ppc.cpp

rev 6900 : 8048169: Change 8037816 breaks HS build on PPC64 and CPP-Interpreter platforms
Summary: Fix the matching of format string parameter types to the actual argument types for the PPC64 and CPP-Interpreter files in the same way as 8037816 already did it for all the other files
Reviewed-by: stefank, coleenp, dholmes
2 lines changed: 0 ins; 0 del; 2 mod; 3165 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/cpu/ppc/vm/methodHandles_ppc.cpp

rev 6900 : 8048169: Change 8037816 breaks HS build on PPC64 and CPP-Interpreter platforms
Summary: Fix the matching of format string parameter types to the actual argument types for the PPC64 and CPP-Interpreter files in the same way as 8037816 already did it for all the other files
Reviewed-by: stefank, coleenp, dholmes
2 lines changed: 0 ins; 0 del; 2 mod; 556 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/cpu/ppc/vm/nativeInst_ppc.cpp

rev 6900 : 8048169: Change 8037816 breaks HS build on PPC64 and CPP-Interpreter platforms
Summary: Fix the matching of format string parameter types to the actual argument types for the PPC64 and CPP-Interpreter files in the same way as 8037816 already did it for all the other files
Reviewed-by: stefank, coleenp, dholmes
10 lines changed: 0 ins; 0 del; 10 mod; 381 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/cpu/ppc/vm/ppc.ad

rev 6900 : 8048169: Change 8037816 breaks HS build on PPC64 and CPP-Interpreter platforms
Summary: Fix the matching of format string parameter types to the actual argument types for the PPC64 and CPP-Interpreter files in the same way as 8037816 already did it for all the other files
Reviewed-by: stefank, coleenp, dholmes
1 line changed: 0 ins; 0 del; 1 mod; 12124 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/cpu/ppc/vm/vm_version_ppc.cpp

rev 6900 : 8048169: Change 8037816 breaks HS build on PPC64 and CPP-Interpreter platforms
Summary: Fix the matching of format string parameter types to the actual argument types for the PPC64 and CPP-Interpreter files in the same way as 8037816 already did it for all the other files
Reviewed-by: stefank, coleenp, dholmes
3 lines changed: 0 ins; 0 del; 3 mod; 485 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/cpu/x86/vm/macroAssembler_x86.cpp

rev 6890 : 8062950: Bug in locking code when UseOptoBiasInlining is disabled: assert(dmw->is_neutral()) failed: invariant
Reviewed-by: dholmes, kvn
1 line changed: 0 ins; 0 del; 1 mod; 8004 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/os/windows/vm/os_windows.cpp

rev 6885 : 8059803: Update use of GetVersionEx to get correct Windows version in hs_err files
Summary: Update use of GetVersionEx to get correct Windows version in hs_err files
Reviewed-by: dcubed, gtriantafill
120 lines changed: 46 ins; 24 del; 50 mod; 5717 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/os/windows/vm/os_windows.hpp

rev 6885 : 8059803: Update use of GetVersionEx to get correct Windows version in hs_err files
Summary: Update use of GetVersionEx to get correct Windows version in hs_err files
Reviewed-by: dcubed, gtriantafill
1 line changed: 0 ins; 1 del; 0 mod; 270 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/os_cpu/linux_ppc/vm/os_linux_ppc.cpp

rev 6900 : 8048169: Change 8037816 breaks HS build on PPC64 and CPP-Interpreter platforms
Summary: Fix the matching of format string parameter types to the actual argument types for the PPC64 and CPP-Interpreter files in the same way as 8037816 already did it for all the other files
Reviewed-by: stefank, coleenp, dholmes
9 lines changed: 0 ins; 0 del; 9 mod; 605 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/os_cpu/windows_x86/vm/os_windows_x86.cpp

rev 6904 : 8054008: Using -XX:-LazyBootClassLoader crashes with ACCESS_VIOLATION on Win 64bit.
Summary: Only enable the assert for current_stack_pointer after stub routines become available.
Reviewed-by: dholmes, roland, lfoltan
5 lines changed: 4 ins; 0 del; 1 mod; 640 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/tools/ProjectCreator/WinGammaPlatformVC10.java

rev 6885 : 8059803: Update use of GetVersionEx to get correct Windows version in hs_err files
Summary: Update use of GetVersionEx to get correct Windows version in hs_err files
Reviewed-by: dcubed, gtriantafill
1 line changed: 0 ins; 0 del; 1 mod; 462 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/classfile/classLoader.cpp

rev 6905 : 8064375: Change certain errors to warnings in CDS output.
Summary: Change CDS non-fatal preloading errors to warnings.
Reviewed-by: minqi, mseledtsov, coleenp
1 line changed: 0 ins; 0 del; 1 mod; 1715 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/classfile/classLoaderData.cpp

rev 6909 : 8065361: Fixup headers and definitions for INCLUDE_TRACE
Reviewed-by: sla, stefank
3 lines changed: 0 ins; 1 del; 2 mod; 984 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/classfile/classLoaderData.hpp

rev 6909 : 8065361: Fixup headers and definitions for INCLUDE_TRACE
Reviewed-by: sla, stefank
2 lines changed: 1 ins; 0 del; 1 mod; 333 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/classfile/dictionary.cpp

rev 6905 : 8064375: Change certain errors to warnings in CDS output.
Summary: Change CDS non-fatal preloading errors to warnings.
Reviewed-by: minqi, mseledtsov, coleenp
1 line changed: 0 ins; 0 del; 1 mod; 779 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/classfile/systemDictionary.cpp

rev 6909 : 8065361: Fixup headers and definitions for INCLUDE_TRACE
Reviewed-by: sla, stefank
3 lines changed: 0 ins; 1 del; 2 mod; 2850 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/gc_implementation/concurrentMarkSweep/compactibleFreeListSpace.cpp

rev 6887 : 8060467: CMS: small OldPLABSize and -XX:-ResizePLAB cause assert(ResizePLAB || n_blks == OldPLABSize) failed: Error
Reviewed-by: tschatzl, jmasa, kbarrett
1 line changed: 0 ins; 0 del; 1 mod; 3050 unchg

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

rev 6902 : 8062036: ConcurrentMarkThread::slt may be invoked before ConcurrentMarkThread::makeSurrogateLockerThread causing intermittent crashes
Summary: Suppress gc_alot during VM init, improve error for SLT uninitialized.
Reviewed-by: jmasa, brutisso, tschatzl
6 lines changed: 4 ins; 0 del; 2 mod; 309 unchg

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

rev 6912 : 8065227: Report allocation context stats at end of cleanup
Summary: Moved allocation context update from remark to the cleanup phase.
Reviewed-by: mgerdin, jmasa
2 lines changed: 1 ins; 1 del; 0 mod; 4828 unchg

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

rev 6912 : 8065227: Report allocation context stats at end of cleanup
Summary: Moved allocation context update from remark to the cleanup phase.
Reviewed-by: mgerdin, jmasa
1 line changed: 0 ins; 0 del; 1 mod; 51 unchg

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

rev 6888 : 8064556: G1: ParallelGCThreads=0 may cause assert(!MetadataOnStackMark::has_buffer_for_thread(Thread::current())) failed: Should be empty
Reviewed-by: mgerdin, tschatzl
3 lines changed: 2 ins; 0 del; 1 mod; 7035 unchg

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

rev 6911 : 8065305: Make it possible to extend the G1CollectorPolicy
Summary: Added a G1CollectorPolicyExt where it is possible to extend the class.
Reviewed-by: sjohanss, tschatzl
12 lines changed: 12 ins; 0 del; 0 mod; 2284 unchg

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

rev 6911 : 8065305: Make it possible to extend the G1CollectorPolicy
Summary: Added a G1CollectorPolicyExt where it is possible to extend the class.
Reviewed-by: sjohanss, tschatzl
12 lines changed: 1 ins; 8 del; 3 mod; 943 unchg

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

rev 6908 : 8058209: Race in G1 card scanning could allow scanning of memory covered by PLABs
Summary: Read _top before _gc_time_stamp in saved_mark_word() with LoadLoad order to ensure we get a consistent view
Reviewed-by: brutisso, dcubed, dholmes, stefank
6 lines changed: 3 ins; 0 del; 3 mod; 1079 unchg

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

rev 6902 : 8062036: ConcurrentMarkThread::slt may be invoked before ConcurrentMarkThread::makeSurrogateLockerThread causing intermittent crashes
Summary: Suppress gc_alot during VM init, improve error for SLT uninitialized.
Reviewed-by: jmasa, brutisso, tschatzl
6 lines changed: 4 ins; 0 del; 2 mod; 260 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/gc_implementation/shared/concurrentGCThread.cpp

rev 6902 : 8062036: ConcurrentMarkThread::slt may be invoked before ConcurrentMarkThread::makeSurrogateLockerThread causing intermittent crashes
Summary: Suppress gc_alot during VM init, improve error for SLT uninitialized.
Reviewed-by: jmasa, brutisso, tschatzl
7 lines changed: 7 ins; 0 del; 0 mod; 201 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/gc_implementation/shared/concurrentGCThread.hpp

rev 6902 : 8062036: ConcurrentMarkThread::slt may be invoked before ConcurrentMarkThread::makeSurrogateLockerThread causing intermittent crashes
Summary: Suppress gc_alot during VM init, improve error for SLT uninitialized.
Reviewed-by: jmasa, brutisso, tschatzl
3 lines changed: 3 ins; 0 del; 0 mod; 106 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/gc_implementation/shared/objectCountEventSender.cpp

rev 6909 : 8065361: Fixup headers and definitions for INCLUDE_TRACE
Reviewed-by: sla, stefank
2 lines changed: 1 ins; 1 del; 0 mod; 58 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/interpreter/bytecodeInterpreter.cpp

rev 6900 : 8048169: Change 8037816 breaks HS build on PPC64 and CPP-Interpreter platforms
Summary: Fix the matching of format string parameter types to the actual argument types for the PPC64 and CPP-Interpreter files in the same way as 8037816 already did it for all the other files
Reviewed-by: stefank, coleenp, dholmes
8 lines changed: 0 ins; 0 del; 8 mod; 3451 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/interpreter/bytecodeInterpreterProfiling.hpp

rev 6900 : 8048169: Change 8037816 breaks HS build on PPC64 and CPP-Interpreter platforms
Summary: Fix the matching of format string parameter types to the actual argument types for the PPC64 and CPP-Interpreter files in the same way as 8037816 already did it for all the other files
Reviewed-by: stefank, coleenp, dholmes
6 lines changed: 0 ins; 0 del; 6 mod; 302 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/interpreter/bytecodes.hpp

rev 6907 : 8056071: compiler/whitebox/IsMethodCompilableTest.java fails with 'method() is not compilable after 3 iterations'
Summary: Always use MDO if valid and always compile trivial methods with C1 if available.
Reviewed-by: kvn, iveresov
2 lines changed: 2 ins; 0 del; 0 mod; 445 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/memory/metaspaceShared.cpp

rev 6905 : 8064375: Change certain errors to warnings in CDS output.
Summary: Change CDS non-fatal preloading errors to warnings.
Reviewed-by: minqi, mseledtsov, coleenp
1 line changed: 0 ins; 0 del; 1 mod; 1146 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/memory/universe.cpp

rev 6911 : 8065305: Make it possible to extend the G1CollectorPolicy
Summary: Added a G1CollectorPolicyExt where it is possible to extend the class.
Reviewed-by: sjohanss, tschatzl
2 lines changed: 0 ins; 0 del; 2 mod; 1527 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/oops/method.cpp

rev 6907 : 8056071: compiler/whitebox/IsMethodCompilableTest.java fails with 'method() is not compilable after 3 iterations'
Summary: Always use MDO if valid and always compile trivial methods with C1 if available.
Reviewed-by: kvn, iveresov
9 lines changed: 9 ins; 0 del; 0 mod; 2035 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/oops/method.hpp

rev 6907 : 8056071: compiler/whitebox/IsMethodCompilableTest.java fails with 'method() is not compilable after 3 iterations'
Summary: Always use MDO if valid and always compile trivial methods with C1 if available.
Reviewed-by: kvn, iveresov
3 lines changed: 3 ins; 0 del; 0 mod; 1073 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/oops/methodData.cpp

rev 6907 : 8056071: compiler/whitebox/IsMethodCompilableTest.java fails with 'method() is not compilable after 3 iterations'
Summary: Always use MDO if valid and always compile trivial methods with C1 if available.
Reviewed-by: kvn, iveresov
1 line changed: 0 ins; 0 del; 1 mod; 1645 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/oops/methodData.hpp

rev 6907 : 8056071: compiler/whitebox/IsMethodCompilableTest.java fails with 'method() is not compilable after 3 iterations'
Summary: Always use MDO if valid and always compile trivial methods with C1 if available.
Reviewed-by: kvn, iveresov
4 lines changed: 1 ins; 0 del; 3 mod; 2478 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/opto/doCall.cpp

rev 6883 : 8057622: java/util/stream/test/org/openjdk/tests/java/util/stream/InfiniteStreamWithLimitOpTest: SEGV inside compiled code (sparc)
Summary: In Parse::array_store_check(), add control edge FROM IfTrue branch of runtime type check of the destination array TO loading _element_klass from destination array.
Reviewed-by: kvn, roland, anoll
Contributed-by: Zoltan Majo <zoltan.majo@oracle.com>
2 lines changed: 0 ins; 0 del; 2 mod; 1069 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/opto/graphKit.cpp

rev 6883 : 8057622: java/util/stream/test/org/openjdk/tests/java/util/stream/InfiniteStreamWithLimitOpTest: SEGV inside compiled code (sparc)
Summary: In Parse::array_store_check(), add control edge FROM IfTrue branch of runtime type check of the destination array TO loading _element_klass from destination array.
Reviewed-by: kvn, roland, anoll
Contributed-by: Zoltan Majo <zoltan.majo@oracle.com>
2 lines changed: 0 ins; 0 del; 2 mod; 4137 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/opto/library_call.cpp

rev 6883 : 8057622: java/util/stream/test/org/openjdk/tests/java/util/stream/InfiniteStreamWithLimitOpTest: SEGV inside compiled code (sparc)
Summary: In Parse::array_store_check(), add control edge FROM IfTrue branch of runtime type check of the destination array TO loading _element_klass from destination array.
Reviewed-by: kvn, roland, anoll
Contributed-by: Zoltan Majo <zoltan.majo@oracle.com>
4 lines changed: 0 ins; 0 del; 4 mod; 6542 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/opto/macro.cpp

rev 6883 : 8057622: java/util/stream/test/org/openjdk/tests/java/util/stream/InfiniteStreamWithLimitOpTest: SEGV inside compiled code (sparc)
Summary: In Parse::array_store_check(), add control edge FROM IfTrue branch of runtime type check of the destination array TO loading _element_klass from destination array.
Reviewed-by: kvn, roland, anoll
Contributed-by: Zoltan Majo <zoltan.majo@oracle.com>
1 line changed: 0 ins; 0 del; 1 mod; 2575 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/opto/memnode.cpp

rev 6883 : 8057622: java/util/stream/test/org/openjdk/tests/java/util/stream/InfiniteStreamWithLimitOpTest: SEGV inside compiled code (sparc)
Summary: In Parse::array_store_check(), add control edge FROM IfTrue branch of runtime type check of the destination array TO loading _element_klass from destination array.
Reviewed-by: kvn, roland, anoll
Contributed-by: Zoltan Majo <zoltan.majo@oracle.com>
14 lines changed: 11 ins; 1 del; 2 mod; 4573 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/opto/memnode.hpp

rev 6883 : 8057622: java/util/stream/test/org/openjdk/tests/java/util/stream/InfiniteStreamWithLimitOpTest: SEGV inside compiled code (sparc)
Summary: In Parse::array_store_check(), add control edge FROM IfTrue branch of runtime type check of the destination array TO loading _element_klass from destination array.
Reviewed-by: kvn, roland, anoll
Contributed-by: Zoltan Majo <zoltan.majo@oracle.com>
11 lines changed: 8 ins; 0 del; 3 mod; 1441 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/opto/parse1.cpp

rev 6883 : 8057622: java/util/stream/test/org/openjdk/tests/java/util/stream/InfiniteStreamWithLimitOpTest: SEGV inside compiled code (sparc)
Summary: In Parse::array_store_check(), add control edge FROM IfTrue branch of runtime type check of the destination array TO loading _element_klass from destination array.
Reviewed-by: kvn, roland, anoll
Contributed-by: Zoltan Majo <zoltan.majo@oracle.com>
1 line changed: 0 ins; 0 del; 1 mod; 2265 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/opto/parseHelper.cpp

rev 6883 : 8057622: java/util/stream/test/org/openjdk/tests/java/util/stream/InfiniteStreamWithLimitOpTest: SEGV inside compiled code (sparc)
Summary: In Parse::array_store_check(), add control edge FROM IfTrue branch of runtime type check of the destination array TO loading _element_klass from destination array.
Reviewed-by: kvn, roland, anoll
Contributed-by: Zoltan Majo <zoltan.majo@oracle.com>
38 lines changed: 28 ins; 3 del; 7 mod; 587 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/prims/whitebox.cpp

rev 6884 : 8061969: [TESTBUG] MallocSiteHashOverflow.java should be enabled for 32-bit platforms
Reviewed-by: ctornqvi, coleenp
rev 6886 : 8058251: assert(_count > 0) failed: Negative counter when running runtime/NMT/MallocTrackingVerify.java
Summary: Fixed an issue when overflowing the MallocSite hash table bucket
Reviewed-by: coleenp, gtriantafill
19 lines changed: 7 ins; 10 del; 2 mod; 1050 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/runtime/advancedThresholdPolicy.cpp

rev 6907 : 8056071: compiler/whitebox/IsMethodCompilableTest.java fails with 'method() is not compilable after 3 iterations'
Summary: Always use MDO if valid and always compile trivial methods with C1 if available.
Reviewed-by: kvn, iveresov
2 lines changed: 0 ins; 0 del; 2 mod; 529 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/runtime/arguments.cpp

rev 6910 : 8064667: Add -XX:+CheckEndorsedAndExtDirs flag to JDK 8
Reviewed-by: coleenp, ccheung
rev 6911 : 8065305: Make it possible to extend the G1CollectorPolicy
Summary: Added a G1CollectorPolicyExt where it is possible to extend the class.
Reviewed-by: sjohanss, tschatzl
164 lines changed: 161 ins; 0 del; 3 mod; 4144 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/runtime/arguments.hpp

rev 6911 : 8065305: Make it possible to extend the G1CollectorPolicy
Summary: Added a G1CollectorPolicyExt where it is possible to extend the class.
Reviewed-by: sjohanss, tschatzl
2 lines changed: 1 ins; 1 del; 0 mod; 621 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/runtime/arguments_ext.hpp

rev 6911 : 8065305: Make it possible to extend the G1CollectorPolicy
Summary: Added a G1CollectorPolicyExt where it is possible to extend the class.
Reviewed-by: sjohanss, tschatzl
8 lines changed: 0 ins; 5 del; 3 mod; 48 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/runtime/globals.hpp

rev 6910 : 8064667: Add -XX:+CheckEndorsedAndExtDirs flag to JDK 8
Reviewed-by: coleenp, ccheung
3 lines changed: 3 ins; 0 del; 0 mod; 4004 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/runtime/interfaceSupport.cpp

rev 6902 : 8062036: ConcurrentMarkThread::slt may be invoked before ConcurrentMarkThread::makeSurrogateLockerThread causing intermittent crashes
Summary: Suppress gc_alot during VM init, improve error for SLT uninitialized.
Reviewed-by: jmasa, brutisso, tschatzl
1 line changed: 0 ins; 0 del; 1 mod; 287 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/runtime/os.cpp

rev 6886 : 8058251: assert(_count > 0) failed: Negative counter when running runtime/NMT/MallocTrackingVerify.java
Summary: Fixed an issue when overflowing the MallocSite hash table bucket
Reviewed-by: coleenp, gtriantafill
17 lines changed: 0 ins; 17 del; 0 mod; 1571 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/runtime/simpleThresholdPolicy.inline.hpp

rev 6907 : 8056071: compiler/whitebox/IsMethodCompilableTest.java fails with 'method() is not compilable after 3 iterations'
Summary: Always use MDO if valid and always compile trivial methods with C1 if available.
Reviewed-by: kvn, iveresov
11 lines changed: 4 ins; 0 del; 7 mod; 61 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/services/mallocTracker.cpp

rev 6886 : 8058251: assert(_count > 0) failed: Negative counter when running runtime/NMT/MallocTrackingVerify.java
Summary: Fixed an issue when overflowing the MallocSite hash table bucket
Reviewed-by: coleenp, gtriantafill
26 lines changed: 2 ins; 20 del; 4 mod; 160 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/services/mallocTracker.hpp

rev 6886 : 8058251: assert(_count > 0) failed: Negative counter when running runtime/NMT/MallocTrackingVerify.java
Summary: Fixed an issue when overflowing the MallocSite hash table bucket
Reviewed-by: coleenp, gtriantafill
rev 6898 : 8062870: src/share/vm/services/mallocTracker.hpp:64 assert(_count > 0) failed: Negative counter
Summary: Signed bitfield size y can only have (1 << y)-1 values.
Reviewed-by: shade, dholmes, jrose, ctornqvi, gtriantafill
55 lines changed: 3 ins; 35 del; 17 mod; 353 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/services/mallocTracker.inline.hpp

rev 6886 : 8058251: assert(_count > 0) failed: Negative counter when running runtime/NMT/MallocTrackingVerify.java
Summary: Fixed an issue when overflowing the MallocSite hash table bucket
Reviewed-by: coleenp, gtriantafill
7 lines changed: 0 ins; 7 del; 0 mod; 36 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/trace/noTraceBackend.hpp

rev 6909 : 8065361: Fixup headers and definitions for INCLUDE_TRACE
Reviewed-by: sla, stefank
1 line changed: 0 ins; 0 del; 1 mod; 43 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/trace/traceBackend.hpp

rev 6909 : 8065361: Fixup headers and definitions for INCLUDE_TRACE
Reviewed-by: sla, stefank
7 lines changed: 0 ins; 4 del; 3 mod; 59 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/trace/traceEvent.hpp

rev 6909 : 8065361: Fixup headers and definitions for INCLUDE_TRACE
Reviewed-by: sla, stefank
4 lines changed: 0 ins; 2 del; 2 mod; 155 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/trace/traceEventClasses.xsl

rev 6909 : 8065361: Fixup headers and definitions for INCLUDE_TRACE
Reviewed-by: sla, stefank
7 lines changed: 0 ins; 4 del; 3 mod; 250 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/trace/traceEventIds.xsl

rev 6909 : 8065361: Fixup headers and definitions for INCLUDE_TRACE
Reviewed-by: sla, stefank
6 lines changed: 0 ins; 2 del; 4 mod; 68 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/trace/traceMacros.hpp

rev 6909 : 8065361: Fixup headers and definitions for INCLUDE_TRACE
Reviewed-by: sla, stefank
3 lines changed: 0 ins; 0 del; 3 mod; 41 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/trace/traceStream.hpp

rev 6909 : 8065361: Fixup headers and definitions for INCLUDE_TRACE
Reviewed-by: sla, stefank
4 lines changed: 0 ins; 2 del; 2 mod; 117 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/trace/traceTypes.xsl

rev 6909 : 8065361: Fixup headers and definitions for INCLUDE_TRACE
Reviewed-by: sla, stefank
5 lines changed: 0 ins; 1 del; 4 mod; 69 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/src/share/vm/trace/tracing.hpp

rev 6909 : 8065361: Fixup headers and definitions for INCLUDE_TRACE
Reviewed-by: sla, stefank
1 line changed: 0 ins; 0 del; 1 mod; 30 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/test/TEST.ROOT

rev 6884 : 8061969: [TESTBUG] MallocSiteHashOverflow.java should be enabled for 32-bit platforms
Reviewed-by: ctornqvi, coleenp
1 line changed: 1 ins; 0 del; 0 mod; 32 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/test/TEST.groups

rev 6886 : 8058251: assert(_count > 0) failed: Negative counter when running runtime/NMT/MallocTrackingVerify.java
Summary: Fixed an issue when overflowing the MallocSite hash table bucket
Reviewed-by: coleenp, gtriantafill
1 line changed: 0 ins; 1 del; 0 mod; 257 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/test/compiler/whitebox/IsMethodCompilableTest.java

rev 6907 : 8056071: compiler/whitebox/IsMethodCompilableTest.java fails with 'method() is not compilable after 3 iterations'
Summary: Always use MDO if valid and always compile trivial methods with C1 if available.
Reviewed-by: kvn, iveresov
1 line changed: 0 ins; 0 del; 1 mod; 137 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/test/runtime/NMT/MallocSiteHashOverflow.java

rev 6884 : 8061969: [TESTBUG] MallocSiteHashOverflow.java should be enabled for 32-bit platforms
Reviewed-by: ctornqvi, coleenp
rev 6898 : 8062870: src/share/vm/services/mallocTracker.hpp:64 assert(_count > 0) failed: Negative counter
Summary: Signed bitfield size y can only have (1 << y)-1 values.
Reviewed-by: shade, dholmes, jrose, ctornqvi, gtriantafill
34 lines changed: 22 ins; 8 del; 4 mod; 52 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New ----- Raw hotspot/test/testlibrary/whitebox/sun/hotspot/WhiteBox.java

rev 6884 : 8061969: [TESTBUG] MallocSiteHashOverflow.java should be enabled for 32-bit platforms
Reviewed-by: ctornqvi, coleenp
rev 6891 : 8062247: [TESTBUG] Allow WhiteBox test to access JVM offsets
Reviewed-by: coleenp, iklam, mseledtsov
Contributed-by: yumin.qi@oracle.com
11 lines changed: 10 ins; 1 del; 0 mod; 210 unchg

------ ------ ------ ------ ------ --- New ----- Raw hotspot/src/share/vm/gc_implementation/g1/g1CollectorPolicy_ext.hpp

rev 6911 : 8065305: Make it possible to extend the G1CollectorPolicy
Summary: Added a G1CollectorPolicyExt where it is possible to extend the class.
Reviewed-by: sjohanss, tschatzl
32 lines changed: 32 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New ----- Raw hotspot/test/gc/concurrentMarkSweep/DisableResizePLAB.java

rev 6887 : 8060467: CMS: small OldPLABSize and -XX:-ResizePLAB cause assert(ResizePLAB || n_blks == OldPLABSize) failed: Error
Reviewed-by: tschatzl, jmasa, kbarrett
44 lines changed: 44 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ --- New ----- Raw hotspot/test/runtime/CheckEndorsedAndExtDirs/EndorsedExtDirs.java

rev 6910 : 8064667: Add -XX:+CheckEndorsedAndExtDirs flag to JDK 8
Reviewed-by: coleenp, ccheung
60 lines changed: 60 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ------ Old --- ----- --- hotspot/make/solaris/makefiles/add_gnu_debuglink.make

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

------ ------ ------ ------ ------ Old --- ----- --- hotspot/make/solaris/makefiles/fix_empty_sec_hdr_flags.make

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

------ ------ ------ ------ ------ Old --- ----- --- hotspot/src/os/solaris/add_gnu_debuglink/add_gnu_debuglink.c

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

------ ------ ------ ------ ------ Old --- ----- --- hotspot/src/os/solaris/fix_empty_sec_hdr_flags/fix_empty_sec_hdr_flags.c

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

------ ------ ------ ------ ------ Old --- ----- --- hotspot/test/runtime/NMT/UnsafeMallocLimit.java

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

------ ------ ------ ------ ------ Old --- ----- --- hotspot/test/runtime/NMT/UnsafeMallocLimit2.java

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

This code review page was prepared using /export/pub/bin/webrev (vers 25.6-hg+openjdk.java.net).