src/share/vm/gc/g1/g1RemSet.cpp

rev 13237 : imported patch 8183226-periodic-rem-set-summary-accesses-uninitialized-stuff

*** 288,300 ****
    _g1p(_g1->g1_policy()),
    _hot_card_cache(hot_card_cache),
    _prev_period_summary(),
    _into_cset_dirty_card_queue_set(false)
  {
-   if (log_is_enabled(Trace, gc, remset)) {
-     _prev_period_summary.initialize(this);
-   }
    // Initialize the card queue set used to hold cards containing
    // references into the collection set.
    _into_cset_dirty_card_queue_set.initialize(NULL, // Should never be called by the Java code
                                               DirtyCardQ_CBL_mon,
                                               DirtyCardQ_FL_lock,
--- 288,297 ----
*** 784,800 ****
    assert(card_processed, "must be");
  
    return update_rs_cl->has_refs_into_cset();
  }
  
  void G1RemSet::print_periodic_summary_info(const char* header, uint period_count) {
    if ((G1SummarizeRSetStatsPeriod > 0) && log_is_enabled(Trace, gc, remset) &&
        (period_count % G1SummarizeRSetStatsPeriod == 0)) {
  
!     if (!_prev_period_summary.initialized()) {
!       _prev_period_summary.initialize(this);
!     }
  
      G1RemSetSummary current;
      current.initialize(this);
      _prev_period_summary.subtract_from(&current);
  
--- 781,801 ----
    assert(card_processed, "must be");
  
    return update_rs_cl->has_refs_into_cset();
  }
  
+ void G1RemSet::initialize_periodic_summary_info() {
+   if (log_is_enabled(Trace, gc, remset)) {
+     _prev_period_summary.initialize(this);
+   }
+ }
+ 
  void G1RemSet::print_periodic_summary_info(const char* header, uint period_count) {
    if ((G1SummarizeRSetStatsPeriod > 0) && log_is_enabled(Trace, gc, remset) &&
        (period_count % G1SummarizeRSetStatsPeriod == 0)) {
  
!     assert(_prev_period_summary.initialized(), "Periodic remembered set summary must be initialized.");
  
      G1RemSetSummary current;
      current.initialize(this);
      _prev_period_summary.subtract_from(&current);