]> Zhao Yanbai Git Server - minix.git/commitdiff
Experimental pthread compatibility library
authorLionel Sambuc <lionel@minix3.org>
Tue, 14 Jan 2014 12:48:43 +0000 (13:48 +0100)
committerLionel Sambuc <lionel@minix3.org>
Mon, 28 Jul 2014 15:05:08 +0000 (17:05 +0200)
This patch adds pthread compatibility by using libmthread.

To use this with a program using pthreads, you have to replace
  #include <pthread>
with
  #define _MTHREADIFY_PTHREADS
  #include <minix/mthreads>

This also changes the initialization function to be a constructor, which
is implicitly called before the call to main. This allows for
conformance with pthreads, while not paying a high price by checking on
each mthread_* call whether the library has been initialized or not.

As mthread_init is now a constructor, it also has been set as static, and
relevent calls removed from programs using it.

Change-Id: I2aa375db557958d2bee9a70d285aabb990c88f00

21 files changed:
distrib/sets/lists/minix/mi
include/minix/mthread.h
lib/Makefile
lib/libblockdriver/driver_mt.c
lib/libmthread/Makefile
lib/libmthread/allocate.c
lib/libmthread/attribute.c
lib/libmthread/condition.c
lib/libmthread/event.c
lib/libmthread/key.c
lib/libmthread/mutex.c
lib/libmthread/proto.h
lib/libmthread/pthread_compat.c [new file with mode: 0644]
lib/libmthread/rwlock.c
lib/libmthread/scheduler.c
lib/libmthread/shlib_version [new file with mode: 0644]
servers/vfs/threads.h
servers/vfs/worker.c
share/mk/bsd.lib.mk
share/mk/bsd.prog.mk
test/test59.c

index b07ba24fa0a8331d6fc70ed0fedbbc0a2e4c8597..96d26bc0322bd3ddf84173692ce0794417bc2999 100644 (file)
 ./usr/lib/libm.so                      minix-sys
 ./usr/lib/libmthread.a                 minix-sys
 ./usr/lib/libmthread_pic.a             minix-sys
+./usr/lib/libmthread.so.0.0            minix-sys
+./usr/lib/libmthread.so.0              minix-sys
+./usr/lib/libmthread.so                        minix-sys
 ./usr/lib/libnetdriver.a               minix-sys
 ./usr/lib/libnetdriver_pic.a           minix-sys
 ./usr/lib/libnetsock.a                 minix-sys
