commit 8ccc2be4b8876d76c1396f608e094dd435fd6a9e
author sewardj <sewardj@a5019735-40e9-0310-863c-91ae7b9d1cf9> Fri May 10 20:26:37 2002 +0000
committer sewardj <sewardj@a5019735-40e9-0310-863c-91ae7b9d1cf9> Fri May 10 20:26:37 2002 +0000
tree de5c059c022296bb82e24066a57b016e5ca0f63e
parent 71889e6d05441db281ba5e907ab653d74547decd

Fix confusing bug in printing of trace info for mutex_{try}lock.
Add a couple of other handy debug printings.


git-svn-id: svn://svn.valgrind.org/valgrind/trunk@254 a5019735-40e9-0310-863c-91ae7b9d1cf9

coregrind/vg_scheduler.c

diff --git a/coregrind/vg_scheduler.c b/coregrind/vg_scheduler.c
index 0048b0e..4109187 100644
--- a/coregrind/vg_scheduler.c
+++ b/coregrind/vg_scheduler.c
@@ -144,9 +144,9 @@
 static void scheduler_sanity ( void );
 
 static void do_pthread_mutex_unlock ( ThreadId, 
-                                      void* /* pthread_cond_t* */ );
+                                      void* /* pthread_mutex_t* */ );
 static void do_pthread_mutex_lock ( ThreadId, Bool, 
-                                    void* /* pthread_cond_t* */ );
+                                    void* /* pthread_mutex_t* */ );
 
 static void do_pthread_getspecific ( ThreadId,
                                      UInt /* pthread_key_t */ );
@@ -623,6 +623,7 @@
    UInt*        arg    = (UInt*)(tst->m_eax);
    UInt         req_no = arg[0];
 
+   /* VG_(printf)("req no = 0x%x\n", req_no); */
    switch (req_no) {
       case VG_USERREQ__MALLOC:
          SIMPLE_RETURN(
@@ -1324,7 +1325,10 @@
          }
 
          if (trc == VG_TRC_EBP_JMP_CLIENTREQ) {
-            Bool done = maybe_do_trivial_clientreq(tid);
+            Bool done; 
+            /* VG_(printf)("request 0x%x\n", 
+                           *(UInt*)(vg_threads[tid].m_eax)); */
+            done = maybe_do_trivial_clientreq(tid);
             if (done) {
                /* The request is done.  We try and continue with the
                   same thread if still runnable.  If not, go back to
@@ -1891,8 +1895,8 @@
 {
    Char  msg_buf[100];
    Char* caller
-      = is_trylock ? "pthread_mutex_lock   "
-                   : "pthread_mutex_trylock";
+      = is_trylock ? "pthread_mutex_trylock"
+                   : "pthread_mutex_lock   ";
 
    pthread_mutex_t* mutex = (pthread_mutex_t*)mutexV;
 

vg_scheduler.c

diff --git a/vg_scheduler.c b/vg_scheduler.c
index 0048b0e..4109187 100644
--- a/vg_scheduler.c
+++ b/vg_scheduler.c
@@ -144,9 +144,9 @@
 static void scheduler_sanity ( void );
 
 static void do_pthread_mutex_unlock ( ThreadId, 
-                                      void* /* pthread_cond_t* */ );
+                                      void* /* pthread_mutex_t* */ );
 static void do_pthread_mutex_lock ( ThreadId, Bool, 
-                                    void* /* pthread_cond_t* */ );
+                                    void* /* pthread_mutex_t* */ );
 
 static void do_pthread_getspecific ( ThreadId,
                                      UInt /* pthread_key_t */ );
@@ -623,6 +623,7 @@
    UInt*        arg    = (UInt*)(tst->m_eax);
    UInt         req_no = arg[0];
 
+   /* VG_(printf)("req no = 0x%x\n", req_no); */
    switch (req_no) {
       case VG_USERREQ__MALLOC:
          SIMPLE_RETURN(
@@ -1324,7 +1325,10 @@
          }
 
          if (trc == VG_TRC_EBP_JMP_CLIENTREQ) {
-            Bool done = maybe_do_trivial_clientreq(tid);
+            Bool done; 
+            /* VG_(printf)("request 0x%x\n", 
+                           *(UInt*)(vg_threads[tid].m_eax)); */
+            done = maybe_do_trivial_clientreq(tid);
             if (done) {
                /* The request is done.  We try and continue with the
                   same thread if still runnable.  If not, go back to
@@ -1891,8 +1895,8 @@
 {
    Char  msg_buf[100];
    Char* caller
-      = is_trylock ? "pthread_mutex_lock   "
-                   : "pthread_mutex_trylock";
+      = is_trylock ? "pthread_mutex_trylock"
+                   : "pthread_mutex_lock   ";
 
    pthread_mutex_t* mutex = (pthread_mutex_t*)mutexV;