# HG changeset patch # User tschatzl # Date 1390214849 -3600 # Node ID cb7ec242320700f5a3edf6010abe493271ae8a2d # Parent 5a32d2a3cc1ebd53ea7a646fab9e9f9503488651 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 5a32d2a3cc1e -r cb7ec2423207 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() {