]> granicus.if.org Git - graphviz/commitdiff
remove vmtx mutex no-ops
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 17 Oct 2020 01:41:22 +0000 (18:41 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 24 Oct 2020 02:16:09 +0000 (19:16 -0700)
lib/sfio/sfclose.c
lib/sfio/sfexit.c
lib/sfio/sfextern.c
lib/sfio/sfmode.c
lib/sfio/vthread.h

index dd8e00759e39525488eb8d7e607f8a6d12dd37b4..f424656298dd12c28d1835a66c65c067033e389c 100644 (file)
@@ -121,9 +121,7 @@ int sfclose(Sfio_t * f)
 
     /* destroy the mutex */
     if (f->mutex) {
-       vtmtxclrlock(f->mutex);
        if (f != sfstdin && f != sfstdout && f != sfstderr) {
-           vtmtxclose(f->mutex);
            f->mutex = NIL(Vtmutex_t *);
        }
     }
index 48e75f5765a22ecda2180f890c08df1ebec82b6a..de63d5b52310e665986214f4cf91b7ded2ae23de 100644 (file)
@@ -44,8 +44,6 @@ waitpid(int pid, int *status, int options)
     if (options != 0)
        return -1;
 
-    vtmtxlock(_Sfmutex);
-
     for (w = Wait, last = NIL(Waitpid_t *); w; last = w, w = w->next) {
        if (pid > 0 && pid != w->pid)
            continue;
@@ -59,7 +57,6 @@ waitpid(int pid, int *status, int options)
        pid = w->pid;
        free(w);
 
-       vtmtxunlock(_Sfmutex);
        return pid;
     }
 
@@ -68,7 +65,6 @@ waitpid(int pid, int *status, int options)
            if (status)
                *status = ps;
 
-           vtmtxunlock(_Sfmutex);
            return pid;
        }
 
@@ -81,7 +77,6 @@ waitpid(int pid, int *status, int options)
        Wait = w;
     }
 
-    vtmtxunlock(_Sfmutex);
     return -1;
 }
 
index a698801d4ead64fa2eb0c9fca8141982ed955c61..830e22159331fd8da7c542593bbb6bc5a05fe60e 100644 (file)
@@ -22,11 +22,6 @@ static Vtmutex_t Sfmutex;
 static Vtonce_t Sfonce = VTONCE_INITDATA;
 static void _sfoncef(void)
 {
-    vtmtxopen(_Sfmutex, VT_INIT);
-    vtmtxopen(&_Sfpool.mutex, VT_INIT);
-    vtmtxopen(sfstdin->mutex, VT_INIT);
-    vtmtxopen(sfstdout->mutex, VT_INIT);
-    vtmtxopen(sfstderr->mutex, VT_INIT);
     _Sfdone = 1;
 }
 
index bca7187ac26bd23830e09ad1012976f7ee8c6417..b092b249a5eb05669296576a2c69ce674581134b 100644 (file)
@@ -194,12 +194,10 @@ int _sfpopen(Sfio_t * f, int fd, int pid, int stdio)
     if (p->sigp) {
        Sfsignal_f handler;
 
-       vtmtxlock(_Sfmutex);
        if ((handler = signal(SIGPIPE, ignoresig)) != SIG_DFL &&
            handler != ignoresig)
            signal(SIGPIPE, handler);   /* honor user handler */
        _Sfsigp += 1;
-       vtmtxunlock(_Sfmutex);
     }
 #endif
 
@@ -236,7 +234,6 @@ int _sfpclose(Sfio_t * f)
            status = -1;
 
 #ifdef SIGPIPE
-       vtmtxlock(_Sfmutex);
        if (p->sigp && (_Sfsigp -= 1) <= 0) {
            Sfsignal_f handler;
            if ((handler = signal(SIGPIPE, SIG_DFL)) != SIG_DFL &&
@@ -244,7 +241,6 @@ int _sfpclose(Sfio_t * f)
                signal(SIGPIPE, handler);       /* honor user handler */
            _Sfsigp = 0;
        }
-       vtmtxunlock(_Sfmutex);
 #endif
     }
 
index d8cc18f09796ac52b91752cb68f882e1a196b18a..897d150435295604b54099fb639e8598fa721e1f 100644 (file)
@@ -61,13 +61,6 @@ extern "C" {
 #define EPERM                  1
 #endif
 
-    extern Vtmutex_t *vtmtxopen(Vtmutex_t *, int);
-    extern int vtmtxclose(Vtmutex_t *);
-    extern int vtmtxlock(Vtmutex_t *);
-    extern int vtmtxtrylock(Vtmutex_t *);
-    extern int vtmtxunlock(Vtmutex_t *);
-    extern int vtmtxclrlock(Vtmutex_t *);
-
     extern void *vtstatus(Vthread_t *);
     extern int vterror(Vthread_t *);
     extern int vtmtxerror(Vtmutex_t *);
@@ -89,13 +82,6 @@ extern "C" {
 
 #define VTONCE_INITDATA                {0}
 
-#define vtmtxopen(mtx,flgs)    (void)((Vtmutex_t*)0)
-#define vtmtxclose(mtx)                (void)(-1)
-#define vtmtxlock(mtx)         (void)(-1)
-#define vtmtxtrylock(mtx)      (-1)
-#define vtmtxunlock(mtx)       (void)(-1)
-#define vtmtxclrlock(mtx)      (void)(-1)
-
 #define vtstatus(vt)           ((void*)0)
 #define vterror(vt)            (0)
 #define vtmtxerror(mtx)                (0)