index 54281adc8b3887d90bd0a910ee2e3eef83a6179d..bb4ba56d62e74c8471ff1b6af59bf281a95da1b8 100644 (file)
@@ -71,6 +71,7 @@ typedef struct {
 #define MTHREAD_STACK_MIN MINSIGSTKSZ
 #define MTHREAD_KEYS_MAX 128
 
+__BEGIN_DECLS
 /* allocate.c */
 int mthread_create(mthread_thread_t *thread, mthread_attr_t *tattr, void
        *(*proc)(void *), void *arg);
@@ -138,8 +139,101 @@ int mthread_rwlock_wrlock(mthread_rwlock_t *rwlock);
 int mthread_rwlock_unlock(mthread_rwlock_t *rwlock);
 
 /* schedule.c */
-void mthread_init(void);
 int mthread_yield(void);
 void mthread_yield_all(void);
+__END_DECLS
+
+#if defined(_MTHREADIFY_PTHREADS)
+typedef mthread_thread_t pthread_t;
+typedef mthread_once_t pthread_once_t;
+typedef mthread_key_t pthread_key_t;
+typedef mthread_cond_t pthread_cond_t;
+typedef mthread_mutex_t pthread_mutex_t;
+typedef mthread_condattr_t pthread_condattr_t;
+typedef mthread_mutexattr_t pthread_mutexattr_t;
+typedef mthread_attr_t pthread_attr_t;
+typedef mthread_event_t pthread_event_t;
+typedef mthread_rwlock_t pthread_rwlock_t;
+
+/* LSC: No equivalent, so void* for now. */
+typedef void *pthread_rwlockattr_t;
+
+#define PTHREAD_ONCE_INIT 0
+#define PTHREAD_MUTEX_INITIALIZER ((pthread_mutex_t) -1)
+#define PTHREAD_COND_INITIALIZER ((pthread_cond_t) -1)
+
+__BEGIN_DECLS
+/* allocate.c */
+int pthread_create(pthread_t *thread, pthread_attr_t *tattr, void
+       *(*proc)(void *), void *arg);
+int pthread_detach(pthread_t thread);
+int pthread_equal(pthread_t l, pthread_t r);
+void pthread_exit(void *value);
+int pthread_join(pthread_t thread, void **value);
+int pthread_once(pthread_once_t *once, void (*proc)(void));
+pthread_t pthread_self(void);
+
+/* attribute.c */
+int pthread_attr_destroy(pthread_attr_t *tattr);
+int pthread_attr_getdetachstate(pthread_attr_t *tattr, int
+       *detachstate);
+int pthread_attr_getstack(pthread_attr_t *tattr, void **stackaddr,
+       size_t *stacksize);
+int pthread_attr_getstacksize(pthread_attr_t *tattr, size_t *stacksize);
+int pthread_attr_init(pthread_attr_t *tattr);
+int pthread_attr_setdetachstate(pthread_attr_t *tattr, int detachstate);
+int pthread_attr_setstack(pthread_attr_t *tattr, void *stackaddr, size_t
+       stacksize);
+int pthread_attr_setstacksize(pthread_attr_t *tattr, size_t stacksize);
+
+/* condition.c */
+int pthread_cond_broadcast(pthread_cond_t *cond);
+int pthread_cond_destroy(pthread_cond_t *cond);
+int pthread_cond_init(pthread_cond_t *cond, pthread_condattr_t *cattr);
+int pthread_cond_signal(pthread_cond_t *cond);
+int pthread_cond_wait(pthread_cond_t *cond, pthread_mutex_t *mutex);
+
+/* key.c */
+int pthread_key_create(pthread_key_t *key, void (*destructor)(void *));
+int pthread_key_delete(pthread_key_t key);
+void *pthread_getspecific(pthread_key_t key);
+int pthread_setspecific(pthread_key_t key, void *value);
+
+/* mutex.c */
+int pthread_mutex_destroy(pthread_mutex_t *mutex);
+int pthread_mutex_init(pthread_mutex_t *mutex, pthread_mutexattr_t
+       *mattr);
+int pthread_mutex_lock(pthread_mutex_t *mutex);
+int pthread_mutex_trylock(pthread_mutex_t *mutex);
+int pthread_mutex_unlock(pthread_mutex_t *mutex);
+
+/* event.c */
+int pthread_event_destroy(pthread_event_t *event);
+int pthread_event_init(pthread_event_t *event);
+int pthread_event_wait(pthread_event_t *event);
+int pthread_event_fire(pthread_event_t *event);
+int pthread_event_fire_all(pthread_event_t *event);
+
+/* rwlock.c */
+int pthread_rwlock_destroy(pthread_rwlock_t *rwlock);
+int pthread_rwlock_init(pthread_rwlock_t *rwlock,
+       pthread_rwlockattr_t *UNUSED(attr));
+int pthread_rwlock_rdlock(pthread_rwlock_t *rwlock);
+int pthread_rwlock_wrlock(pthread_rwlock_t *rwlock);
+int pthread_rwlock_unlock(pthread_rwlock_t *rwlock);
+
+/* schedule.c */
+int pthread_yield(void);
+int sched_yield(void);
+void pthread_yield_all(void);
+
+/* LSC: FIXME: Maybe we should really do something with those... */
+#define pthread_mutexattr_init(u) (0)
+#define pthread_mutexattr_destroy(u) (0)
+
+#define PTHREAD_MUTEX_RECURSIVE 0
+#define pthread_mutexattr_settype(x, y) (EINVAL)
+__END_DECLS
 
+#endif /* defined(_MTHREADIFY_PTHREADS) */
 #endif
index 19fc36bdb49e5aa9ae52e19635070f2567703e23..ef93459b48a2667935ad2ad3366a55e311c21f42 100644 (file)
@@ -19,11 +19,14 @@ SUBDIR+=    ../external/gpl3/gcc/lib/libgcc .WAIT
 . endif
 .endif
 
-#LSC MINIX libc depends on libminlib because of minix/malloc-debug.c
+#LSC MINIX libc depends on
+# - libminlib because of minix/malloc-debug.c
+# - libmthread because of sys/lib/libunwind
 SUBDIR+=       libminlib
 SUBDIR+=       .WAIT
 
 SUBDIR+=       libsys
+SUBDIR+=       libmthread
 SUBDIR+=       .WAIT
 
 SUBDIR+=       libc
@@ -43,7 +46,6 @@ SUBDIR+=      libasyn \
                libinputdriver \
                libminc \
                libminixfs \
-               libmthread \
                libnetdriver \
                libsffs \
                libtimers \
index 6a8a5ef67e57c5c16a27f264d9b14990547203d5..fb1df760e9e238722ea8446e656deba8cabd2891 100644 (file)
@@ -343,8 +343,6 @@ static void master_init(struct blockdriver *bdp)
   assert(bdp != NULL);
   assert(bdp->bdr_device != NULL);
 
-  mthread_init();
-
   bdtab = bdp;
 
   /* Initialize device-specific data structures. */
index f5449c433bda7f904ee7225dd3f73c11df83ec22..909782ca2e7e46a81063b13e5e8d3970f52da940 100644 (file)
@@ -5,14 +5,15 @@ LIB=  mthread
 SRCS=  \
        allocate.c \
        attribute.c \
-       mutex.c \
+       condition.c \
        event.c \
-       rwlock.c \
+       key.c \
        misc.c \
+       mutex.c \
+       pthread_compat.c \
        queue.c \
-       condition.c \
+       rwlock.c \
        scheduler.c \
-       key.c
 
 WARNS?=        4
 
index 4a6945922e60416d0463b13782930698ac6e7247..776525414165776b8ec73cc450db616355260b47 100644 (file)
@@ -36,7 +36,6 @@ mthread_thread_t l;
 mthread_thread_t r;
 {
 /* Compare two thread ids */
-  MTHREAD_CHECK_INIT();        /* Make sure mthreads is initialized */
 
   return(l == r);
 }
@@ -54,8 +53,6 @@ void *arg;
 /* Register procedure proc for execution in a thread. */
   mthread_thread_t thread;
 
-  MTHREAD_CHECK_INIT();        /* Make sure mthreads is initialized */
-
   if (proc == NULL)
        return(EINVAL);
 
@@ -88,7 +85,6 @@ mthread_thread_t detach;
  * this thread are automatically freed.
  */
   mthread_tcb_t *tcb;
-  MTHREAD_CHECK_INIT();        /* Make sure libmthread is initialized */
 
   if (!isokthreadid(detach)) 
        return(ESRCH);
@@ -116,8 +112,6 @@ void *value;
 /* Make a thread stop running and store the result value. */
   mthread_tcb_t *tcb;
 
-  MTHREAD_CHECK_INIT();        /* Make sure libmthread is initialized */
-
   tcb = mthread_find_tcb(current_thread);
 
   if (tcb->m_state == MS_EXITING)      /* Already stopping, nothing to do. */
@@ -226,7 +220,7 @@ static int mthread_increase_thread_pool(void)
 /*===========================================================================*
  *                             mthread_init                                 *
  *===========================================================================*/
-void mthread_init(void)
+static void __attribute__((__constructor__, __used__)) mthread_init(void)
 {
 /* Initialize thread system; allocate thread structures and start creating
  * threads.
@@ -267,8 +261,6 @@ void **value;
 
   mthread_tcb_t *tcb;
 
-  MTHREAD_CHECK_INIT();        /* Make sure libmthread is initialized */
-
   if (!isokthreadid(join))
        return(ESRCH);
   else if (join == current_thread) 
@@ -323,8 +315,6 @@ void (*proc)(void);
 {
 /* Run procedure proc just once */
 
-  MTHREAD_CHECK_INIT();        /* Make sure libmthread is initialized */
-
   if (once == NULL || proc == NULL) 
        return(EINVAL);
 
@@ -341,8 +331,6 @@ mthread_thread_t mthread_self(void)
 {
 /* Return the thread id of the thread calling this function. */
 
-  MTHREAD_CHECK_INIT();        /* Make sure libmthread is initialized */
-
   return(current_thread);
 }
 
@@ -529,3 +517,12 @@ static void mthread_trampoline(void)
   mthread_exit(r); 
 }
 
+/* pthread compatibility layer. */
+__weak_alias(pthread_create, mthread_create)
+__weak_alias(pthread_detach, mthread_detach)
+__weak_alias(pthread_equal, mthread_equal)
+__weak_alias(pthread_exit, mthread_exit)
+__weak_alias(pthread_join, mthread_join)
+__weak_alias(pthread_once, mthread_once)
+__weak_alias(pthread_self, mthread_self)
+
index 0a96bbb2a9e6ed1fbf10fe674f062252b236bfd7..fe63352cc89c5ea0994a380495fd22f0fce76e1f 100644 (file)
@@ -46,8 +46,6 @@ mthread_attr_t *attr;
 {
 /* Invalidate attribute and deallocate resources. */
 
-  MTHREAD_CHECK_INIT();        /* Make sure mthreads is initialized */
-
   if (attr == NULL)
        return(EINVAL);
 
@@ -72,8 +70,6 @@ mthread_attr_t *attr; /* Attribute */
 /* Initialize the attribute to a known state. */
   struct __mthread_attr *a;
 
-  MTHREAD_CHECK_INIT();        /* Make sure mthreads is initialized */
-
   if (attr == NULL) 
        return(EAGAIN);
   else if (mthread_attr_valid(attr)) 
@@ -102,8 +98,6 @@ int *detachstate;
 /* Get detachstate of a thread attribute */
   struct __mthread_attr *a;
 
-  MTHREAD_CHECK_INIT();        /* Make sure mthreads is initialized */
-
   if (attr == NULL) 
        return(EINVAL);
 
@@ -127,8 +121,6 @@ int detachstate;
 /* Set detachstate of a thread attribute */
   struct __mthread_attr *a;
 
-  MTHREAD_CHECK_INIT();        /* Make sure mthreads is initialized */
-
   if (attr == NULL) 
        return(EINVAL);
 
@@ -156,8 +148,6 @@ size_t *stacksize;
 /* Get stack attribute */
   struct __mthread_attr *a;
 
-  MTHREAD_CHECK_INIT();        /* Make sure mthreads is initialized */
-
   if (attr == NULL) 
        return(EINVAL);
 
@@ -182,8 +172,6 @@ size_t *stacksize;
 /* Get stack size attribute */
   struct __mthread_attr *a;
 
-  MTHREAD_CHECK_INIT();        /* Make sure mthreads is initialized */
-
   if (attr == NULL)
        return(EINVAL);
 
@@ -208,8 +196,6 @@ size_t stacksize;
 /* Set stack attribute */
   struct __mthread_attr *a;
 
-  MTHREAD_CHECK_INIT();        /* Make sure mthreads is initialized */
-
   if (attr == NULL) 
        return(EINVAL);
 
@@ -239,8 +225,6 @@ size_t stacksize;
 /* Set stack size attribute */
   struct __mthread_attr *a;
 
-  MTHREAD_CHECK_INIT();        /* Make sure mthreads is initialized */
-
   if (attr == NULL)
        return(EINVAL);
 
@@ -283,8 +267,6 @@ mthread_attr_t *a;
 /* Check to see if attribute is on the list of valid attributes */
   struct __mthread_attr *loopitem;
 
-  MTHREAD_CHECK_INIT();        /* Make sure mthreads is initialized */
-
   loopitem = va_front;
 
   while (loopitem != NULL) {
@@ -307,8 +289,6 @@ int mthread_attr_verify(void)
 /* Return true when no attributes are in use */
   struct __mthread_attr *loopitem;
 
-  MTHREAD_CHECK_INIT();        /* Make sure mthreads is initialized */
-
   loopitem = va_front;
 
   while (loopitem != NULL) {
@@ -320,4 +300,13 @@ int mthread_attr_verify(void)
 }
 #endif
 
+/* pthread compatibility layer. */
+__weak_alias(pthread_attr_destroy, mthread_attr_destroy)
+__weak_alias(pthread_attr_getdetachstate, mthread_attr_getdetachstate)
+__weak_alias(pthread_attr_getstack, mthread_attr_getstack)
+__weak_alias(pthread_attr_getstacksize, mthread_attr_getstacksize)
+__weak_alias(pthread_attr_init, mthread_attr_init)
+__weak_alias(pthread_attr_setdetachstate, mthread_attr_setdetachstate)
+__weak_alias(pthread_attr_setstack, mthread_attr_setstack)
+__weak_alias(pthread_attr_setstacksize, mthread_attr_setstacksize)
 
index ec3bcc43dc350314a4a9fbfb8aaf313dc5345b45..3488a5f7c5530879f754bc0c8d3bd4790aa354b0 100644 (file)
@@ -58,8 +58,6 @@ mthread_cond_t *cond;
   mthread_thread_t t;
   mthread_tcb_t *tcb;
 
-  MTHREAD_CHECK_INIT();        /* Make sure libmthread is initialized */
-
   if (cond == NULL) 
        return(EINVAL);
   else if (!mthread_cond_valid(cond))
@@ -89,8 +87,6 @@ mthread_cond_t *cond;
   mthread_thread_t t;
   mthread_tcb_t *tcb;
 
-  MTHREAD_CHECK_INIT();        /* Make sure libmthread is initialized */
-
   if (cond == NULL)
        return(EINVAL);
   else if (!mthread_cond_valid(cond))
@@ -126,8 +122,6 @@ mthread_condattr_t *cattr;
 /* Initialize condition variable to a known state. cattr is ignored */
   struct __mthread_cond *c;
 
-  MTHREAD_CHECK_INIT();        /* Make sure libmthread is initialized */
-
   if (cond == NULL) 
        return(EINVAL);
   else if (cattr != NULL) 
@@ -181,8 +175,6 @@ mthread_cond_t *cond;
   mthread_thread_t t;
   mthread_tcb_t *tcb;
 
-  MTHREAD_CHECK_INIT();        /* Make sure libmthread is initialized */
-
   if (cond == NULL)
        return(EINVAL);
   else if (!mthread_cond_valid(cond))
@@ -214,8 +206,6 @@ mthread_cond_t *c;
 /* Check to see if cond is on the list of valid conditions */
   struct __mthread_cond *loopitem;
 
-  MTHREAD_CHECK_INIT();
-
   loopitem = vc_front;
 
   while (loopitem != NULL) {
@@ -237,8 +227,6 @@ int mthread_cond_verify(void)
 {
 /* Return true in case no condition variables are in use. */
 
-  MTHREAD_CHECK_INIT();        /* Make sure libmthread is initialized */
-
   return(vc_front == NULL);
 }
 #endif
@@ -256,8 +244,6 @@ mthread_mutex_t *mutex;
   struct __mthread_cond *c;
   struct __mthread_mutex *m;
 
-  MTHREAD_CHECK_INIT();        /* Make sure libmthread is initialized */
-
   if (cond == NULL || mutex == NULL)
        return(EINVAL);
   
@@ -284,4 +270,6 @@ mthread_mutex_t *mutex;
   return(0);
 }
 
+/* pthread compatibility layer. */
+__weak_alias(pthread_cond_init, mthread_cond_init)
 
index 1dea4e87eb4fb3b0f1e5107eda68966c1955e229..e26da58fc23f9b700ff3c0f0c8e779721e2ef5a4 100644 (file)
@@ -124,3 +124,11 @@ mthread_event_t *event; /* The event to be fired */
 
   return mthread_mutex_unlock(&event->mutex);
 }
+
+/* pthread compatibility layer. */
+__weak_alias(pthread_event_destroy, mthread_event_destroy)
+__weak_alias(pthread_event_init, mthread_event_init)
+__weak_alias(pthread_event_wait, mthread_event_wait)
+__weak_alias(pthread_event_fire, mthread_event_fire)
+__weak_alias(pthread_event_fire_all, mthread_event_fire_all)
+
index f07b26f45fb9911f802164edeac937341b5e3339..cfb3b55f246d701c347be37aa8f5af090c6274ef 100644 (file)
@@ -34,7 +34,6 @@ int mthread_key_create(mthread_key_t *key, void (*destructor)(void *))
  */
   mthread_key_t k;
 
-  MTHREAD_CHECK_INIT();        /* Make sure libmthread is initialized */
   keys_used = 1;
 
   /* We do not yet allocate storage space for the values here, because we can
@@ -65,8 +64,6 @@ int mthread_key_delete(mthread_key_t key)
 /* Free up a key, as well as any associated storage space.
  */
 
-  MTHREAD_CHECK_INIT();        /* Make sure libmthread is initialized */
-
   if (key < 0 || key >= MTHREAD_KEYS_MAX || !keys[key].used)
        return(EINVAL);
 
@@ -85,8 +82,6 @@ void *mthread_getspecific(mthread_key_t key)
 /* Get this thread's local value for the given key. The default is NULL.
  */
 
-  MTHREAD_CHECK_INIT();        /* Make sure libmthread is initialized */
-
   if (key < 0 || key >= MTHREAD_KEYS_MAX || !keys[key].used)
        return(NULL);
 
@@ -109,8 +104,6 @@ int mthread_setspecific(mthread_key_t key, void *value)
  */
   void **p;
 
-  MTHREAD_CHECK_INIT();        /* Make sure libmthread is initialized */
-
   if (key < 0 || key >= MTHREAD_KEYS_MAX || !keys[key].used)
        return(EINVAL);
 
@@ -188,3 +181,10 @@ void mthread_cleanup_values(void)
        }
   } while (found);
 }
