X-Git-Url: https://git.kernel.dk/?p=fio.git;a=blobdiff_plain;f=mutex.c;h=94e11884424cd203cd1f3357fef33250ffbd94eb;hp=0f13064af8f26929d3aeac09a22de8331d0e9fa3;hb=23a2a8e35df958d2f3ffd478accfd2ba09eecdf0;hpb=42124d37bd92d865b0c27751c520f671c2a91101 diff --git a/mutex.c b/mutex.c index 0f13064a..94e11884 100644 --- a/mutex.c +++ b/mutex.c @@ -3,77 +3,131 @@ #include #include #include +#include +#include #include #include +#include "fio.h" +#include "log.h" #include "mutex.h" #include "arch/arch.h" +#include "os/os.h" +#include "helpers.h" +#include "time.h" +#include "gettime.h" + +static clockid_t fio_clk_id = CLOCK_REALTIME; void fio_mutex_remove(struct fio_mutex *mutex) { - close(mutex->mutex_fd); + pthread_cond_destroy(&mutex->cond); munmap((void *) mutex, sizeof(*mutex)); } struct fio_mutex *fio_mutex_init(int value) { - char mutex_name[] = "/tmp/.fio_mutex.XXXXXX"; struct fio_mutex *mutex = NULL; pthread_mutexattr_t attr; pthread_condattr_t cond; - int fd; - - fd = mkstemp(mutex_name); - if (fd < 0) { - perror("open mutex"); - return NULL; - } - - if (ftruncate(fd, sizeof(struct fio_mutex)) < 0) { - perror("ftruncate mutex"); - goto err; - } + int ret; mutex = (void *) mmap(NULL, sizeof(struct fio_mutex), - PROT_READ | PROT_WRITE, MAP_SHARED, fd, 0); + PROT_READ | PROT_WRITE, + OS_MAP_ANON | MAP_SHARED, -1, 0); if (mutex == MAP_FAILED) { perror("mmap mutex"); - close(fd); mutex = NULL; goto err; } - unlink(mutex_name); - mutex->mutex_fd = fd; mutex->value = value; - if (pthread_mutexattr_init(&attr) < 0) { - perror("pthread_mutexattr_init"); + ret = pthread_mutexattr_init(&attr); + if (ret) { + log_err("pthread_mutexattr_init: %s\n", strerror(ret)); goto err; } - if (pthread_mutexattr_setpshared(&attr, PTHREAD_PROCESS_SHARED) < 0) { - perror("pthread_mutexattr_setpshared"); + + /* + * Not all platforms support process shared mutexes (FreeBSD) + */ +#ifdef FIO_HAVE_PSHARED_MUTEX + ret = pthread_mutexattr_setpshared(&attr, PTHREAD_PROCESS_SHARED); + if (ret) { + log_err("pthread_mutexattr_setpshared: %s\n", strerror(ret)); goto err; } +#endif pthread_condattr_init(&cond); +#ifdef FIO_HAVE_PSHARED_MUTEX pthread_condattr_setpshared(&cond, PTHREAD_PROCESS_SHARED); +#endif +#ifdef FIO_HAVE_PTHREAD_CONDATTR_SETCLOCK + pthread_condattr_setclock(&cond, fio_clk_id); +#endif pthread_cond_init(&mutex->cond, &cond); - if (pthread_mutex_init(&mutex->lock, &attr) < 0) { - perror("pthread_mutex_init"); + ret = pthread_mutex_init(&mutex->lock, &attr); + if (ret) { + log_err("pthread_mutex_init: %s\n", strerror(ret)); goto err; } + pthread_condattr_destroy(&cond); + pthread_mutexattr_destroy(&attr); + return mutex; err: if (mutex) fio_mutex_remove(mutex); - unlink(mutex_name); return NULL; } +static int mutex_timed_out(struct timeval *t, unsigned int seconds) +{ + return mtime_since_now(t) >= seconds * 1000; +} + +int fio_mutex_down_timeout(struct fio_mutex *mutex, unsigned int seconds) +{ + struct timeval tv_s; + struct timespec t; + int ret = 0; + + fio_gettime(&tv_s, NULL); + + clock_gettime(fio_clk_id, &t); + t.tv_sec += seconds; + + pthread_mutex_lock(&mutex->lock); + + while (!mutex->value && !ret) { + mutex->waiters++; + + /* + * Some platforms (FreeBSD 9?) seems to return timed out + * way too early, double check. + */ + ret = pthread_cond_timedwait(&mutex->cond, &mutex->lock, &t); + if (ret == ETIMEDOUT && !mutex_timed_out(&tv_s, seconds)) { + pthread_mutex_lock(&mutex->lock); + ret = 0; + } + + mutex->waiters--; + } + + if (!ret) { + mutex->value--; + pthread_mutex_unlock(&mutex->lock); + } + + return ret; +} + void fio_mutex_down(struct fio_mutex *mutex) { pthread_mutex_lock(&mutex->lock); @@ -145,3 +199,19 @@ void fio_mutex_up_write(struct fio_mutex *mutex) pthread_cond_signal(&mutex->cond); pthread_mutex_unlock(&mutex->lock); } + +static void fio_init fio_mutex_global_init(void) +{ +#ifdef FIO_HAVE_PTHREAD_CONDATTR_SETCLOCK +#ifdef FIO_HAVE_CLOCK_MONOTONIC + pthread_condattr_t cond; + + pthread_condattr_init(&cond); + + if (!pthread_condattr_setclock(&cond, CLOCK_MONOTONIC)) + fio_clk_id = CLOCK_MONOTONIC; + + pthread_condattr_destroy(&cond); +#endif +#endif +}