comparison src/share/vm/services/management.cpp @ 1757:1c63587d925b

6980206: G1: assert(has_undefined_max_size, "Undefined max size"); Summary: An assert in the management.cpp is too strong and assumes the max size is always defined on memory pools, even when we don't need to use it. Reviewed-by: mchung, johnc
author tonyp
date Fri, 27 Aug 2010 13:34:14 -0400
parents f6f3eef8a521
children f95d63e2154a
comparison
equal deleted inserted replaced
1755:8e5955ddf8e4 1757:1c63587d925b
783 total_max += u.max_size(); 783 total_max += u.max_size();
784 } 784 }
785 } 785 }
786 } 786 }
787 787
788 // In our current implementation, all pools should have 788 // In our current implementation, we make sure that all non-heap
789 // defined init and max size 789 // pools have defined init and max sizes. Heap pools do not matter,
790 assert(!has_undefined_init_size, "Undefined init size"); 790 // as we never use total_init and total_max for them.
791 assert(!has_undefined_max_size, "Undefined max size"); 791 assert(heap || !has_undefined_init_size, "Undefined init size");
792 assert(heap || !has_undefined_max_size, "Undefined max size");
792 793
793 MemoryUsage usage((heap ? InitialHeapSize : total_init), 794 MemoryUsage usage((heap ? InitialHeapSize : total_init),
794 total_used, 795 total_used,
795 total_committed, 796 total_committed,
796 (heap ? Universe::heap()->max_capacity() : total_max)); 797 (heap ? Universe::heap()->max_capacity() : total_max));