+
+/* pthread compatibility layer. */
+__weak_alias(pthread_key_create, mthread_key_create)
+__weak_alias(pthread_key_delete, mthread_key_delete)
+__weak_alias(pthread_getspecific, mthread_getspecific)
+__weak_alias(pthread_setspecific, mthread_setspecific)
+
index 06009b88c1b7a2f5d9b19a11f35975056b8c8ec2..7f4683a6d34432ac06c368d707a1f5d6c4928be1 100644 (file)
@@ -56,8 +56,6 @@ mthread_mutex_t *mutex;
   mthread_thread_t t;
   mthread_tcb_t *tcb;
 
-  MTHREAD_CHECK_INIT();        /* Make sure mthreads is initialized */
-
   if (mutex == NULL)
        return(EINVAL);
 
@@ -95,8 +93,6 @@ mthread_mutexattr_t *mattr;   /* Mutex attribute */
 
   struct __mthread_mutex *m;
 
-  MTHREAD_CHECK_INIT();        /* Make sure mthreads is initialized */
-
   if (mutex == NULL)
        return(EAGAIN);
   else if (mattr != NULL)
@@ -127,8 +123,6 @@ mthread_mutex_t *mutex;     /* Mutex that is to be locked */
 
   struct __mthread_mutex *m;
 
