8251392: Consolidate Metaspace Statistics

Reviewed-by: coleenp, zgu

Author: tstuefe

src/hotspot/share/gc/epsilon/epsilonHeap.cpp

@@ -331,9 +331,10 @@ void EpsilonHeap::print_heap_info(size_t used) const {
 }
 
 void EpsilonHeap::print_metaspace_info() const {
-  size_t reserved  = MetaspaceUtils::reserved_bytes();
-  size_t committed = MetaspaceUtils::committed_bytes();
-  size_t used      = MetaspaceUtils::used_bytes();
+  MetaspaceCombinedStats stats = MetaspaceUtils::get_combined_statistics();
+  size_t reserved  = stats.reserved();
+  size_t committed = stats.committed();
+  size_t used      = stats.used();
 
   if (reserved != 0) {
     log_info(gc, metaspace)("Metaspace: " SIZE_FORMAT "%s reserved, " SIZE_FORMAT "%s (%.2f%%) committed, "

src/hotspot/share/gc/epsilon/epsilonMonitoringSupport.cpp

@@ -111,7 +111,6 @@ void EpsilonMonitoringSupport::update_counters() {
     _heap_counters->update_all();
     _space_counters->update_all(capacity, used);
     MetaspaceCounters::update_performance_counters();
-    CompressedClassSpaceCounters::update_performance_counters();
   }
 }
 

src/hotspot/share/gc/g1/g1HeapTransition.cpp

@@ -35,6 +35,7 @@ G1HeapTransition::Data::Data(G1CollectedHeap* g1_heap) :
   _old_length(g1_heap->old_regions_count()),
   _archive_length(g1_heap->archive_regions_count()),
   _humongous_length(g1_heap->humongous_regions_count()),
+  _meta_sizes(MetaspaceUtils::get_combined_statistics()),
   _eden_length_per_node(NULL),
   _survivor_length_per_node(NULL) {
 

src/hotspot/share/gc/g1/g1HeapTransition.hpp

@@ -26,7 +26,7 @@
 #define SHARE_GC_G1_G1HEAPTRANSITION_HPP
 
 #include "gc/shared/plab.hpp"
-#include "memory/metaspace/metaspaceSizesSnapshot.hpp"
+#include "memory/metaspaceStats.hpp"
 
 class G1CollectedHeap;
 
@@ -37,7 +37,7 @@ class G1HeapTransition {
     size_t _old_length;
     size_t _archive_length;
     size_t _humongous_length;
-    const metaspace::MetaspaceSizesSnapshot _meta_sizes;
+    const MetaspaceCombinedStats _meta_sizes;
 
     // Only includes current eden regions.
     uint* _eden_length_per_node;

src/hotspot/share/gc/g1/g1MonitoringSupport.cpp

@@ -303,7 +303,6 @@ void G1MonitoringSupport::update_sizes() {
     _old_gen_counters->update_all();
 
     MetaspaceCounters::update_performance_counters();
-    CompressedClassSpaceCounters::update_performance_counters();
   }
 }
 

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

@@ -184,7 +184,6 @@ void ParallelScavengeHeap::update_counters() {
   young_gen()->update_counters();
   old_gen()->update_counters();
   MetaspaceCounters::update_performance_counters();
-  CompressedClassSpaceCounters::update_performance_counters();
 }
 
 size_t ParallelScavengeHeap::capacity() const {

src/hotspot/share/gc/shared/collectedHeap.cpp

@@ -128,25 +128,12 @@ GCHeapSummary CollectedHeap::create_heap_summary() {
 }
 
 MetaspaceSummary CollectedHeap::create_metaspace_summary() {
-  const MetaspaceSizes meta_space(
-      MetaspaceUtils::committed_bytes(),
-      MetaspaceUtils::used_bytes(),
-      MetaspaceUtils::reserved_bytes());
-  const MetaspaceSizes data_space(
-      MetaspaceUtils::committed_bytes(Metaspace::NonClassType),
-      MetaspaceUtils::used_bytes(Metaspace::NonClassType),
-      MetaspaceUtils::reserved_bytes(Metaspace::NonClassType));
-  const MetaspaceSizes class_space(
-      MetaspaceUtils::committed_bytes(Metaspace::ClassType),
-      MetaspaceUtils::used_bytes(Metaspace::ClassType),
-      MetaspaceUtils::reserved_bytes(Metaspace::ClassType));
-
   const MetaspaceChunkFreeListSummary& ms_chunk_free_list_summary =
     MetaspaceUtils::chunk_free_list_summary(Metaspace::NonClassType);
   const MetaspaceChunkFreeListSummary& class_chunk_free_list_summary =
     MetaspaceUtils::chunk_free_list_summary(Metaspace::ClassType);
-
-  return MetaspaceSummary(MetaspaceGC::capacity_until_GC(), meta_space, data_space, class_space,
+  return MetaspaceSummary(MetaspaceGC::capacity_until_GC(),
+                          MetaspaceUtils::get_combined_statistics(),
                           ms_chunk_free_list_summary, class_chunk_free_list_summary);
 }
 

src/hotspot/share/gc/shared/gcHeapSummary.hpp

@@ -26,6 +26,7 @@
 #define SHARE_GC_SHARED_GCHEAPSUMMARY_HPP
 
 #include "memory/allocation.hpp"
+#include "memory/metaspaceStats.hpp"
 #include "memory/metaspaceChunkFreeListSummary.hpp"
 
 class VirtualSpaceSummary : public StackObj {
@@ -61,21 +62,6 @@ class SpaceSummary : public StackObj {
   size_t size() const { return (uintptr_t)_end - (uintptr_t)_start; }
 };
 
-class MetaspaceSizes : public StackObj {
-  size_t _committed;
-  size_t _used;
-  size_t _reserved;
-
- public:
-  MetaspaceSizes() : _committed(0), _used(0), _reserved(0) {}
-  MetaspaceSizes(size_t committed, size_t used, size_t reserved) :
-    _committed(committed), _used(used), _reserved(reserved) {}
-
-  size_t committed() const { return _committed; }
-  size_t used() const { return _used; }
-  size_t reserved() const { return _reserved; }
-};
-
 class GCHeapSummary;
 class PSHeapSummary;
 class G1HeapSummary;
@@ -147,39 +133,29 @@ class G1HeapSummary : public GCHeapSummary {
 
 class MetaspaceSummary : public StackObj {
   size_t _capacity_until_GC;
-  MetaspaceSizes _meta_space;
-  MetaspaceSizes _data_space;
-  MetaspaceSizes _class_space;
+  MetaspaceCombinedStats _stats;
   MetaspaceChunkFreeListSummary _metaspace_chunk_free_list_summary;
   MetaspaceChunkFreeListSummary _class_chunk_free_list_summary;
 
  public:
   MetaspaceSummary() :
     _capacity_until_GC(0),
-    _meta_space(),
-    _data_space(),
-    _class_space(),
+    _stats(),
     _metaspace_chunk_free_list_summary(),
     _class_chunk_free_list_summary()
   {}
   MetaspaceSummary(size_t capacity_until_GC,
-                   const MetaspaceSizes& meta_space,
-                   const MetaspaceSizes& data_space,
-                   const MetaspaceSizes& class_space,
+                   const MetaspaceCombinedStats& stats,
                    const MetaspaceChunkFreeListSummary& metaspace_chunk_free_list_summary,
                    const MetaspaceChunkFreeListSummary& class_chunk_free_list_summary) :
     _capacity_until_GC(capacity_until_GC),
-    _meta_space(meta_space),
-    _data_space(data_space),
-    _class_space(class_space),
+    _stats(stats),
     _metaspace_chunk_free_list_summary(metaspace_chunk_free_list_summary),
     _class_chunk_free_list_summary(class_chunk_free_list_summary)
   {}
 
   size_t capacity_until_GC() const { return _capacity_until_GC; }
-  const MetaspaceSizes& meta_space() const { return _meta_space; }
-  const MetaspaceSizes& data_space() const { return _data_space; }
-  const MetaspaceSizes& class_space() const { return _class_space; }
+  const MetaspaceCombinedStats& stats() const { return _stats; }
 
   const MetaspaceChunkFreeListSummary& metaspace_chunk_free_list_summary() const {
     return _metaspace_chunk_free_list_summary;

src/hotspot/share/gc/shared/gcTraceSend.cpp

@@ -265,13 +265,11 @@ void GCTracer::send_gc_heap_summary_event(GCWhen::Type when, const GCHeapSummary
   heap_summary.accept(&visitor);
 }
 
-static JfrStructMetaspaceSizes to_struct(const MetaspaceSizes& sizes) {
+static JfrStructMetaspaceSizes to_struct(const MetaspaceStats& sizes) {
   JfrStructMetaspaceSizes meta_sizes;
-
   meta_sizes.set_committed(sizes.committed());
   meta_sizes.set_used(sizes.used());
   meta_sizes.set_reserved(sizes.reserved());
-
   return meta_sizes;
 }
 
@@ -281,9 +279,9 @@ void GCTracer::send_meta_space_summary_event(GCWhen::Type when, const MetaspaceS
     e.set_gcId(GCId::current());
     e.set_when((u1) when);
     e.set_gcThreshold(meta_space_summary.capacity_until_GC());
-    e.set_metaspace(to_struct(meta_space_summary.meta_space()));
-    e.set_dataSpace(to_struct(meta_space_summary.data_space()));
-    e.set_classSpace(to_struct(meta_space_summary.class_space()));
+    e.set_metaspace(to_struct(meta_space_summary.stats())); // total stats (class + nonclass)
+    e.set_dataSpace(to_struct(meta_space_summary.stats().non_class_space_stats())); // "dataspace" aka non-class space
+    e.set_classSpace(to_struct(meta_space_summary.stats().class_space_stats()));
     e.commit();
   }
 }

src/hotspot/share/gc/shared/genCollectedHeap.cpp

@@ -57,7 +57,6 @@
 #include "gc/shared/weakProcessor.hpp"
 #include "gc/shared/workgroup.hpp"
 #include "memory/iterator.hpp"
-#include "memory/metaspace/metaspaceSizesSnapshot.hpp"
 #include "memory/metaspaceCounters.hpp"
 #include "memory/metaspaceUtils.hpp"
 #include "memory/resourceArea.hpp"
@@ -1228,7 +1227,6 @@ void GenCollectedHeap::gc_epilogue(bool full) {
   generation_iterate(&blk, false);  // not old-to-young.
 
   MetaspaceCounters::update_performance_counters();
-  CompressedClassSpaceCounters::update_performance_counters();
 };
 
 #ifndef PRODUCT