src/share/vm/gc_implementation/g1/g1ParScanThreadState.cpp

rev 6334 : 8035400: Move G1ParScanThreadState into its own files
Summary: Extract the G1ParScanThreadState class from G1CollectedHeap.?pp into its own files.
Reviewed-by: brutisso, mgerdin
rev 6335 : 8035401: Fix visibility of G1ParScanThreadState members
Summary: After JDK-8035400 there were several opportunities to fix the visibility of several members of the G1ParScanThreadState class.
Reviewed-by: brutisso, mgerdin
rev 6336 : 8040977: G1 crashes when run with -XX:-G1DeferredRSUpdate
Summary: When G1 is run with -XX:-G1DeferredRSUpdate, the VM crashes because of wrong initialization order of member variables. The change makes the initalization explicit, not relying on initialization order any more.
Reviewed-by: brutisso, mgerdin

*** 28,54 ****
  #include "gc_implementation/g1/g1ParScanThreadState.inline.hpp"
  #include "oops/oop.inline.hpp"
  #include "oops/oop.pcgc.inline.hpp"
  #include "runtime/prefetch.inline.hpp"
  
- #ifdef _MSC_VER // the use of 'this' below gets a warning, make it go away
- #pragma warning( disable:4355 ) // 'this' : used in base member initializer list
- #endif // _MSC_VER
- 
  G1ParScanThreadState::G1ParScanThreadState(G1CollectedHeap* g1h, uint queue_num, ReferenceProcessor* rp)
    : _g1h(g1h),
      _refs(g1h->task_queue(queue_num)),
      _dcq(&g1h->dirty_card_queue_set()),
      _ct_bs(g1h->g1_barrier_set()),
      _g1_rem(g1h->g1_rem_set()),
      _hash_seed(17), _queue_num(queue_num),
      _term_attempts(0),
      _surviving_alloc_buffer(g1h->desired_plab_sz(GCAllocForSurvived)),
      _tenured_alloc_buffer(g1h->desired_plab_sz(GCAllocForTenured)),
!     _age_table(false), _scanner(g1h, this, rp),
      _strong_roots_time(0), _term_time(0),
      _alloc_buffer_waste(0), _undo_waste(0) {
    // we allocate G1YoungSurvRateNumRegions plus one entries, since
    // we "sacrifice" entry 0 to keep track of surviving bytes for
    // non-young regions (where the age is -1)
    // We also add a few elements at the beginning and at the end in
    // an attempt to eliminate cache contention
--- 28,51 ----
  #include "gc_implementation/g1/g1ParScanThreadState.inline.hpp"
  #include "oops/oop.inline.hpp"
  #include "oops/oop.pcgc.inline.hpp"
  #include "runtime/prefetch.inline.hpp"
  
  G1ParScanThreadState::G1ParScanThreadState(G1CollectedHeap* g1h, uint queue_num, ReferenceProcessor* rp)
    : _g1h(g1h),
      _refs(g1h->task_queue(queue_num)),
      _dcq(&g1h->dirty_card_queue_set()),
      _ct_bs(g1h->g1_barrier_set()),
      _g1_rem(g1h->g1_rem_set()),
      _hash_seed(17), _queue_num(queue_num),
      _term_attempts(0),
      _surviving_alloc_buffer(g1h->desired_plab_sz(GCAllocForSurvived)),
      _tenured_alloc_buffer(g1h->desired_plab_sz(GCAllocForTenured)),
!     _age_table(false), _scanner(g1h, rp),
      _strong_roots_time(0), _term_time(0),
      _alloc_buffer_waste(0), _undo_waste(0) {
+   _scanner.set_par_scan_thread_state(this);
    // we allocate G1YoungSurvRateNumRegions plus one entries, since
    // we "sacrifice" entry 0 to keep track of surviving bytes for
    // non-young regions (where the age is -1)
    // We also add a few elements at the beginning and at the end in
    // an attempt to eliminate cache contention