-  MTHREAD_CHECK_INIT();        /* Make sure mthreads is initialized */
-
   if (mutex == NULL)
        return(EINVAL);
 
@@ -180,8 +174,6 @@ mthread_mutex_t *mutex;     /* Mutex that is to be locked */
 
   struct __mthread_mutex *m;
 
-  MTHREAD_CHECK_INIT();        /* Make sure mthreads is initialized */
-
   if (mutex == NULL) 
        return(EINVAL);
 
@@ -210,8 +202,6 @@ mthread_mutex_t *mutex;     /* Mutex that is to be unlocked */
 
   struct __mthread_mutex *m;
 
-  MTHREAD_CHECK_INIT();        /* Make sure mthreads is initialized */
-
   if (mutex == NULL) 
        return(EINVAL);
 
@@ -237,8 +227,6 @@ mthread_mutex_t *m;
 /* Check to see if mutex is on the list of valid mutexes */
   struct __mthread_mutex *loopitem;
 
-  MTHREAD_CHECK_INIT();        /* Make sure mthreads is initialized */
-
   loopitem = vm_front;
 
   while (loopitem != NULL) {
@@ -262,8 +250,6 @@ int mthread_mutex_verify(void)
   int r = 1;
   struct __mthread_mutex *loopitem;
 
-  MTHREAD_CHECK_INIT();        /* Make sure mthreads is initialized */
-
 #ifdef MTHREAD_STRICT
   loopitem = vm_front;
 
@@ -277,5 +263,3 @@ int mthread_mutex_verify(void)
   return(r);
 }
 #endif
