hg: jdk7/jdk7/hotspot: 35 new changesets
erik.trimble at oracle.com
erik.trimble at oracle.com
Tue Aug 31 17:18:13 PDT 2010
Changeset: f6f3eef8a521
Author: kevinw
Date: 2010-07-30 22:43 +0100
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/f6f3eef8a521
6581734: CMS Old Gen's collection usage is zero after GC which is incorrect
Summary: Management code enabled for use by a concurrent collector.
Reviewed-by: mchung, ysr
! src/share/vm/gc_implementation/concurrentMarkSweep/concurrentMarkSweepGeneration.cpp
! src/share/vm/gc_implementation/concurrentMarkSweep/concurrentMarkSweepGeneration.hpp
! src/share/vm/gc_implementation/includeDB_gc_concurrentMarkSweep
! src/share/vm/services/management.cpp
! src/share/vm/services/memoryManager.cpp
! src/share/vm/services/memoryManager.hpp
! src/share/vm/services/memoryService.cpp
! src/share/vm/services/memoryService.hpp
+ test/gc/6581734/Test6581734.java
Changeset: 63f4675ac87d
Author: kevinw
Date: 2010-07-31 15:10 +0100
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/63f4675ac87d
Merge
- src/os/linux/vm/vtune_linux.cpp
- src/os/solaris/vm/vtune_solaris.cpp
- src/os/windows/vm/vtune_windows.cpp
! src/share/vm/gc_implementation/concurrentMarkSweep/concurrentMarkSweepGeneration.cpp
- src/share/vm/runtime/vtune.hpp
Changeset: 2d160770d2e5
Author: johnc
Date: 2010-08-02 12:51 -0700
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/2d160770d2e5
6814437: G1: remove the _new_refs array
Summary: The per-worker _new_refs array is used to hold references that point into the collection set. It is populated during RSet updating and subsequently processed. In the event of an evacuation failure it processed again to recreate the RSets of regions in the collection set. Remove the per-worker _new_refs array by processing the references directly. Use a DirtyCardQueue to hold the cards containing the references so that the RSets of regions in the collection set can be recreated when handling an evacuation failure.
Reviewed-by: iveresov, jmasa, tonyp
! src/share/vm/gc_implementation/g1/concurrentG1Refine.cpp
! src/share/vm/gc_implementation/g1/concurrentG1Refine.hpp
! src/share/vm/gc_implementation/g1/dirtyCardQueue.cpp
! src/share/vm/gc_implementation/g1/dirtyCardQueue.hpp
! src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp
! src/share/vm/gc_implementation/g1/g1CollectedHeap.hpp
! src/share/vm/gc_implementation/g1/g1CollectorPolicy.cpp
! src/share/vm/gc_implementation/g1/g1CollectorPolicy.hpp
! src/share/vm/gc_implementation/g1/g1OopClosures.inline.hpp
! src/share/vm/gc_implementation/g1/g1RemSet.cpp
! src/share/vm/gc_implementation/g1/g1RemSet.hpp
! src/share/vm/gc_implementation/g1/g1RemSet.inline.hpp
! src/share/vm/gc_implementation/g1/heapRegion.cpp
! src/share/vm/gc_implementation/includeDB_gc_g1
Changeset: 9d7a8ab3736b
Author: tonyp
Date: 2010-07-22 10:27 -0400
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/9d7a8ab3736b
6962589: remove breadth first scanning code from parallel gc
Summary: Remove the breadth-first copying order from ParallelScavenge and use depth-first by default.
Reviewed-by: jcoomes, ysr, johnc
! src/share/vm/gc_implementation/includeDB_gc_parallelScavenge
! src/share/vm/gc_implementation/parallelScavenge/cardTableExtension.cpp
- src/share/vm/gc_implementation/parallelScavenge/prefetchQueue.hpp
! src/share/vm/gc_implementation/parallelScavenge/psPromotionManager.cpp
! src/share/vm/gc_implementation/parallelScavenge/psPromotionManager.hpp
! src/share/vm/gc_implementation/parallelScavenge/psPromotionManager.inline.hpp
! src/share/vm/gc_implementation/parallelScavenge/psScavenge.cpp
! src/share/vm/gc_implementation/parallelScavenge/psScavenge.inline.hpp
! src/share/vm/gc_implementation/parallelScavenge/psTasks.cpp
! src/share/vm/oops/arrayKlassKlass.cpp
! src/share/vm/oops/compiledICHolderKlass.cpp
! src/share/vm/oops/constMethodKlass.cpp
! src/share/vm/oops/constantPoolKlass.cpp
! src/share/vm/oops/cpCacheKlass.cpp
! src/share/vm/oops/instanceKlass.cpp
! src/share/vm/oops/instanceKlass.hpp
! src/share/vm/oops/instanceKlassKlass.cpp
! src/share/vm/oops/instanceRefKlass.cpp
! src/share/vm/oops/klassKlass.cpp
! src/share/vm/oops/klassPS.hpp
! src/share/vm/oops/methodDataKlass.cpp
! src/share/vm/oops/methodKlass.cpp
! src/share/vm/oops/objArrayKlass.cpp
! src/share/vm/oops/objArrayKlassKlass.cpp
! src/share/vm/oops/oop.hpp
! src/share/vm/oops/oop.psgc.inline.hpp
! src/share/vm/oops/symbolKlass.cpp
! src/share/vm/oops/typeArrayKlass.cpp
! src/share/vm/runtime/arguments.cpp
! src/share/vm/runtime/globals.hpp
Changeset: 0ce1569c90e5
Author: tonyp
Date: 2010-08-04 13:03 -0400
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/0ce1569c90e5
6963209: G1: remove the concept of abandoned pauses
Summary: As part of 6944166 we disabled the concept of abandoned pauses (i.e., if the collection set is empty, we would still try to do a pause even if it is to update the RSets and scan the roots). This changeset removes the code and structures associated with abandoned pauses.
Reviewed-by: iveresov, johnc
! src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp
! src/share/vm/gc_implementation/g1/g1CollectorPolicy.cpp
! src/share/vm/gc_implementation/g1/g1CollectorPolicy.hpp
Changeset: a03ae377b2e8
Author: johnc
Date: 2010-08-06 10:17 -0700
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/a03ae377b2e8
6930581: G1: assert(ParallelGCThreads > 1 || n_yielded() == _hrrs->occupied(),"Should have yielded all the ..
Summary: During RSet updating, when ParallelGCThreads is zero, references that point into the collection set are added directly the referenced region's RSet. This can cause the sparse table in the RSet to expand. RSet scanning and the "occupied" routine will then operate on different instances of the sparse table causing the assert to trip. This may also cause some cards added post expansion to be missed during RSet scanning. When ParallelGCThreads is non-zero such references are recorded on the "references to be scanned" queue and the card containing the reference is recorded in a dirty card queue for use in the event of an evacuation failure. Employ the parallel code in the serial case to avoid expanding the RSets of regions in the collection set.
Reviewed-by: iveresov, ysr, tonyp
! src/share/vm/gc_implementation/g1/g1RemSet.cpp
! src/share/vm/gc_implementation/g1/g1RemSet.hpp
! src/share/vm/gc_implementation/g1/g1RemSet.inline.hpp
! src/share/vm/gc_implementation/g1/sparsePRT.cpp
Changeset: 5f429ee79634
Author: jcoomes
Date: 2010-08-09 05:41 -0700
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/5f429ee79634
6966222: G1: simplify TaskQueue overflow handling
Reviewed-by: tonyp, ysr
! src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp
! src/share/vm/gc_implementation/g1/g1CollectedHeap.hpp
! src/share/vm/gc_implementation/g1/g1CollectedHeap.inline.hpp
! src/share/vm/utilities/taskqueue.cpp
! src/share/vm/utilities/taskqueue.hpp
Changeset: 94251661de76
Author: jcoomes
Date: 2010-08-09 18:03 -0700
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/94251661de76
6970376: ParNew: shared TaskQueue statistics
Reviewed-by: ysr
! src/share/vm/gc_implementation/parNew/parNewGeneration.cpp
! src/share/vm/gc_implementation/parNew/parNewGeneration.hpp
Changeset: a6bff45449bc
Author: ysr
Date: 2010-08-10 14:53 -0700
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/a6bff45449bc
6973570: OrderAccess::storestore() scales poorly on multi-socket x64 and sparc: cache-line ping-ponging
Summary: volatile store to static variable removed in favour of a volatile store to stack to avoid excessive cache coherency traffic; verified that the volatile store is not elided by any of our current compilers.
Reviewed-by: dholmes, dice, jcoomes, kvn
! src/os_cpu/linux_sparc/vm/orderAccess_linux_sparc.inline.hpp
! src/os_cpu/linux_x86/vm/orderAccess_linux_x86.inline.hpp
! src/os_cpu/solaris_sparc/vm/orderAccess_solaris_sparc.inline.hpp
! src/os_cpu/solaris_x86/vm/orderAccess_solaris_x86.inline.hpp
! src/os_cpu/windows_x86/vm/orderAccess_windows_x86.inline.hpp
! src/share/vm/runtime/orderAccess.cpp
! src/share/vm/runtime/orderAccess.hpp
Changeset: 2d6b74c9a797
Author: jcoomes
Date: 2010-08-11 13:12 -0700
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/2d6b74c9a797
6976378: ParNew: stats are printed unconditionally in debug builds
Reviewed-by: tonyp
! src/share/vm/gc_implementation/parNew/parNewGeneration.cpp
Changeset: 7fcd5f39bd7a
Author: johnc
Date: 2010-08-14 00:47 -0700
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/7fcd5f39bd7a
Merge
- src/share/vm/gc_implementation/parallelScavenge/prefetchQueue.hpp
! src/share/vm/oops/arrayKlassKlass.cpp
! src/share/vm/oops/compiledICHolderKlass.cpp
! src/share/vm/oops/constMethodKlass.cpp
! src/share/vm/oops/constantPoolKlass.cpp
! src/share/vm/oops/cpCacheKlass.cpp
! src/share/vm/oops/klassKlass.cpp
! src/share/vm/runtime/arguments.cpp
! src/share/vm/runtime/globals.hpp
Changeset: f121b2772674
Author: trims
Date: 2010-08-18 16:11 -0700
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/f121b2772674
Merge
- src/share/vm/gc_implementation/parallelScavenge/prefetchQueue.hpp
Changeset: 495caa35b1b5
Author: asaha
Date: 2010-08-17 22:52 -0700
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/495caa35b1b5
6977952: Test: Sync missing tests from hs16.3 to hs17.x
Reviewed-by: wrockett
+ test/compiler/6894807/IsInstanceTest.java
+ test/compiler/6894807/Test6894807.sh
+ test/runtime/6626217/IFace.java
+ test/runtime/6626217/Loader2.java
+ test/runtime/6626217/Test6626217.sh
+ test/runtime/6626217/You_Have_Been_P0wned.java
+ test/runtime/6626217/bug_21227.java
+ test/runtime/6626217/from_loader2.java
+ test/runtime/6626217/many_loader1.java.foo
+ test/runtime/6626217/many_loader2.java.foo
Changeset: be3f9c242c9d
Author: ysr
Date: 2010-08-16 15:58 -0700
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/be3f9c242c9d
6948538: CMS: BOT walkers can fall into object allocation and initialization cracks
Summary: GC workers now recognize an intermediate transient state of blocks which are allocated but have not yet completed initialization. blk_start() calls do not attempt to determine the size of a block in the transient state, rather waiting for the block to become initialized so that it is safe to query its size. Audited and ensured the order of initialization of object fields (klass, free bit and size) to respect block state transition protocol. Also included some new assertion checking code enabled in debug mode.
Reviewed-by: chrisphi, johnc, poonam
! src/share/vm/gc_implementation/concurrentMarkSweep/compactibleFreeListSpace.cpp
! src/share/vm/gc_implementation/concurrentMarkSweep/compactibleFreeListSpace.hpp
! src/share/vm/gc_implementation/concurrentMarkSweep/concurrentMarkSweepGeneration.cpp
! src/share/vm/gc_implementation/concurrentMarkSweep/concurrentMarkSweepGeneration.hpp
! src/share/vm/gc_implementation/concurrentMarkSweep/freeChunk.hpp
! src/share/vm/gc_implementation/concurrentMarkSweep/promotionInfo.cpp
! src/share/vm/gc_implementation/includeDB_gc_concurrentMarkSweep
! src/share/vm/includeDB_core
! src/share/vm/memory/blockOffsetTable.cpp
! src/share/vm/memory/blockOffsetTable.hpp
! src/share/vm/memory/blockOffsetTable.inline.hpp
! src/share/vm/runtime/globals.hpp
Changeset: 688c3755d7af
Author: tonyp
Date: 2010-08-17 14:40 -0400
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/688c3755d7af
6959014: G1: assert(minimum_desired_capacity <= maximum_desired_capacity) failed: sanity check
Summary: There are a few issues in the code that calculates whether to resize the heap and by how much: a) some calculations can overflow 32-bit size_t's, b) min_desired_capacity is not bounded by the max heap size, and c) the assrt that fires is in the wrong place. The fix also includes some tidying up of the related verbose code.
Reviewed-by: ysr, jmasa
! src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp
Changeset: bb847e31b836
Author: tonyp
Date: 2010-08-17 14:40 -0400
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/bb847e31b836
6974928: G1: sometimes humongous objects are allocated in young regions
Summary: as the title says, sometimes we are allocating humongous objects in young regions and we shouldn't.
Reviewed-by: ysr, johnc
! src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp
! src/share/vm/gc_implementation/g1/g1CollectedHeap.hpp
! src/share/vm/gc_implementation/g1/g1CollectedHeap.inline.hpp
! src/share/vm/gc_implementation/g1/heapRegion.cpp
Changeset: b63010841f78
Author: tonyp
Date: 2010-08-17 14:40 -0400
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/b63010841f78
6975964: G1: print out a more descriptive message for evacuation failure when +PrintGCDetails is set
Summary: we're renaming "evacuation failure" to "to-space overflow". I'm also piggy-backing a small additional change which removes the "Mark closure took..." output.
Reviewed-by: ysr, johnc
! src/share/vm/gc_implementation/g1/concurrentMark.cpp
! src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp
Changeset: 5ed703250bff
Author: ysr
Date: 2010-08-18 11:39 -0700
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/5ed703250bff
6977970: CMS: concurrentMarkSweepGeneration.cpp:7947 assert(addr <= _limit) failed: sweep invariant
Summary: Allow for the possibility (when the heap is expanding) that the sweep might skip over and past, rather than necessarily step on, the sweep limit determined at the beginning of a concurrent marking cycle.
Reviewed-by: jmasa, tonyp
! src/share/vm/gc_implementation/concurrentMarkSweep/concurrentMarkSweepGeneration.cpp
Changeset: 413ad0331a0c
Author: johnc
Date: 2010-08-18 10:59 -0700
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/413ad0331a0c
6977924: Changes for 6975078 produce build error with certain gcc versions
Summary: The changes introduced for 6975078 assign badHeapOopVal to the _allocation field in the ResourceObj class. In 32 bit linux builds with certain versions of gcc this assignment will be flagged as an error while compiling allocation.cpp. In 32 bit builds the constant value badHeapOopVal (which is cast to an intptr_t) is negative. The _allocation field is typed as an unsigned intptr_t and gcc catches this as an error.
Reviewed-by: jcoomes, ysr, phh
! src/share/vm/memory/allocation.cpp
Changeset: effb55808a18
Author: johnc
Date: 2010-08-18 17:44 -0400
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/effb55808a18
Merge
Changeset: 1b0104ab1e5e
Author: tonyp
Date: 2010-08-19 14:08 -0400
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/1b0104ab1e5e
Merge
Changeset: 0e509ddd9962
Author: trims
Date: 2010-08-20 03:47 -0700
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/0e509ddd9962
6978726: Bump the HS19 build number to 07
Summary: Update the HS19 build number to 07
Reviewed-by: jcoomes
! make/hotspot_version
Changeset: 09cdb1e1c77b
Author: trims
Date: 2010-08-20 04:08 -0700
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/09cdb1e1c77b
Merge
- src/share/vm/gc_implementation/parallelScavenge/prefetchQueue.hpp
Changeset: 71faaa8e3ccc
Author: never
Date: 2010-08-12 16:38 -0700
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/71faaa8e3ccc
6974176: ShouldNotReachHere, instanceKlass.cpp:1426
Reviewed-by: kvn, twisti
! src/share/vm/code/nmethod.cpp
! src/share/vm/code/nmethod.hpp
Changeset: da877bdc9000
Author: never
Date: 2010-08-12 23:34 -0700
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/da877bdc9000
6975006: assert(check.is_deoptimized_frame()) failed: missed deopt
Reviewed-by: kvn, twisti
! src/share/vm/runtime/frame.cpp
! src/share/vm/runtime/frame.hpp
! src/share/vm/runtime/safepoint.cpp
! src/share/vm/runtime/safepoint.hpp
! src/share/vm/runtime/thread.cpp
Changeset: a62d332029cf
Author: never
Date: 2010-08-13 15:14 -0700
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/a62d332029cf
6976372: # assert(_owner == Thread::current()) failed: invariant
Reviewed-by: kvn, twisti
! src/share/vm/compiler/compileBroker.cpp
! src/share/vm/runtime/sharedRuntime.cpp
Changeset: 13b87063b4d8
Author: twisti
Date: 2010-08-18 01:22 -0700
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/13b87063b4d8
6977640: Zero and Shark fixes
Summary: A number of fixes for Zero and Shark.
Reviewed-by: twisti
Contributed-by: Gary Benson <gbenson at redhat.com>
! src/cpu/zero/vm/bytecodeInterpreter_zero.inline.hpp
! src/cpu/zero/vm/javaFrameAnchor_zero.hpp
! src/os_cpu/linux_zero/vm/os_linux_zero.cpp
! src/os_cpu/linux_zero/vm/thread_linux_zero.cpp
! src/share/vm/interpreter/bytecodeInterpreter.cpp
Changeset: f55c4f82ab9d
Author: never
Date: 2010-08-19 14:51 -0700
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/f55c4f82ab9d
6978249: spill between cpu and fpu registers when those moves are fast
Reviewed-by: kvn
! src/cpu/sparc/vm/vm_version_sparc.cpp
! src/cpu/x86/vm/vm_version_x86.cpp
! src/cpu/x86/vm/x86_32.ad
! src/cpu/x86/vm/x86_64.ad
! src/share/vm/opto/c2_globals.hpp
! src/share/vm/opto/coalesce.cpp
! src/share/vm/opto/matcher.cpp
! src/share/vm/opto/reg_split.cpp
! src/share/vm/runtime/arguments.cpp
! src/share/vm/runtime/init.cpp
Changeset: ee5cc9e78493
Author: never
Date: 2010-08-20 09:55 -0700
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/ee5cc9e78493
Merge
! src/share/vm/runtime/arguments.cpp
! src/share/vm/runtime/thread.cpp
Changeset: 52f2bc645da5
Author: ysr
Date: 2010-08-19 12:02 -0700
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/52f2bc645da5
6978533: CMS: Elide BOT update asserts until 6977974 is fixed correctly
Reviewed-by: jcoomes, jmasa, tonyp
! src/share/vm/memory/blockOffsetTable.hpp
Changeset: 66b9f90a9211
Author: tonyp
Date: 2010-08-20 13:17 -0400
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/66b9f90a9211
Merge
Changeset: 26faca352942
Author: tonyp
Date: 2010-08-20 12:01 -0700
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/26faca352942
Merge
Changeset: 571f6b35140b
Author: trims
Date: 2010-08-20 12:57 -0700
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/571f6b35140b
6978889: Remove premature change of build number to Hotspot 19 Build 07
Summary: Change the build number back to 06
Reviewed-by: jcoomes
! make/hotspot_version
Changeset: b0b9d64ed9bc
Author: trims
Date: 2010-08-20 14:24 -0700
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/b0b9d64ed9bc
6978915: Remove Mercurial tags for Hotspot 19 Build 06
Summary: Delete the hs19-b06 Hg tag, as it was put on incorrectly
Reviewed-by: jcoomes
! .hgtags
Changeset: 6c43216df135
Author: trims
Date: 2010-08-31 16:48 -0700
URL: http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/6c43216df135
Merge
! .hgtags
More information about the jdk7-dev
mailing list