# HG changeset patch
# User tschatzl
# Date 1390214849 -3600
# Node ID a2866d45e99efc29be4f000a89e8c058f0c0b5d7
# Parent  893ce66f7473776e3a327aefc33396a43f02cb92
8027454: Do not traverse string table during G1 remark when treating them as strong roots during initial mark
Summary: Do not try to unlink string table entries unconditionally during remark. This is without side effects as the preceding initial mark always uses the string table as strong roots. Needs to be fixed with class unloading during concurrent mark.
Reviewed-by: brutisso, mgerdin

diff -r 893ce66f7473 -r a2866d45e99e src/share/vm/gc_implementation/g1/concurrentMark.cpp
--- a/src/share/vm/gc_implementation/g1/concurrentMark.cpp	Mon Jan 20 11:47:07 2014 +0100
+++ b/src/share/vm/gc_implementation/g1/concurrentMark.cpp	Mon Jan 20 11:47:29 2014 +0100
@@ -2528,7 +2528,9 @@
     assert(!rp->discovery_enabled(), "Post condition");
   }
 
-  g1h->unlink_string_and_symbol_table(&g1_is_alive);
+  g1h->unlink_string_and_symbol_table(&g1_is_alive,
+                                      /* process_strings */ false, // currently strings are always roots
+                                      /* process_symbols */ true);
 }
 
 void ConcurrentMark::swapMarkBitMaps() {