changeset 59698:7589335ecde5

8246272: Make use of GCLogPrecious for G1, Parallel and Serial Reviewed-by: pliden, eosterlund, sjohanss
author stefank
date Wed, 10 Jun 2020 08:57:50 +0200
parents b1bd79b486bc
children 34a4e4291d10
files src/hotspot/share/gc/g1/g1InitLogger.cpp src/hotspot/share/gc/shared/gcInitLogger.cpp
diffstat 2 files changed, 28 insertions(+), 27 deletions(-) [+]
line wrap: on
line diff
--- a/src/hotspot/share/gc/g1/g1InitLogger.cpp	Wed Jun 10 12:43:22 2020 +0530
+++ b/src/hotspot/share/gc/g1/g1InitLogger.cpp	Wed Jun 10 08:57:50 2020 +0200
@@ -24,29 +24,29 @@
 
 #include "precompiled.hpp"
 #include "gc/g1/g1InitLogger.hpp"
-#include "logging/log.hpp"
+#include "gc/shared/gcLogPrecious.hpp"
 #include "runtime/globals.hpp"
 #include "utilities/globalDefinitions.hpp"
 
 void G1InitLogger::print_heap() {
-  log_info(gc, init)("Heap Region Size: " SIZE_FORMAT "M", G1HeapRegionSize / M);
+  log_info_p(gc, init)("Heap Region Size: " SIZE_FORMAT "M", G1HeapRegionSize / M);
   GCInitLogger::print_heap();
 }
 
 void G1InitLogger::print_workers() {
   GCInitLogger::print_workers();
   if (G1ConcRefinementThreads > 0) {
-    log_info(gc, init)("Concurrent Refinement Workers: %u", G1ConcRefinementThreads);
+    log_info_p(gc, init)("Concurrent Refinement Workers: %u", G1ConcRefinementThreads);
   }
 }
 
 void G1InitLogger::print_gc_specific() {
   // Print a message about periodic GC configuration.
   if (G1PeriodicGCInterval != 0) {
-    log_info(gc, init)("Periodic GC: Enabled");
-    log_info(gc, init)("Periodic GC Interval: " UINTX_FORMAT "ms", G1PeriodicGCInterval);
+    log_info_p(gc, init)("Periodic GC: Enabled");
+    log_info_p(gc, init)("Periodic GC Interval: " UINTX_FORMAT "ms", G1PeriodicGCInterval);
   } else {
-    log_info(gc, init)("Periodic GC: Disabled");
+    log_info_p(gc, init)("Periodic GC: Disabled");
   }
 }
 
--- a/src/hotspot/share/gc/shared/gcInitLogger.cpp	Wed Jun 10 12:43:22 2020 +0530
+++ b/src/hotspot/share/gc/shared/gcInitLogger.cpp	Wed Jun 10 08:57:50 2020 +0200
@@ -24,6 +24,7 @@
 
 #include "precompiled.hpp"
 #include "gc/shared/gcInitLogger.hpp"
+#include "gc/shared/gcLogPrecious.hpp"
 #include "logging/log.hpp"
 #include "oops/compressedOops.hpp"
 #include "runtime/globals.hpp"
@@ -54,56 +55,56 @@
 }
 
 void GCInitLogger::print_cpu() {
-  log_info(gc, init)("CPUs: %u total, %u available",
-                     os::processor_count(),
-                     os::initial_active_processor_count());
+  log_info_p(gc, init)("CPUs: %u total, %u available",
+                       os::processor_count(),
+                       os::initial_active_processor_count());
 }
 
 void GCInitLogger::print_memory() {
   julong memory = os::physical_memory();
-  log_info(gc, init)("Memory: " JULONG_FORMAT "%s",
-                     byte_size_in_proper_unit(memory), proper_unit_for_byte_size(memory));
+  log_info_p(gc, init)("Memory: " JULONG_FORMAT "%s",
+                       byte_size_in_proper_unit(memory), proper_unit_for_byte_size(memory));
 }
 
 void GCInitLogger::print_large_pages() {
-  log_info(gc, init)("Large Page Support: %s", large_pages_support());
+  log_info_p(gc, init)("Large Page Support: %s", large_pages_support());
 }
 
 void GCInitLogger::print_numa() {
   if (UseNUMA) {
-    log_info(gc, init)("NUMA Support: Enabled");
-    log_info(gc, init)("NUMA Nodes: " SIZE_FORMAT, os::numa_get_groups_num());
+    log_info_p(gc, init)("NUMA Support: Enabled");
+    log_info_p(gc, init)("NUMA Nodes: " SIZE_FORMAT, os::numa_get_groups_num());
   } else {
-    log_info(gc, init)("NUMA Support: Disabled");
+    log_info_p(gc, init)("NUMA Support: Disabled");
   }
 }
 
 void GCInitLogger::print_compressed_oops() {
   if (UseCompressedOops) {
-    log_info(gc, init)("Compressed Oops: Enabled (%s)",
-                       CompressedOops::mode_to_string(CompressedOops::mode()));
+    log_info_p(gc, init)("Compressed Oops: Enabled (%s)",
+                         CompressedOops::mode_to_string(CompressedOops::mode()));
   } else {
-    log_info(gc, init)("Compressed Oops: Disabled");
+    log_info_p(gc, init)("Compressed Oops: Disabled");
   }
 }
 
 void GCInitLogger::print_heap() {
-  log_info(gc, init)("Heap Min Capacity: " SIZE_FORMAT "%s",
-                     byte_size_in_exact_unit(MinHeapSize), exact_unit_for_byte_size(MinHeapSize));
-  log_info(gc, init)("Heap Initial Capacity: " SIZE_FORMAT "%s",
-                     byte_size_in_exact_unit(InitialHeapSize), exact_unit_for_byte_size(InitialHeapSize));
-  log_info(gc, init)("Heap Max Capacity: " SIZE_FORMAT "%s",
-                     byte_size_in_exact_unit(MaxHeapSize), exact_unit_for_byte_size(MaxHeapSize));
+  log_info_p(gc, init)("Heap Min Capacity: " SIZE_FORMAT "%s",
+                       byte_size_in_exact_unit(MinHeapSize), exact_unit_for_byte_size(MinHeapSize));
+  log_info_p(gc, init)("Heap Initial Capacity: " SIZE_FORMAT "%s",
+                       byte_size_in_exact_unit(InitialHeapSize), exact_unit_for_byte_size(InitialHeapSize));
+  log_info_p(gc, init)("Heap Max Capacity: " SIZE_FORMAT "%s",
+                       byte_size_in_exact_unit(MaxHeapSize), exact_unit_for_byte_size(MaxHeapSize));
 
-  log_info(gc, init)("Pre-touch: %s", AlwaysPreTouch ? "Enabled" : "Disabled");
+  log_info_p(gc, init)("Pre-touch: %s", AlwaysPreTouch ? "Enabled" : "Disabled");
 }
 
 void GCInitLogger::print_workers() {
   if (ParallelGCThreads > 0) {
-    log_info(gc, init)("Parallel Workers: %u", ParallelGCThreads);
+    log_info_p(gc, init)("Parallel Workers: %u", ParallelGCThreads);
   }
   if (ConcGCThreads > 0) {
-    log_info(gc, init)("Concurrent Workers: %u", ConcGCThreads);
+    log_info_p(gc, init)("Concurrent Workers: %u", ConcGCThreads);
   }
 }