8273372: Remove scavenge trace message in psPromotionManager

Reviewed-by: tschatzl, lkorinth
openjdk · Sep 16, 2021 · 14dc517 · 14dc517 · openjdk-notifier · Sep 16, 2021
1 parent 241ac89
commit 14dc517
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 10 deletions.
diff --git a/src/hotspot/share/gc/parallel/psPromotionManager.cpp b/src/hotspot/share/gc/parallel/psPromotionManager.cpp
@@ -369,7 +369,5 @@ oop PSPromotionManager::oop_promotion_failed(oop obj, markWord obj_mark) {
     obj = obj->forwardee();
   }
 
-  log_develop_trace(gc, scavenge)("{promotion-failure %s " PTR_FORMAT " (%d)}", obj->klass()->internal_name(), p2i(obj), obj->size());
-
   return obj;
 }
diff --git a/src/hotspot/share/gc/parallel/psPromotionManager.inline.hpp b/src/hotspot/share/gc/parallel/psPromotionManager.inline.hpp
@@ -267,11 +267,6 @@ inline oop PSPromotionManager::copy_unmarked_to_survivor_space(oop o,
       assert(young_space()->contains(new_obj), "Attempt to push non-promoted obj");
     }
 
-    log_develop_trace(gc, scavenge)("{%s %s " PTR_FORMAT " -> " PTR_FORMAT " (%d)}",
-                                    new_obj_is_tenured ? "copying" : "tenuring",
-                                    new_obj->klass()->internal_name(),
-                                    p2i((void *)o), p2i((void *)new_obj), new_obj->size());
-
     // Do the size comparison first with new_obj_size, which we
     // already have. Hopefully, only a few objects are larger than
     // _min_array_size_for_chunking, and most of them will be arrays.

diff --git a/src/hotspot/share/gc/shared/referenceProcessor.cpp b/src/hotspot/share/gc/shared/referenceProcessor.cpp
@@ -469,7 +469,6 @@ class RefProcSoftWeakFinalPhaseTask: public RefProcTask {
                BoolObjectClosure* is_alive,
                OopClosure* keep_alive,
                VoidClosure* complete_gc) override {
-    ResourceMark rm;
     RefProcWorkerTimeTracker t(_phase_times->soft_weak_final_refs_phase_worker_time_sec(), tracker_id(worker_id));
 
     process_discovered_list(worker_id, REF_SOFT, is_alive, keep_alive);
@@ -495,7 +494,6 @@ class RefProcKeepAliveFinalPhaseTask: public RefProcTask {
                BoolObjectClosure* is_alive,
                OopClosure* keep_alive,
                VoidClosure* complete_gc) override {
-    ResourceMark rm;
     RefProcSubPhasesWorkerTimeTracker tt(ReferenceProcessor::KeepAliveFinalRefsSubPhase, _phase_times, tracker_id(worker_id));
     _ref_processor.process_final_keep_alive_work(_ref_processor._discoveredFinalRefs[worker_id], keep_alive);
     // Close the reachable set
@@ -514,7 +512,6 @@ class RefProcPhantomPhaseTask: public RefProcTask {
                BoolObjectClosure* is_alive,
                OopClosure* keep_alive,
                VoidClosure* complete_gc) override {
-    ResourceMark rm;
     process_discovered_list(worker_id, REF_PHANTOM, is_alive, keep_alive);
 
     // Close the reachable set; needed for collectors which keep_alive_closure do