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

*** 3537,3546 ****
--- 3537,3554 ----
    // Fill TLAB's and such
    ensure_parsability(true);
  }
  
  void G1CollectedHeap::gc_epilogue(bool full /* Ignored */) {
+ 
+   if (G1SummarizeRSetStats &&
+       (G1SummarizeRSetStatsPeriod > 0) &&
+       // we are at the end of the GC. Total collections has already been increased.
+       ((total_collections() - 1) % G1SummarizeRSetStatsPeriod == 0)) {
+     g1_rem_set()->print_periodic_summary_info();
+   }
+ 
    // FIXME: what is this about?
    // I'm ignoring the "fill_newgen()" call if "alloc_event_enabled"
    // is set.
    COMPILER2_PRESENT(assert(DerivedPointerTable::is_empty(),
                          "derived pointer present"));
*** 4091,4106 ****
      // TraceMemoryManagerStats is called) so that the G1 memory pools are updated
      // before any GC notifications are raised.
      g1mm()->update_sizes();
    }
  
-   if (G1SummarizeRSetStats &&
-       (G1SummarizeRSetStatsPeriod > 0) &&
-       (total_collections() % G1SummarizeRSetStatsPeriod == 0)) {
-     g1_rem_set()->print_summary_info();
-   }
- 
    // It should now be safe to tell the concurrent mark thread to start
    // without its logging output interfering with the logging output
    // that came from the pause.
  
    if (should_start_conc_mark) {
--- 4099,4108 ----