changeset 5180:d2782fe9769e

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
author tschatzl
date Fri, 31 Jan 2014 10:09:02 +0100
parents c76437921025
children b6d3ebe42303
files src/share/vm/gc_implementation/g1/concurrentMark.cpp
diffstat 1 files changed, 3 insertions(+), 1 deletions(-) [+]
line wrap: on
line diff
--- a/src/share/vm/gc_implementation/g1/concurrentMark.cpp	Fri Jan 31 10:08:49 2014 +0100
+++ b/src/share/vm/gc_implementation/g1/concurrentMark.cpp	Fri Jan 31 10:09:02 2014 +0100
@@ -2406,7 +2406,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() {