-
-
index dc89ffbd2a9c12f1ff617e9cf3212223241cb4f2..7e80219e764fd44fca7a27c21d18ab47a4cae9d6 100644 (file)
@@ -38,10 +38,8 @@ void mthread_init_valid_mutexes(void);
 
 #ifdef MTHREAD_STRICT
 int mthread_mutex_valid(mthread_mutex_t *mutex);
-# define MTHREAD_CHECK_INIT()  mthread_init()
 #else
 # define mthread_mutex_valid(x) ((*x)->mm_magic == MTHREAD_INIT_MAGIC)
-# define MTHREAD_CHECK_INIT()
 #endif
 
 #ifdef MDEBUG
diff --git a/lib/libmthread/pthread_compat.c b/lib/libmthread/pthread_compat.c
new file mode 100644 (file)
index 0000000..0071580
--- /dev/null
@@ -0,0 +1,145 @@
+#define _MTHREADIFY_PTHREADS
+#include <minix/mthread.h>
+#include "global.h"
+#include "proto.h"
+
+/* WARNING:
+ * The following works under the hypothesis that we have only green threads,
+ * which implies that we have no preemption, unless explicit yield or possible
+ * calls done to mthread functions.
+ *
+ * This has impact on the fact we do not maintain a table of currently being
+ * initialized mutexes or condition variables, to prevent double initialization
+ * and/or TOCTU problems. TOCTU could appear between the test against the
+ * initializer value, and the actual initialization, which could lead to double
+ * initialization of the same mutex AND get two threads at the same time in the
+ * critical section as they both hold a (different) mutex.
+ */
+
+
+/*===========================================================================*
+ *                             pthread_mutex_init                           *
+ *===========================================================================*/
+int pthread_mutex_init(pthread_mutex_t *mutex, pthread_mutexattr_t *mattr)
+{
+       return mthread_mutex_init(mutex, mattr);
+}
+
+/*===========================================================================*
+ *                             pthread_mutex_destroy                        *
+ *===========================================================================*/
+int pthread_mutex_destroy(pthread_mutex_t *mutex)
+{
+       if (PTHREAD_MUTEX_INITIALIZER == *mutex) {
+               *mutex = NULL;
+               return 0;
+       }
+
+       return mthread_mutex_destroy(mutex);
+}
+
+/*===========================================================================*
+ *                             pthread_mutex_lock                           *
+ *===========================================================================*/
+int pthread_mutex_lock(pthread_mutex_t *mutex)
+{
+       if (PTHREAD_MUTEX_INITIALIZER == *mutex) {
+               mthread_mutex_init(mutex, NULL);        
+       }
+
+       return mthread_mutex_lock(mutex);
+}
+
+/*===========================================================================*
+ *                             pthread_mutex_trylock                        *
+ *===========================================================================*/
+int pthread_mutex_trylock(pthread_mutex_t *mutex)
+{
+       if (PTHREAD_MUTEX_INITIALIZER == *mutex) {
+               mthread_mutex_init(mutex, NULL);        
+       }
+
+       return pthread_mutex_trylock(mutex);
+}
+
+/*===========================================================================*
+ *                             pthread_mutex_unlock                         *
+ *===========================================================================*/
+int pthread_mutex_unlock(pthread_mutex_t *mutex)
+{
+       if (PTHREAD_MUTEX_INITIALIZER == *mutex) {
+               mthread_mutex_init(mutex, NULL);        
+       }
+
+       return mthread_mutex_unlock(mutex);
+}
+
+/*===========================================================================*
+ *                             pthread_cond_init                            *
+ *===========================================================================*/
+int pthread_cond_init(pthread_cond_t *cond, pthread_condattr_t *cattr)
+{
+       return mthread_cond_init(cond, cattr);
+}
+
+/*===========================================================================*
+ *                             pthread_cond_broadcast                       *
+ *===========================================================================*/
+int pthread_cond_broadcast(pthread_cond_t *cond)
+{
+       if (PTHREAD_COND_INITIALIZER == *cond) {
+               mthread_cond_init(cond, NULL);
+       }
+
+       return mthread_cond_broadcast(cond);
+}
+
+/*===========================================================================*
+ *                             pthread_cond_destroy                         *
+ *===========================================================================*/
+int pthread_cond_destroy(pthread_cond_t *cond)
+{
+       if (PTHREAD_COND_INITIALIZER == *cond) {
+               *cond = NULL;
+               return 0;
+       }
+
+       return mthread_cond_destroy(cond);
+}
+
+/*===========================================================================*
+ *                             pthread_cond_signal                          *
+ *===========================================================================*/
+int pthread_cond_signal(pthread_cond_t *cond)
+{
+       if (PTHREAD_COND_INITIALIZER == *cond) {
+               mthread_cond_init(cond, NULL);
+       }
+
+       return mthread_cond_signal(cond);
+}
+
+/*===========================================================================*
+ *                             pthread_cond_wait                            *
+ *===========================================================================*/
+int pthread_cond_wait(pthread_cond_t *cond, pthread_mutex_t *mutex)
+{
+       if (PTHREAD_COND_INITIALIZER == *cond) {
+               mthread_cond_init(cond, NULL);
+       }
+
+       return mthread_cond_wait(cond, mutex);
+}
+
+/*===========================================================================*
+ *                             pthread_rwlock_init                          *
+ *===========================================================================*/
+int pthread_rwlock_init(pthread_rwlock_t *rwlock, pthread_rwlockattr_t *UNUSED(attr))
+{
+       return mthread_rwlock_init(rwlock);
+}
+
+#if !defined(__weak_alias)
+#error __weak_alias is required to compile the pthread compat library
+#endif
+
index f06c41cb6b197cc3396a4ce469f371a0fb96cbb4..95e255c4f264f60d79fa52cd7ebdb34d5916bc68 100644 (file)
@@ -128,3 +128,10 @@ mthread_rwlock_t *rwlock; /* The rwlock to be unlocked */
 
   return r;
 }
