src/hotspot/share/gc/g1/g1RootProcessor.cpp

rev 50285 : 8195097: Make it possible to process StringTable outside safepoint
Reviewed-by:

*** 36,45 ****
--- 36,46 ----
  #include "gc/g1/g1ParScanThreadState.inline.hpp"
  #include "gc/g1/g1Policy.hpp"
  #include "gc/g1/g1RootClosures.hpp"
  #include "gc/g1/g1RootProcessor.hpp"
  #include "gc/g1/heapRegion.inline.hpp"
+ #include "gc/shared/oopStorageParState.hpp"
  #include "gc/shared/referenceProcessor.hpp"
  #include "gc/shared/weakProcessor.hpp"
  #include "memory/allocation.inline.hpp"
  #include "runtime/mutex.hpp"
  #include "services/management.hpp"
*** 70,79 ****
--- 71,81 ----
  G1RootProcessor::G1RootProcessor(G1CollectedHeap* g1h, uint n_workers) :
      _g1h(g1h),
      _process_strong_tasks(G1RP_PS_NumElements),
      _srs(n_workers),
      _lock(Mutex::leaf, "G1 Root Scanning barrier lock", false, Monitor::_safepoint_check_never),
+     _par_state_string(StringTable::weak_storage()),
      _n_workers_discovered_strong_classes(0) {}
  
  void G1RootProcessor::evacuate_roots(G1ParScanThreadState* pss, uint worker_i) {
    G1GCPhaseTimes* phase_times = _g1h->g1_policy()->phase_times();
  
*** 300,310 ****
                                                   uint worker_i) {
    assert(closures->weak_oops() != NULL, "Should only be called when all roots are processed");
    G1GCParPhaseTimesTracker x(phase_times, G1GCPhaseTimes::StringTableRoots, worker_i);
    // All threads execute the following. A specific chunk of buckets
    // from the StringTable are the individual tasks.
!   StringTable::possibly_parallel_oops_do(closures->weak_oops());
  }
  
  void G1RootProcessor::process_code_cache_roots(CodeBlobClosure* code_closure,
                                                 G1GCPhaseTimes* phase_times,
                                                 uint worker_i) {
--- 302,312 ----
                                                   uint worker_i) {
    assert(closures->weak_oops() != NULL, "Should only be called when all roots are processed");
    G1GCParPhaseTimesTracker x(phase_times, G1GCPhaseTimes::StringTableRoots, worker_i);
    // All threads execute the following. A specific chunk of buckets
    // from the StringTable are the individual tasks.
!   StringTable::possibly_parallel_oops_do(&_par_state_string, closures->weak_oops());
  }
  
  void G1RootProcessor::process_code_cache_roots(CodeBlobClosure* code_closure,
                                                 G1GCPhaseTimes* phase_times,
                                                 uint worker_i) {