hg: hsx/hsx20/baseline: 3 new changesets

tony.printezis at oracle.com tony.printezis at oracle.com
Wed Feb 9 11:44:54 PST 2011

Changeset: bd2e08334e84
Author:    tonyp
Date:      2011-01-21 11:30 -0500
URL:       http://hg.openjdk.java.net/hsx/hsx20/baseline/rev/bd2e08334e84

7013718: G1: small fixes for two assert/guarantee failures
Summary: Two small fixes to deal with a guarantee failure (the marking thread should join the SuspendibleThreadSet before calling a method that does pause prediction work so that said method is never called during a pause) and an assert failure (an assert is too strong).
Reviewed-by: iveresov, johnc

! src/share/vm/gc_implementation/g1/concurrentMarkThread.cpp
! src/share/vm/gc_implementation/g1/heapRegionSeq.cpp

Changeset: 4e66274b6bb3
Author:    tonyp
Date:      2011-01-25 17:58 -0500
URL:       http://hg.openjdk.java.net/hsx/hsx20/baseline/rev/4e66274b6bb3

7014261: G1: RSet-related failures
Summary: A race between the concurrent cleanup thread and the VM thread while it is processing the "expanded sparse table list" causes both threads to try to free the same sparse table entry and either causes one of the threads to fail or leaves the entry in an inconsistent state. The solution is purge all entries on the expanded list that correspond go regions that are being cleaned up.
Reviewed-by: brutisso, johnc

! src/share/vm/gc_implementation/g1/concurrentMark.cpp
! src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp
! src/share/vm/gc_implementation/g1/g1CollectedHeap.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
! src/share/vm/gc_implementation/g1/sparsePRT.hpp

Changeset: 908c5092d72a
Author:    tonyp
Date:      2011-01-31 16:28 -0500
URL:       http://hg.openjdk.java.net/hsx/hsx20/baseline/rev/908c5092d72a

7014679: G1: deadlock during concurrent cleanup
Summary: There's a potential deadlock between the concurrent cleanup thread and the GC workers that are trying to allocate and waiting for more free regions to be made available.
Reviewed-by: iveresov, jcoomes

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

More information about the hotspot-dev mailing list