src/share/vm/gc/shared/collectorPolicy.hpp

rev 10911 : 8154467: Cleanup initialization of GCPolicyCounters
Reviewed-by:

@@ -56,12 +56,10 @@
 class GCPolicyCounters;
 class MarkSweepPolicy;
 
 class CollectorPolicy : public CHeapObj<mtGC> {
  protected:
-  GCPolicyCounters* _gc_policy_counters;
-
   virtual void initialize_alignments() = 0;
   virtual void initialize_flags();
   virtual void initialize_size_info();
 
   DEBUG_ONLY(virtual void assert_flags();)

@@ -147,19 +145,10 @@
 
   MetaWord* satisfy_failed_metadata_allocation(ClassLoaderData* loader_data,
                                                size_t size,
                                                Metaspace::MetadataType mdtype);
 
-  // Performance Counter support
-  GCPolicyCounters* counters()     { return _gc_policy_counters; }
-
-  // Create the jstat counters for the GC policy.  By default, policy's
-  // don't have associated counters, and we complain if this is invoked.
-  virtual void initialize_gc_policy_counters() {
-    ShouldNotReachHere();
-  }
-
   // Do any updates required to global flags that are due to heap initialization
   // changes
   virtual void post_heap_initialize() = 0;
 };
 

@@ -195,10 +184,12 @@
   size_t _gen_alignment;
 
   GenerationSpec* _young_gen_spec;
   GenerationSpec* _old_gen_spec;
 
+  GCPolicyCounters* _gc_policy_counters;
+
   // Return true if an allocation should be attempted in the older generation
   // if it fails in the younger generation.  Return false, otherwise.
   virtual bool should_try_older_generation_allocation(size_t word_size) const;
 
   void initialize_flags();

@@ -241,10 +232,19 @@
   GenerationSpec* old_gen_spec() const {
     assert(_old_gen_spec != NULL, "_old_gen_spec should have been initialized");
     return _old_gen_spec;
   }
 
+  // Performance Counter support
+  GCPolicyCounters* counters()     { return _gc_policy_counters; }
+
+  // Create the jstat counters for the GC policy.  By default, policies
+  // don't have associated counters, and we complain if this is invoked.
+  virtual void initialize_gc_policy_counters() {
+    ShouldNotReachHere();
+  }
+
   virtual GenCollectorPolicy* as_generation_policy() { return this; }
 
   virtual void initialize_generations() { };
 
   virtual void initialize_all() {