commit cd74c4b3a6893c876c6e03fd99a1264249653d80
author Brian Carlstrom <bdc@google.com> Mon Jan 23 13:21:00 2012 -0800
committer Brian Carlstrom <bdc@google.com> Wed Jan 25 11:07:15 2012 -0800
tree fd59dad11cb63ab6ff40be13e1572e1c2becab67
parent 844f9a072454eb9dca1022299b6bf99ef4294008

Fix thread hang

- Primary problem was ScopedThreadListLock was releasing heap lock in constructor instead of destructor
- Secondary problem was ScopedThreadListLock should not be used with Mutex::Wait
- Added Thread.getStackTrace case to ThreadStress that reproduces YouTube problem
- Added Mutex::GetDepth and related methods that were useful in diagnoising this issue

Change-Id: I1bdc7245e9b411378b98f4dcf498ad66eb96366d

test/ThreadStress/ThreadStress.java

diff --git a/test/ThreadStress/ThreadStress.java b/test/ThreadStress/ThreadStress.java
index 1f8fb2d..5389c20 100644
--- a/test/ThreadStress/ThreadStress.java
+++ b/test/ThreadStress/ThreadStress.java
@@ -31,7 +31,8 @@
     enum Operation {
         OOM(1),
         SIGQUIT(19),
-        ALLOC(80),
+        ALLOC(60),
+        STACKTRACE(20),
         EXIT(50),
         WAIT(50);
 
@@ -231,6 +232,10 @@
                         }
                         break;
                     }
+                    case STACKTRACE: {
+                        Thread.currentThread().getStackTrace();
+                        break;
+                    }
                     default: {
                         throw new AssertionError(operation.toString());
                     }