EVALUATION

http://hg.openjdk.java.net/jdk7/hotspot/hotspot/rev/8f6f7587d292
                                     

EVALUATION

http://hg.openjdk.java.net/hsx/hsx19/baseline/rev/53adfc40121e
                                     

EVALUATION

http://hg.openjdk.java.net/jdk7/hotspot-gc/hotspot/rev/8f6f7587d292
                                     

EVALUATION

The fix for "6423256 GC stacks should use a better data structure" included some debugging code that was not intended to be included in the product--a simple "#if 0 ... #endif" around the fatal error deadlock handling code in WatcherThread::run().  That change should be reverted.
                                     

SUGGESTED FIX

diff --git a/src/share/vm/runtime/thread.cpp b/src/share/vm/runtime/thread.cpp
--- a/src/share/vm/runtime/thread.cpp
+++ b/src/share/vm/runtime/thread.cpp
@@ -1073,7 +1073,6 @@
       }
     }
 
-#if 0
     if (is_error_reported()) {
       // A fatal error has happened, the error handler(VMError::report_and_die)
       // should abort JVM after creating an error log file. However in some
@@ -1101,7 +1100,6 @@
         os::sleep(this, 5 * 1000, false);
       }
     }
-#endif // #if 0
 
     PeriodicTask::real_time_tick(time_to_wait);