RFR (sh/jdk11): [backport] 8224043: Shenandoah: Remove clear_claimed_marks() from start of concurrent_traversal()

Roman Kennke rkennke at redhat.com
Thu May 16 20:32:37 UTC 2019


Let's backport today's change to remove clear_claimed_marks() to jdk11.

Testing: hotspot_gc_shenandoah and some point-testing specjvm

Ok?

The other remaining changes seem to be LRB or arraycopy related, which I
would do separately. (And arraycopy is actually missing the real change
JDK-8223240 and JDK-8222859, so maybe we wait for them...)

Roman


# HG changeset patch
# User rkennke
# Date 1558026451 -7200
#      Thu May 16 19:07:31 2019 +0200
# Node ID a2f8baebe8ce4254881fdb53c4ddffe5bed9b3b1
# Parent  649b5a98e4138a1e4e066579f4c89fa7dce15b1c
[backport] 8224043: Shenandoah: Remove clear_claimed_marks() from start
of concurrent_traversal()
Reviewed-by: shade

diff -r 649b5a98e413 -r a2f8baebe8ce
src/hotspot/share/gc/shenandoah/shenandoahTraversalGC.cpp
--- a/src/hotspot/share/gc/shenandoah/shenandoahTraversalGC.cpp	Fri May
10 01:41:45 2019 +0800
+++ b/src/hotspot/share/gc/shenandoah/shenandoahTraversalGC.cpp	Thu May
16 19:07:31 2019 +0200
@@ -564,8 +564,6 @@
 }

 void ShenandoahTraversalGC::concurrent_traversal_collection() {
-  ClassLoaderDataGraph::clear_claimed_marks();
-
   ShenandoahGCPhase phase_work(ShenandoahPhaseTimings::conc_traversal);
   if (!_heap->cancelled_gc()) {
     uint nworkers = _heap->workers()->active_workers();



More information about the shenandoah-dev mailing list