commit 1ff876da3c8316ebe4a39e29b5db697953421511
author Carl Shapiro <cshapiro@google.com> Sun Apr 04 01:56:48 2010 -0700
committer Carl Shapiro <cshapiro@google.com> Mon Apr 05 18:03:16 2010 -0700
tree de82cc08649971081034f3bd1ee78fa3862fce3a
parent 50244069bced9a9cc5b1f42da2c5228b98c93f59

Unlock the monitor mutex before destroying it.  This resolves an issue
with simulator builds where, unlike bionic, libc checks whether a
mutex is in the unlocked state before a destroy and returns EBUSY if
the check fails.  Until similar checks are added to bionic we now do a
trylock, unlock sequence in the assert statements so as not to confuse
the simulator build.

Change-Id: Iafa3c52edddb17b9a8a2c648c5fd3a58d5d77988

vm/Sync.c

diff --git a/vm/Sync.c b/vm/Sync.c
index c00a630..c9624d0 100644
--- a/vm/Sync.c
+++ b/vm/Sync.c
@@ -310,7 +310,8 @@
      * the object, in which case we've got some bad
      * native code somewhere.
      */
-    assert(dvmTryLockMutex(&mon->lock) == 0);
+    assert(pthread_mutex_trylock(&mon->lock) == 0);
+    assert(pthread_mutex_unlock(&mon->lock) == 0);
     dvmDestroyMutex(&mon->lock);
 #ifdef WITH_DEADLOCK_PREDICTION
     expandObjClear(&mon->historyChildren);