hg: jdk7/hotspot/hotspot: 7 new changesets

antonios.printezis at sun.com antonios.printezis at sun.com
Sun Apr 11 17:03:32 PDT 2010

Changeset: 56507bcd639e
Author:    tonyp
Date:      2010-03-30 15:36 -0400
URL:       http://hg.openjdk.java.net/jdk7/hotspot/hotspot/rev/56507bcd639e

6937160: G1: should observe GCTimeRatio
Summary: Remove the G1GCPercent parameter, that specifies the desired GC overhead percentage in G1, and observe the GCTimeRatio parameter instead.
Reviewed-by: jmasa, johnc

! src/share/vm/gc_implementation/g1/g1CollectorPolicy.cpp
! src/share/vm/gc_implementation/g1/g1CollectorPolicy.hpp
! src/share/vm/gc_implementation/g1/g1_globals.hpp
! src/share/vm/runtime/arguments.cpp

Changeset: 781e29eb8e08
Author:    tonyp
Date:      2010-04-02 12:10 -0400
URL:       http://hg.openjdk.java.net/jdk7/hotspot/hotspot/rev/781e29eb8e08

6677708: G1: re-enable parallel RSet updating and scanning
Summary: Enable parallel RSet updating and scanning.
Reviewed-by: iveresov, jmasa

! src/share/vm/gc_implementation/g1/g1_globals.hpp

Changeset: 72f725c5a7be
Author:    tonyp
Date:      2010-04-05 12:19 -0400
URL:       http://hg.openjdk.java.net/jdk7/hotspot/hotspot/rev/72f725c5a7be

6940310: G1: MT-unsafe calls to CM::region_stack_push() / CM::region_stack_pop()
Summary: Calling the methods region_stack_push() and region_stack_pop() concurrent is not MT-safe. The assumption is that we will only call region_stack_push() during a GC pause and region_stack_pop() during marking. Unfortunately, we also call region_stack_push() during marking which seems to be introducing subtle marking failures. This change introduces lock-based methods for pushing / popping to be called during marking.
Reviewed-by: iveresov, johnc

! src/share/vm/gc_implementation/g1/concurrentMark.cpp
! src/share/vm/gc_implementation/g1/concurrentMark.hpp
! src/share/vm/runtime/mutexLocker.cpp
! src/share/vm/runtime/mutexLocker.hpp

Changeset: 23b1b27ac76c
Author:    tonyp
Date:      2010-04-06 10:59 -0400
URL:       http://hg.openjdk.java.net/jdk7/hotspot/hotspot/rev/23b1b27ac76c

6909756: G1: guarantee(G1CollectedHeap::heap()->mark_in_progress(),"Precondition.")
Summary: Make sure that two marking cycles do not overlap, i.e., a new one can only start after the concurrent marking thread finishes all its work. In the fix I piggy-back a couple of minor extra fixes: some general code reformatting for consistency (only around the code I modified), the removal of a field (G1CollectorPolicy::_should_initiate_conc_mark) which doesn't seem to be used at all (it's only set but never read), as well as moving the "is GC locker active" test earlier into the G1 pause / Full GC and using a more appropriate method for it.
Reviewed-by: johnc, jmasa, jcoomes, ysr

! src/share/vm/gc_implementation/g1/concurrentMark.cpp
! src/share/vm/gc_implementation/g1/concurrentMarkThread.hpp
! 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: bda703475ded
Author:    johnc
Date:      2010-04-07 11:43 -0700
URL:       http://hg.openjdk.java.net/jdk7/hotspot/hotspot/rev/bda703475ded

6940894: G1: assert(new_obj != 0 || ... "should be forwarded") for compaction tests
Summary: Humongous regions may contain multiple objects as a result of being retained as to-space from a previous GC and then re-used as to-space after being tagged as humongous. These changes include a check that causes retained to-space regions that are now tagged as humongous to be disregarded and a new to-space region allocated.
Reviewed-by: tonyp, iveresov

! src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp

Changeset: 6b73e879f1c2
Author:    tonyp
Date:      2010-04-09 13:08 -0400
URL:       http://hg.openjdk.java.net/jdk7/hotspot/hotspot/rev/6b73e879f1c2


Changeset: e4c77b879561
Author:    tonyp
Date:      2010-04-09 15:01 -0700
URL:       http://hg.openjdk.java.net/jdk7/hotspot/hotspot/rev/e4c77b879561


More information about the hotspot-dev mailing list