+
+/* pthread compatibility layer. */
+__weak_alias(pthread_rwlock_destroy, mthread_rwlock_destroy)
+__weak_alias(pthread_rwlock_rdlock, mthread_rwlock_rdlock)
+__weak_alias(pthread_rwlock_wrlock, mthread_rwlock_wrlock)
+__weak_alias(pthread_rwlock_unlock, mthread_rwlock_unlock)
+
index 455525241954f0901dc57b597bc829ce6284d93d..eb5a1c2009756471673454a13ee72e3c9d9b1e77 100644 (file)
@@ -38,8 +38,6 @@ void mthread_schedule(void)
   mthread_tcb_t *new_tcb, *old_tcb;
   ucontext_t *new_ctx, *old_ctx;
 
-  MTHREAD_CHECK_INIT();        /* Make sure libmthread is initialized */
-
   old_thread = current_thread;
 
   if (mthread_queue_isempty(&run_queue)) {
@@ -154,8 +152,6 @@ int mthread_yield(void)
   mthread_tcb_t *tcb;
   mthread_thread_t t;
 
-  MTHREAD_CHECK_INIT();        /* Make sure libmthread is initialized */
-
   /* Detached threads cannot clean themselves up. This is a perfect moment to
    * do it */
   for (t = (mthread_thread_t) 0; need_reset > 0 && t < no_threads; t++) {
@@ -192,8 +188,6 @@ void mthread_yield_all(void)
  * this function will lead to a deadlock.
  */
 
-  MTHREAD_CHECK_INIT();        /* Make sure libmthread is initialized */
-
   if (yield_all) mthread_panic("Deadlock: two threads trying to yield_all");
   yield_all = 1;
 
@@ -214,3 +208,8 @@ void mthread_yield_all(void)
   yield_all = 0;
 }
 
+/* pthread compatibility layer. */
+__weak_alias(pthread_yield, mthread_yield)
+__weak_alias(sched_yield, mthread_yield)
+__weak_alias(pthread_yield_all, mthread_yield_all)
+
diff --git a/lib/libmthread/shlib_version b/lib/libmthread/shlib_version
new file mode 100644 (file)
index 0000000..97c9f92
--- /dev/null
@@ -0,0 +1,2 @@
+major=0
+minor=0
index ff2f54b97519483b10208a12e1e1874b3d496647..6758cd123118e1887408a223da97ad192bea8693 100644 (file)
@@ -7,7 +7,6 @@
 #define cond_t         mthread_cond_t
 #define attr_t         mthread_attr_t
 
-#define threads_init   mthread_init
 #define yield          mthread_yield
 #define yield_all      mthread_yield_all
 
index b34dada803c8540a0620c1ff47521c4dd4d53acf..8c1f0d771a67234e5838334b816933a8d896a37d 100644 (file)
@@ -24,7 +24,6 @@ void worker_init(void)
   struct worker_thread *wp;
   int i;
 
-  threads_init();
   if (mthread_attr_init(&tattr) != 0)
        panic("failed to initialize attribute");
   if (mthread_attr_setstacksize(&tattr, TH_STACKSIZE) != 0)
index f75ef9890b1a7c936a31be3262c093d75f3b8598..40ccc9e2a2d178bcdbce6baa4ef84510e4607e6b 100644 (file)
@@ -614,7 +614,7 @@ DPLIBC ?= ${DESTDIR}${LIBC_SO}
 .endif
 .else
 .if ${LIB} != "c" && ${LIB:Mgcc*} == "" \
-    && ${LIB} != "minlib" && ${LIB} != "sys" && ${LIB} != "minc" # Minix-specific libs
+    && ${LIB} != "minlib" && ${LIB} != "sys" && ${LIB} != "minc" && ${LIB} != "mthread" # Minix-specific libs
 .if !empty(LIBC_SO)
 DPLIBC ?= ${DESTDIR}${LIBC_SO}
 .endif
index 9aa3201d99f02f87ca434f5499c4a46f0080c51d..d4a39015801773134b4b1ddf895ca4d9bf081395 100644 (file)
@@ -221,6 +221,7 @@ LIB${_lib:tu}=      ${DESTDIR}/usr/lib/lib${_lib:S/xx/++/:S/atf_c/atf-c/}.a
        devman \
        minc \
        minlib \
+       mthread \
        netdriver \
        sffs \
        sys \
index b67d86236ca5e037731c1bc47b660edae23a97d8..8d99453ef20b8ef4b73dcaa7b51ccab938a42f25 100644 (file)
@@ -1116,7 +1116,6 @@ int main(void)
   once = MTHREAD_ONCE_INIT;
 
   start(59);
-  mthread_init(); 
   test_scheduling();
   test_mutex();
   test_event();