src/hotspot/share/gc/parallel/pcTasks.cpp

8211446_01: revision due to comments from StefanJ and Thomas

*** 57,67 ****
    ResourceMark rm;
  
    ParCompactionManager* cm =
      ParCompactionManager::gc_thread_compaction_manager(which);
  
!   PSMarkAndPushClosure mark_and_push_closure(cm);
    MarkingCodeBlobClosure mark_and_push_in_blobs(&mark_and_push_closure, !CodeBlobToOopClosure::FixRelocations);
  
    _thread->oops_do(&mark_and_push_closure, &mark_and_push_in_blobs);
  
    // Do the real work
--- 57,67 ----
    ResourceMark rm;
  
    ParCompactionManager* cm =
      ParCompactionManager::gc_thread_compaction_manager(which);
  
!   PCMarkAndPushClosure mark_and_push_closure(cm);
    MarkingCodeBlobClosure mark_and_push_in_blobs(&mark_and_push_closure, !CodeBlobToOopClosure::FixRelocations);
  
    _thread->oops_do(&mark_and_push_closure, &mark_and_push_in_blobs);
  
    // Do the real work
***************
*** 72,82 ****
  void MarkFromRootsTask::do_it(GCTaskManager* manager, uint which) {
    assert(ParallelScavengeHeap::heap()->is_gc_active(), "called outside gc");
  
    ParCompactionManager* cm =
      ParCompactionManager::gc_thread_compaction_manager(which);
!   PSMarkAndPushClosure mark_and_push_closure(cm);
  
    switch (_root_type) {
      case universe:
        Universe::oops_do(&mark_and_push_closure);
        break;
--- 72,82 ----
  void MarkFromRootsTask::do_it(GCTaskManager* manager, uint which) {
    assert(ParallelScavengeHeap::heap()->is_gc_active(), "called outside gc");
  
    ParCompactionManager* cm =
      ParCompactionManager::gc_thread_compaction_manager(which);
!   PCMarkAndPushClosure mark_and_push_closure(cm);
  
    switch (_root_type) {
      case universe:
        Universe::oops_do(&mark_and_push_closure);
        break;
***************
*** 138,148 ****
  {
    assert(ParallelScavengeHeap::heap()->is_gc_active(), "called outside gc");
  
    ParCompactionManager* cm =
      ParCompactionManager::gc_thread_compaction_manager(which);
!   PSMarkAndPushClosure mark_and_push_closure(cm);
    ParCompactionManager::FollowStackClosure follow_stack_closure(cm);
    _rp_task.work(_work_id, *PSParallelCompact::is_alive_closure(),
                  mark_and_push_closure, follow_stack_closure);
  }
  
--- 138,148 ----
  {
    assert(ParallelScavengeHeap::heap()->is_gc_active(), "called outside gc");
  
    ParCompactionManager* cm =
      ParCompactionManager::gc_thread_compaction_manager(which);
!   PCMarkAndPushClosure mark_and_push_closure(cm);
    ParCompactionManager::FollowStackClosure follow_stack_closure(cm);
    _rp_task.work(_work_id, *PSParallelCompact::is_alive_closure(),
                  mark_and_push_closure, follow_stack_closure);
  }