-/* -*- mode: C; c-file-style: "gnu" -*- */
+/* -*- mode: C; c-file-style: "gnu"; indent-tabs-mode: nil; -*- */
/* dbus-sysdeps-pthread.c Implements threads using pthreads (internal to libdbus)
*
* Copyright (C) 2002, 2003, 2006 Red Hat, Inc.
*
* You should have received a copy of the GNU General Public License
* along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
*
*/
+#include <config.h>
#include "dbus-internals.h"
#include "dbus-sysdeps.h"
#include "dbus-threads.h"
#include <pthread.h>
#include <string.h>
-typedef struct {
- pthread_mutex_t lock;
- int count;
- pthread_t holder;
-} DBusMutexPThread;
+#ifdef HAVE_ERRNO_H
+#include <errno.h>
+#endif
-typedef struct {
- pthread_cond_t cond;
-} DBusCondVarPThread;
+#include <config.h>
+
+#ifdef HAVE_MONOTONIC_CLOCK
+/* Whether we have a "monotonic" clock; i.e. a clock not affected by
+ * changes in system time.
+ * This is initialized once in check_monotonic_clock below.
+ * https://bugs.freedesktop.org/show_bug.cgi?id=18121
+ */
+static dbus_bool_t have_monotonic_clock = 0;
+#endif
+
+struct DBusRMutex {
+ pthread_mutex_t lock; /**< the lock */
+};
+
+struct DBusCMutex {
+ pthread_mutex_t lock; /**< the lock */
+};
+
+struct DBusCondVar {
+ pthread_cond_t cond; /**< the condition */
+};
#define DBUS_MUTEX(m) ((DBusMutex*) m)
#define DBUS_MUTEX_PTHREAD(m) ((DBusMutexPThread*) m)
#define DBUS_COND_VAR_PTHREAD(c) ((DBusCondVarPThread*) c)
+#ifdef DBUS_DISABLE_ASSERT
+/* (tmp != 0) is a no-op usage to silence compiler */
+#define PTHREAD_CHECK(func_name, result_or_call) \
+ do { int tmp = (result_or_call); if (tmp != 0) {;} } while (0)
+#else
#define PTHREAD_CHECK(func_name, result_or_call) do { \
int tmp = (result_or_call); \
if (tmp != 0) { \
func_name, tmp, strerror(tmp), _DBUS_FUNCTION_NAME); \
} \
} while (0)
-
-static DBusMutex*
-_dbus_pthread_mutex_new (void)
+#endif /* !DBUS_DISABLE_ASSERT */
+
+DBusCMutex *
+_dbus_platform_cmutex_new (void)
{
- DBusMutexPThread *pmutex;
+ DBusCMutex *pmutex;
int result;
-
- pmutex = dbus_new (DBusMutexPThread, 1);
+
+ pmutex = dbus_new (DBusCMutex, 1);
if (pmutex == NULL)
return NULL;
PTHREAD_CHECK ("pthread_mutex_init", result);
}
- /* Only written */
- pmutex->count = 0;
-
- /* There's no portable way to have a "null" pthread afaik so we
- * can't set pmutex->holder to anything sensible. We only access it
- * once the lock is held (which means we've set it).
- */
-
- return DBUS_MUTEX (pmutex);
+ return pmutex;
}
-static void
-_dbus_pthread_mutex_free (DBusMutex *mutex)
+DBusRMutex *
+_dbus_platform_rmutex_new (void)
{
- DBusMutexPThread *pmutex = DBUS_MUTEX_PTHREAD (mutex);
+ DBusRMutex *pmutex;
+ pthread_mutexattr_t mutexattr;
+ int result;
- _dbus_assert (pmutex->count == 0);
-
- PTHREAD_CHECK ("pthread_mutex_destroy", pthread_mutex_destroy (&pmutex->lock));
+ pmutex = dbus_new (DBusRMutex, 1);
+ if (pmutex == NULL)
+ return NULL;
- dbus_free (pmutex);
-}
+ pthread_mutexattr_init (&mutexattr);
+ pthread_mutexattr_settype (&mutexattr, PTHREAD_MUTEX_RECURSIVE);
+ result = pthread_mutex_init (&pmutex->lock, &mutexattr);
+ pthread_mutexattr_destroy (&mutexattr);
-static void
-_dbus_pthread_mutex_lock (DBusMutex *mutex)
-{
- DBusMutexPThread *pmutex = DBUS_MUTEX_PTHREAD (mutex);
- pthread_t self = pthread_self ();
-
- /* If the count is > 0 then someone had the lock, maybe us. If it is
- * 0, then it might immediately change right after we read it,
- * but it will be changed by another thread; i.e. if we read 0,
- * we assume that this thread doesn't have the lock.
- *
- * Not 100% sure this is safe, but ... seems like it should be.
- */
- if (pmutex->count == 0)
+ if (result == ENOMEM || result == EAGAIN)
{
- /* We know we don't have the lock; someone may have the lock. */
-
- PTHREAD_CHECK ("pthread_mutex_lock", pthread_mutex_lock (&pmutex->lock));
-
- /* We now have the lock. Count must be 0 since it was before when we
- * did not have the lock, and we have not changed it in this thread.
- */
- _dbus_assert (pmutex->count == 0);
-
- pmutex->holder = self;
- pmutex->count = 1;
+ dbus_free (pmutex);
+ return NULL;
}
else
{
- /* We know someone had the lock, possibly us. Thus
- * pmutex->holder is not pointing to junk, though it may not be
- * the lock holder anymore if the lock holder is not us.
- * If the lock holder is us, then we definitely have the lock.
- */
-
- if (pthread_equal (pmutex->holder, self))
- {
- /* We already have the lock. */
- }
- else
- {
- /* Wait for the lock */
- PTHREAD_CHECK ("pthread_mutex_lock", pthread_mutex_lock (&pmutex->lock));
- _dbus_assert (pmutex->count == 0);
- }
-
- pmutex->count += 1;
+ PTHREAD_CHECK ("pthread_mutex_init", result);
}
+
+ return pmutex;
}
-static void
-_dbus_pthread_mutex_unlock (DBusMutex *mutex)
+void
+_dbus_platform_cmutex_free (DBusCMutex *mutex)
{
- DBusMutexPThread *pmutex = DBUS_MUTEX_PTHREAD (mutex);
+ PTHREAD_CHECK ("pthread_mutex_destroy", pthread_mutex_destroy (&mutex->lock));
+ dbus_free (mutex);
+}
- _dbus_assert (pmutex->count > 0);
-
- pmutex->count -= 1;
+void
+_dbus_platform_rmutex_free (DBusRMutex *mutex)
+{
+ PTHREAD_CHECK ("pthread_mutex_destroy", pthread_mutex_destroy (&mutex->lock));
+ dbus_free (mutex);
+}
- if (pmutex->count == 0)
- PTHREAD_CHECK ("pthread_mutex_unlock", pthread_mutex_unlock (&pmutex->lock));
-
- /* We leave pmutex->holder set to ourselves, its content is undefined if count is 0 */
+void
+_dbus_platform_cmutex_lock (DBusCMutex *mutex)
+{
+ PTHREAD_CHECK ("pthread_mutex_lock", pthread_mutex_lock (&mutex->lock));
+}
+
+void
+_dbus_platform_rmutex_lock (DBusRMutex *mutex)
+{
+ PTHREAD_CHECK ("pthread_mutex_lock", pthread_mutex_lock (&mutex->lock));
+}
+
+void
+_dbus_platform_cmutex_unlock (DBusCMutex *mutex)
+{
+ PTHREAD_CHECK ("pthread_mutex_unlock", pthread_mutex_unlock (&mutex->lock));
}
-static DBusCondVar *
-_dbus_pthread_condvar_new (void)
+void
+_dbus_platform_rmutex_unlock (DBusRMutex *mutex)
{
- DBusCondVarPThread *pcond;
+ PTHREAD_CHECK ("pthread_mutex_unlock", pthread_mutex_unlock (&mutex->lock));
+}
+
+DBusCondVar *
+_dbus_platform_condvar_new (void)
+{
+ DBusCondVar *pcond;
+ pthread_condattr_t attr;
int result;
- pcond = dbus_new (DBusCondVarPThread, 1);
+ pcond = dbus_new (DBusCondVar, 1);
if (pcond == NULL)
return NULL;
- result = pthread_cond_init (&pcond->cond, NULL);
+ pthread_condattr_init (&attr);
+#ifdef HAVE_MONOTONIC_CLOCK
+ if (have_monotonic_clock)
+ pthread_condattr_setclock (&attr, CLOCK_MONOTONIC);
+#endif
+
+ result = pthread_cond_init (&pcond->cond, &attr);
+ pthread_condattr_destroy (&attr);
if (result == EAGAIN || result == ENOMEM)
{
PTHREAD_CHECK ("pthread_cond_init", result);
}
- return DBUS_COND_VAR (pcond);
+ return pcond;
}
-static void
-_dbus_pthread_condvar_free (DBusCondVar *cond)
-{
- DBusCondVarPThread *pcond = DBUS_COND_VAR_PTHREAD (cond);
-
- PTHREAD_CHECK ("pthread_cond_destroy", pthread_cond_destroy (&pcond->cond));
-
- dbus_free (pcond);
+void
+_dbus_platform_condvar_free (DBusCondVar *cond)
+{
+ PTHREAD_CHECK ("pthread_cond_destroy", pthread_cond_destroy (&cond->cond));
+ dbus_free (cond);
}
-static void
-_dbus_pthread_condvar_wait (DBusCondVar *cond,
- DBusMutex *mutex)
+void
+_dbus_platform_condvar_wait (DBusCondVar *cond,
+ DBusCMutex *mutex)
{
- DBusMutexPThread *pmutex = DBUS_MUTEX_PTHREAD (mutex);
- DBusCondVarPThread *pcond = DBUS_COND_VAR_PTHREAD (cond);
- int old_count;
-
- _dbus_assert (pmutex->count > 0);
- _dbus_assert (pthread_equal (pmutex->holder, pthread_self ()));
-
- old_count = pmutex->count;
- pmutex->count = 0;
- PTHREAD_CHECK ("pthread_cond_wait", pthread_cond_wait (&pcond->cond, &pmutex->lock));
- _dbus_assert (pmutex->count == 0);
- pmutex->count = old_count;
+ PTHREAD_CHECK ("pthread_cond_wait", pthread_cond_wait (&cond->cond, &mutex->lock));
}
-static dbus_bool_t
-_dbus_pthread_condvar_wait_timeout (DBusCondVar *cond,
- DBusMutex *mutex,
- int timeout_milliseconds)
+dbus_bool_t
+_dbus_platform_condvar_wait_timeout (DBusCondVar *cond,
+ DBusCMutex *mutex,
+ int timeout_milliseconds)
{
- DBusMutexPThread *pmutex = DBUS_MUTEX_PTHREAD (mutex);
- DBusCondVarPThread *pcond = DBUS_COND_VAR_PTHREAD (cond);
struct timeval time_now;
struct timespec end_time;
int result;
- int old_count;
-
- _dbus_assert (pmutex->count > 0);
- _dbus_assert (pthread_equal (pmutex->holder, pthread_self ()));
-
+
+#ifdef HAVE_MONOTONIC_CLOCK
+ if (have_monotonic_clock)
+ {
+ struct timespec monotonic_timer;
+ clock_gettime (CLOCK_MONOTONIC,&monotonic_timer);
+ time_now.tv_sec = monotonic_timer.tv_sec;
+ time_now.tv_usec = monotonic_timer.tv_nsec / 1000;
+ }
+ else
+ /* This else falls through to gettimeofday */
+#endif
gettimeofday (&time_now, NULL);
end_time.tv_sec = time_now.tv_sec + timeout_milliseconds / 1000;
end_time.tv_nsec -= 1000*1000*1000;
}
- old_count = pmutex->count;
- pmutex->count = 0;
- result = pthread_cond_timedwait (&pcond->cond, &pmutex->lock, &end_time);
+ result = pthread_cond_timedwait (&cond->cond, &mutex->lock, &end_time);
if (result != ETIMEDOUT)
{
PTHREAD_CHECK ("pthread_cond_timedwait", result);
}
- _dbus_assert (pmutex->count == 0);
- pmutex->count = old_count;
-
/* return true if we did not time out */
return result != ETIMEDOUT;
}
+void
+_dbus_platform_condvar_wake_one (DBusCondVar *cond)
+{
+ PTHREAD_CHECK ("pthread_cond_signal", pthread_cond_signal (&cond->cond));
+}
+
static void
-_dbus_pthread_condvar_wake_one (DBusCondVar *cond)
+check_monotonic_clock (void)
+{
+#ifdef HAVE_MONOTONIC_CLOCK
+ struct timespec dummy;
+ if (clock_getres (CLOCK_MONOTONIC, &dummy) == 0)
+ have_monotonic_clock = TRUE;
+#endif
+}
+
+dbus_bool_t
+_dbus_threads_init_platform_specific (void)
{
- DBusCondVarPThread *pcond = DBUS_COND_VAR_PTHREAD (cond);
+ /* These have static variables, and we need to handle both the case
+ * where dbus_threads_init() has been called and when it hasn't;
+ * so initialize them before any threads are allowed to enter.
+ */
+ check_monotonic_clock ();
+ (void) _dbus_check_setuid ();
- PTHREAD_CHECK ("pthread_cond_signal", pthread_cond_signal (&pcond->cond));
+ return TRUE;
}
-static void
-_dbus_pthread_condvar_wake_all (DBusCondVar *cond)
+static pthread_mutex_t init_mutex = PTHREAD_MUTEX_INITIALIZER;
+
+void
+_dbus_threads_lock_platform_specific (void)
{
- DBusCondVarPThread *pcond = DBUS_COND_VAR_PTHREAD (cond);
-
- PTHREAD_CHECK ("pthread_cond_broadcast", pthread_cond_broadcast (&pcond->cond));
+ pthread_mutex_lock (&init_mutex);
}
-static const DBusThreadFunctions pthread_functions =
+void
+_dbus_threads_unlock_platform_specific (void)
{
- DBUS_THREAD_FUNCTIONS_RECURSIVE_MUTEX_NEW_MASK |
- DBUS_THREAD_FUNCTIONS_RECURSIVE_MUTEX_FREE_MASK |
- DBUS_THREAD_FUNCTIONS_RECURSIVE_MUTEX_LOCK_MASK |
- DBUS_THREAD_FUNCTIONS_RECURSIVE_MUTEX_UNLOCK_MASK |
- DBUS_THREAD_FUNCTIONS_CONDVAR_NEW_MASK |
- DBUS_THREAD_FUNCTIONS_CONDVAR_FREE_MASK |
- DBUS_THREAD_FUNCTIONS_CONDVAR_WAIT_MASK |
- DBUS_THREAD_FUNCTIONS_CONDVAR_WAIT_TIMEOUT_MASK |
- DBUS_THREAD_FUNCTIONS_CONDVAR_WAKE_ONE_MASK|
- DBUS_THREAD_FUNCTIONS_CONDVAR_WAKE_ALL_MASK,
- NULL, NULL, NULL, NULL,
- _dbus_pthread_condvar_new,
- _dbus_pthread_condvar_free,
- _dbus_pthread_condvar_wait,
- _dbus_pthread_condvar_wait_timeout,
- _dbus_pthread_condvar_wake_one,
- _dbus_pthread_condvar_wake_all,
- _dbus_pthread_mutex_new,
- _dbus_pthread_mutex_free,
- _dbus_pthread_mutex_lock,
- _dbus_pthread_mutex_unlock
+ pthread_mutex_unlock (&init_mutex);
+}
+
+#ifdef DBUS_HAVE_STATIC_RECURSIVE_MUTEXES
+
+static pthread_mutex_t global_locks[] = {
+ /* 0-4 */
+ PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP,
+ PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP,
+ PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP,
+ PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP,
+ PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP,
+ /* 5-9 */
+ PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP,
+ PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP,
+ PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP,
+ PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP,
+ PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP,
+ /* 10-11 */
+ PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP,
+ PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP
};
+_DBUS_STATIC_ASSERT (_DBUS_N_ELEMENTS (global_locks) == _DBUS_N_GLOBAL_LOCKS);
+
dbus_bool_t
-_dbus_threads_init_platform_specific (void)
+_dbus_lock (DBusGlobalLock lock)
+{
+ /* No initialization is needed. */
+ _dbus_assert (lock >= 0);
+ _dbus_assert (lock < _DBUS_N_GLOBAL_LOCKS);
+
+ PTHREAD_CHECK ("pthread_mutex_lock",
+ pthread_mutex_lock (&(global_locks[lock])));
+ return TRUE;
+}
+
+void
+_dbus_unlock (DBusGlobalLock lock)
{
- return dbus_threads_init (&pthread_functions);
+ /* No initialization is needed. */
+ _dbus_assert (lock >= 0);
+ _dbus_assert (lock < _DBUS_N_GLOBAL_LOCKS);
+
+ PTHREAD_CHECK ("pthread_mutex_unlock",
+ pthread_mutex_unlock (&(global_locks[lock])));
}
+
+#endif