stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

6975964: G1: print out a more descriptive message for evacuation failure when +PrintGCDetails is set

Browse Source 
We're renaming "evacuation failure" to "to-space overflow". I'm also piggy-backing a small additional change which removes the "Mark closure took..." output.

Reviewed-by: ysr, johnc
This commit is contained in:
Antonios Printezis 2010-08-17 14:40:00 -04:00
parent 7125621712
commit ce2883fbbd
2 changed files with 2 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
hotspot/src/share/vm/gc_implementation/g1/concurrentMark.cpp
View File

@ -2586,9 +2586,6 @@ void ConcurrentMark::complete_marking_in_collection_set() {
double end_time = os::elapsedTime(); double end_time = os::elapsedTime();
double elapsed_time_ms = (end_time - start) * 1000.0; double elapsed_time_ms = (end_time - start) * 1000.0;
g1h->g1_policy()->record_mark_closure_time(elapsed_time_ms); g1h->g1_policy()->record_mark_closure_time(elapsed_time_ms);
if (PrintGCDetails) {
gclog_or_tty->print_cr("Mark closure took %5.2f ms.", elapsed_time_ms);
}
ClearMarksInHRClosure clr(nextMarkBitMap()); ClearMarksInHRClosure clr(nextMarkBitMap());
g1h->collection_set_iterate(&clr); g1h->collection_set_iterate(&clr);

4
hotspot/src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp
View File

@ -3608,7 +3608,7 @@ void G1CollectedHeap::handle_evacuation_failure_common(oop old, markOop m) {
if (!r->evacuation_failed()) { if (!r->evacuation_failed()) {
r->set_evacuation_failed(true); r->set_evacuation_failed(true);
if (G1PrintHeapRegions) { if (G1PrintHeapRegions) {
gclog_or_tty->print("evacuation failed in heap region "PTR_FORMAT" " gclog_or_tty->print("overflow in heap region "PTR_FORMAT" "
"["PTR_FORMAT","PTR_FORMAT")\n", "["PTR_FORMAT","PTR_FORMAT")\n",
r, r->bottom(), r->end()); r, r->bottom(), r->end());
} }
@ -4321,7 +4321,7 @@ void G1CollectedHeap::evacuate_collection_set() {
if (evacuation_failed()) { if (evacuation_failed()) {
remove_self_forwarding_pointers(); remove_self_forwarding_pointers();
if (PrintGCDetails) { if (PrintGCDetails) {
gclog_or_tty->print(" (evacuation failed)"); gclog_or_tty->print(" (to-space overflow)");
} else if (PrintGC) { } else if (PrintGC) {
gclog_or_tty->print("--"); gclog_or_tty->print("--");
} }