src/share/vm/memory/cardTableModRefBS.cpp

*** 427,446 ****
  void CardTableModRefBS::non_clean_card_iterate_possibly_parallel(Space* sp,
                                                                   MemRegion mr,
                                                                   OopsInGenClosure* cl,
                                                                   CardTableRS* ct) {
    if (!mr.is_empty()) {
!     // Caller (process_strong_roots()) claims that all GC threads
      // execute this call.  With UseDynamicNumberOfGCThreads now all
      // active GC threads execute this call.  The number of active GC
      // threads needs to be passed to par_non_clean_card_iterate_work()
      // to get proper partitioning and termination.
      //
      // This is an example of where n_par_threads() is used instead
      // of workers()->active_workers().  n_par_threads can be set to 0 to
      // turn off parallelism.  For example when this code is called as
!     // part of verification and SharedHeap::process_strong_roots() is being
      // used, then n_par_threads() may have been set to 0.  active_workers
      // is not overloaded with the meaning that it is a switch to disable
      // parallelism and so keeps the meaning of the number of
      // active gc workers.  If parallelism has not been shut off by
      // setting n_par_threads to 0, then n_par_threads should be
--- 427,446 ----
  void CardTableModRefBS::non_clean_card_iterate_possibly_parallel(Space* sp,
                                                                   MemRegion mr,
                                                                   OopsInGenClosure* cl,
                                                                   CardTableRS* ct) {
    if (!mr.is_empty()) {
!     // Caller (process_roots()) claims that all GC threads
      // execute this call.  With UseDynamicNumberOfGCThreads now all
      // active GC threads execute this call.  The number of active GC
      // threads needs to be passed to par_non_clean_card_iterate_work()
      // to get proper partitioning and termination.
      //
      // This is an example of where n_par_threads() is used instead
      // of workers()->active_workers().  n_par_threads can be set to 0 to
      // turn off parallelism.  For example when this code is called as
!     // part of verification and SharedHeap::process_roots() is being
      // used, then n_par_threads() may have been set to 0.  active_workers
      // is not overloaded with the meaning that it is a switch to disable
      // parallelism and so keeps the meaning of the number of
      // active gc workers.  If parallelism has not been shut off by
      // setting n_par_threads to 0, then n_par_threads should be