6908215: G1: SEGV with G1PolicyVerbose=2 debug flag

Change CollectionSetChooser::printSortedHeapRegions to handle null entries in _markedRegions growable array.

Reviewed-by: jmasa, tonyp, iveresov
This commit is contained in:
John Cuthbertson 2009-12-09 23:51:38 -08:00
parent d090b4fe29
commit 93d5e80a7e

View file

@ -351,9 +351,16 @@ void
CollectionSetChooser::printSortedHeapRegions() { CollectionSetChooser::printSortedHeapRegions() {
gclog_or_tty->print_cr("Printing %d Heap Regions sorted by amount of known garbage", gclog_or_tty->print_cr("Printing %d Heap Regions sorted by amount of known garbage",
_numMarkedRegions); _numMarkedRegions);
DEBUG_ONLY(int marked_count = 0;)
for (int i = 0; i < _markedRegions.length(); i++) { for (int i = 0; i < _markedRegions.length(); i++) {
printHeapRegion(_markedRegions.at(i)); HeapRegion* r = _markedRegions.at(i);
if (r != NULL) {
printHeapRegion(r);
DEBUG_ONLY(marked_count++;)
} }
}
assert(marked_count == _numMarkedRegions, "must be");
gclog_or_tty->print_cr("Done sorted heap region print"); gclog_or_tty->print_cr("Done sorted heap region print");
} }