diff src/share/vm/services/memRecorder.cpp @ 6231:7e5976e66c62

7182543: NMT ON: Aggregate a few NMT related bugs Summary: 1) Fixed MemTrackWorker::generations_in_used() calculation 2) Ensured NMT not to leak memory recorders after shutdown 3) Used ThreadCritical to block safepoint safe threads Reviewed-by: acorn, coleenp, dholmes, kvn
author zgu
date Thu, 19 Jul 2012 09:05:42 -0400
parents d2a62e0f25eb
children 716c64bda5ba
line wrap: on
line diff
--- a/src/share/vm/services/memRecorder.cpp	Wed Jul 04 15:55:45 2012 -0400
+++ b/src/share/vm/services/memRecorder.cpp	Thu Jul 19 09:05:42 2012 -0400
@@ -45,11 +45,11 @@
 }
 
 
-debug_only(volatile jint MemRecorder::_instance_count = 0;)
+volatile jint MemRecorder::_instance_count = 0;
 
 MemRecorder::MemRecorder() {
   assert(MemTracker::is_on(), "Native memory tracking is off");
-  debug_only(Atomic::inc(&_instance_count);)
+  Atomic::inc(&_instance_count);
   debug_only(set_generation();)
 
   if (MemTracker::track_callsite()) {
@@ -83,9 +83,7 @@
     delete _next;
   }
 
-#ifdef ASSERT
   Atomic::dec(&_instance_count);
-#endif
 }
 
 // Sorting order: