hg: jdk8u/hs-dev/hotspot: 15 new changesets
thomas.schatzl at oracle.com
thomas.schatzl at oracle.com
Mon Jul 21 09:05:13 UTC 2014
Changeset: 82693fb204a5
Author: tschatzl
Date: 2014-04-16 10:14 +0200
URL: http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/82693fb204a5
8038930: G1CodeRootSet::test fails with assert(_num_chunks_handed_out == 0) failed: No elements must have been handed out yet
Summary: The test incorrectly assumed that it had been started with no other previous compilation activity. Fix this by allowing multiple code root free chunk lists, and use one separate from the global one to perform the test.
Reviewed-by: brutisso
! src/share/vm/gc_implementation/g1/g1CodeCacheRemSet.cpp
! src/share/vm/gc_implementation/g1/g1CodeCacheRemSet.hpp
! src/share/vm/gc_implementation/g1/heapRegionRemSet.hpp
Changeset: d7e2d5f2846b
Author: tschatzl
Date: 2014-07-21 09:40 +0200
URL: http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/d7e2d5f2846b
8027553: Change the in_cset_fast_test functionality to use the G1BiasedArray abstraction
Summary: Instead of using a manually managed array for the in_cset_fast_test array, use a G1BiasedArray instance.
Reviewed-by: brutisso, mgerdin
! 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
Changeset: ff7b317d2af8
Author: tschatzl
Date: 2014-07-21 09:40 +0200
URL: http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/ff7b317d2af8
8037344: Use the "next" field to iterate over fine remembered instead of using the hash table
Summary: After changes to the PerRegionTable where all these PRTs are linked together in an additional field, simplify iterating over all PRTs by using these links instead of walki
Reviewed-by: mgerdin, jwilhelm, brutisso
! src/share/vm/gc_implementation/g1/heapRegionRemSet.cpp
! src/share/vm/gc_implementation/g1/heapRegionRemSet.hpp
Changeset: 8cc89a893545
Author: tschatzl
Date: 2014-07-21 09:40 +0200
URL: http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/8cc89a893545
8039596: Remove HeapRegionRemSet::clear_incoming_entry
Summary: The mentioned method is never used and out of date. So it is removed.
Reviewed-by: mgerdin, brutisso
! src/share/vm/gc_implementation/g1/heapRegionRemSet.cpp
! src/share/vm/gc_implementation/g1/heapRegionRemSet.hpp
Changeset: 983092f35ff7
Author: tschatzl
Date: 2014-07-21 09:40 +0200
URL: http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/983092f35ff7
8028710: G1 does not retire allocation buffers after reference processing work
Summary: G1 does not retire allocation buffers after reference processing work when -XX:+ParallelRefProcEnabled is enabled. This causes wrong calculation of PLAB sizes, as the amount of space wasted is not updated correctly.
Reviewed-by: brutisso
! src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp
! src/share/vm/gc_implementation/g1/g1CollectedHeap.hpp
! src/share/vm/gc_implementation/shared/parGCAllocBuffer.hpp
Changeset: 570cb6369f17
Author: tschatzl
Date: 2014-07-21 09:40 +0200
URL: http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/570cb6369f17
8019342: G1: High "Other" time most likely due to card redirtying
Summary: Parallelize card redirtying to decrease the time it takes.
Reviewed-by: brutisso
! src/share/vm/gc_implementation/g1/concurrentG1Refine.cpp
! src/share/vm/gc_implementation/g1/concurrentG1Refine.hpp
! src/share/vm/gc_implementation/g1/concurrentG1RefineThread.cpp
! src/share/vm/gc_implementation/g1/concurrentG1RefineThread.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/g1GCPhaseTimes.cpp
! src/share/vm/gc_implementation/g1/g1GCPhaseTimes.hpp
! test/gc/g1/TestGCLogMessages.java
Changeset: 6b52700a896b
Author: tschatzl
Date: 2014-07-21 09:40 +0200
URL: http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/6b52700a896b
8040002: Clean up code and code duplication in re-diryting cards for verification
Summary: Card re-dirtying code for verification and actual redirtying uses two different, almost completely identical card closures. Also the verification code still assumes a perm gen.
Reviewed-by: brutisso, jmasa
! src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp
Changeset: 828056cf311f
Author: tschatzl
Date: 2014-07-21 09:40 +0200
URL: http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/828056cf311f
8040792: G1: Memory usage calculation uses sizeof(this) instead of sizeof(classname)
Summary: A few locations in the code use sizeof(this) which returns the size of the pointer instead of sizeof(classname) which returns the size of the sum of its members. This change fixes these errors and adds a few tests.
Reviewed-by: mgerdin, brutisso
! src/share/vm/gc_implementation/g1/g1CodeCacheRemSet.cpp
! src/share/vm/gc_implementation/g1/g1CodeCacheRemSet.hpp
! src/share/vm/gc_implementation/g1/heapRegionRemSet.cpp
! src/share/vm/gc_implementation/g1/heapRegionRemSet.hpp
! src/share/vm/gc_implementation/g1/sparsePRT.cpp
Changeset: f40816c5e359
Author: tschatzl
Date: 2014-07-21 09:40 +0200
URL: http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/f40816c5e359
8026784: Error message in AdaptiveFreeList<Chunk>::verify_stats is wrong
Summary: Changed faulty error message in the verification code
Reviewed-by: stefank, tschatzl
Contributed-by: Andreas Sjoberg <andreas.sjoberg at oracle.com>
! src/share/vm/gc_implementation/concurrentMarkSweep/adaptiveFreeList.cpp
Changeset: d15367d92f0d
Author: tschatzl
Date: 2014-07-21 09:40 +0200
URL: http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/d15367d92f0d
8043723: max_heap_for_compressed_oops() declared with size_t, but defined with uintx
Summary: The mismatch in declaration and definition has been encountered when compiling on a platform where the sizes are different. Change the method definition to match the declaration.
Reviewed-by: tschatzl, sjohanss
Contributed-by: Dan Horak <dhorak at redhat.com>
! src/share/vm/runtime/arguments.cpp
Changeset: ab5fbf410512
Author: tschatzl
Date: 2014-07-21 09:40 +0200
URL: http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/ab5fbf410512
8043722: Swapped usage of idx_t and bm_word_t types in parMarkBitMap.cpp
Summary: Clean up usage of idx_t and uintptr_t when using it in conjunction with BitMap::set_map(), casting to the appropriate type. Fixes compilation on S390.
Reviewed-by: tschatzl
Contributed-by: Dan Horak <dhorak at redhat.com>
! src/share/vm/compiler/methodLiveness.cpp
! src/share/vm/gc_implementation/g1/concurrentMark.cpp
! src/share/vm/gc_implementation/parallelScavenge/parMarkBitMap.cpp
Changeset: 0abcece2ee27
Author: tschatzl
Date: 2014-07-21 09:40 +0200
URL: http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/0abcece2ee27
8026847: [TESTBUG] gc/g1/TestSummarizeRSetStats* tests launch 32bit jvm with UseCompressedOops
Summary: The test launched a 32 bit VM with UseCompressedOops enabled. This is not supported on 32 bit VMs, causing a test failure. Investigation showed that the use of this flag is not required at all, so simply remove it.
Reviewed-by: tschatzl, jwilhelm
Contributed-by: Andrey Zakharov <andrey.x.zakharov at oracle.com>
! test/gc/g1/TestSummarizeRSetStatsTools.java
Changeset: b0c374311c4e
Author: tschatzl
Date: 2014-07-21 09:41 +0200
URL: http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/b0c374311c4e
8035400: Move G1ParScanThreadState into its own files
Summary: Extract the G1ParScanThreadState class from G1CollectedHeap.?pp into its own files.
Reviewed-by: brutisso, mgerdin
! 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/g1OopClosures.cpp
! src/share/vm/gc_implementation/g1/g1OopClosures.inline.hpp
+ src/share/vm/gc_implementation/g1/g1ParScanThreadState.cpp
+ src/share/vm/gc_implementation/g1/g1ParScanThreadState.hpp
+ src/share/vm/gc_implementation/g1/g1ParScanThreadState.inline.hpp
! src/share/vm/gc_implementation/g1/g1RemSet.inline.hpp
Changeset: a2328cbebb23
Author: tschatzl
Date: 2014-07-21 09:41 +0200
URL: http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/a2328cbebb23
8035401: Fix visibility of G1ParScanThreadState members
Summary: After JDK-8035400 there were several opportunities to fix the visibility of several members of the G1ParScanThreadState class.
Reviewed-by: brutisso, mgerdin
! src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp
! src/share/vm/gc_implementation/g1/g1ParScanThreadState.cpp
! src/share/vm/gc_implementation/g1/g1ParScanThreadState.hpp
! src/share/vm/gc_implementation/g1/g1ParScanThreadState.inline.hpp
Changeset: cd43876f692e
Author: tschatzl
Date: 2014-07-21 09:41 +0200
URL: http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/cd43876f692e
8040977: G1 crashes when run with -XX:-G1DeferredRSUpdate
Summary: When G1 is run with -XX:-G1DeferredRSUpdate, the VM crashes because of wrong initialization order of member variables. The change makes the initalization explicit, not relying on initialization order any more.
Reviewed-by: brutisso, mgerdin
! src/share/vm/gc_implementation/g1/g1OopClosures.cpp
! src/share/vm/gc_implementation/g1/g1OopClosures.hpp
! src/share/vm/gc_implementation/g1/g1ParScanThreadState.cpp
+ test/gc/g1/TestDeferredRSUpdate.java
More information about the jdk8u-all-changes
mailing list