--- old/src/hotspot/share/services/memoryService.cpp 2017-11-28 11:57:14.420652030 +0100 +++ new/src/hotspot/share/services/memoryService.cpp 2017-11-28 11:57:14.160655047 +0100 @@ -76,15 +76,15 @@ heap->gc_threads_do(&gctcc); int count = gctcc.count(); - GrowableArray gc_mem_mgrs = heap->memory_managers(); - for (int i = 0; i < gc_mem_mgrs.length(); i++) { - GCMemoryManager* gc_mgr = gc_mem_mgrs.at(i); + GrowableArray gc_memory_managers = heap->memory_managers(); + for (int i = 0; i < gc_memory_managers.length(); i++) { + GCMemoryManager* gc_manager = gc_memory_managers.at(i); if (count > 0) { - gc_mgr->set_num_gc_threads(count); + gc_manager->set_num_gc_threads(count); } - gc_mgr->initialize_gc_stat_info(); - _managers_list->append(gc_mgr); + gc_manager->initialize_gc_stat_info(); + _managers_list->append(gc_manager); } } @@ -162,11 +162,11 @@ } } -void MemoryService::gc_begin(GCMemoryManager* mgr, bool recordGCBeginTime, +void MemoryService::gc_begin(GCMemoryManager* manager, bool recordGCBeginTime, bool recordAccumulatedGCTime, bool recordPreGCUsage, bool recordPeakUsage) { - mgr->gc_begin(recordGCBeginTime, recordPreGCUsage, recordAccumulatedGCTime); + manager->gc_begin(recordGCBeginTime, recordPreGCUsage, recordAccumulatedGCTime); // Track the peak memory usage when GC begins if (recordPeakUsage) { @@ -177,13 +177,13 @@ } } -void MemoryService::gc_end(GCMemoryManager* mgr, bool recordPostGCUsage, +void MemoryService::gc_end(GCMemoryManager* manager, bool recordPostGCUsage, bool recordAccumulatedGCTime, bool recordGCEndTime, bool countCollection, GCCause::Cause cause) { // register the GC end statistics and memory usage - mgr->gc_end(recordPostGCUsage, recordAccumulatedGCTime, recordGCEndTime, - countCollection, cause); + manager->gc_end(recordPostGCUsage, recordAccumulatedGCTime, recordGCEndTime, + countCollection, cause); } void MemoryService::oops_do(OopClosure* f) { @@ -234,7 +234,7 @@ return obj; } -TraceMemoryManagerStats::TraceMemoryManagerStats(GCMemoryManager* gc_mem_mgr, +TraceMemoryManagerStats::TraceMemoryManagerStats(GCMemoryManager* gc_memory_manager, GCCause::Cause cause, bool recordGCBeginTime, bool recordPreGCUsage, @@ -243,14 +243,14 @@ bool recordAccumulatedGCTime, bool recordGCEndTime, bool countCollection) { - initialize(gc_mem_mgr, cause, recordGCBeginTime, recordPreGCUsage, recordPeakUsage, + initialize(gc_memory_manager, cause, recordGCBeginTime, recordPreGCUsage, recordPeakUsage, recordPostGCUsage, recordAccumulatedGCTime, recordGCEndTime, countCollection); } // for a subclass to create then initialize an instance before invoking // the MemoryService -void TraceMemoryManagerStats::initialize(GCMemoryManager* gc_mem_mgr, +void TraceMemoryManagerStats::initialize(GCMemoryManager* gc_memory_manager, GCCause::Cause cause, bool recordGCBeginTime, bool recordPreGCUsage, @@ -259,7 +259,7 @@ bool recordAccumulatedGCTime, bool recordGCEndTime, bool countCollection) { - _gc_mem_mgr = gc_mem_mgr; + _gc_memory_manager = gc_memory_manager; _recordGCBeginTime = recordGCBeginTime; _recordPreGCUsage = recordPreGCUsage; _recordPeakUsage = recordPeakUsage; @@ -269,11 +269,11 @@ _countCollection = countCollection; _cause = cause; - MemoryService::gc_begin(_gc_mem_mgr, _recordGCBeginTime, _recordAccumulatedGCTime, + MemoryService::gc_begin(_gc_memory_manager, _recordGCBeginTime, _recordAccumulatedGCTime, _recordPreGCUsage, _recordPeakUsage); } TraceMemoryManagerStats::~TraceMemoryManagerStats() { - MemoryService::gc_end(_gc_mem_mgr, _recordPostGCUsage, _recordAccumulatedGCTime, + MemoryService::gc_end(_gc_memory_manager, _recordPostGCUsage, _recordAccumulatedGCTime, _recordGCEndTime, _countCollection, _cause); }