From 72242057a25b59d01b3d4e1343c94cc7ac354950 Mon Sep 17 00:00:00 2001 From: Jens Axboe Date: Wed, 2 Apr 2014 15:45:25 -0600 Subject: [PATCH] mutex: add open init function and down_trylock() variant Signed-off-by: Jens Axboe --- mutex.c | 55 +++++++++++++++++++++++++++++++++++++------------------ mutex.h | 2 ++ 2 files changed, 39 insertions(+), 18 deletions(-) diff --git a/mutex.c b/mutex.c index e1fbb607..466e20ed 100644 --- a/mutex.c +++ b/mutex.c @@ -25,29 +25,19 @@ void fio_mutex_remove(struct fio_mutex *mutex) munmap((void *) mutex, sizeof(*mutex)); } -struct fio_mutex *fio_mutex_init(int value) +int __fio_mutex_init(struct fio_mutex *mutex, int value) { - struct fio_mutex *mutex = NULL; pthread_mutexattr_t attr; pthread_condattr_t cond; int ret; - mutex = (void *) mmap(NULL, sizeof(struct fio_mutex), - PROT_READ | PROT_WRITE, - OS_MAP_ANON | MAP_SHARED, -1, 0); - if (mutex == MAP_FAILED) { - perror("mmap mutex"); - mutex = NULL; - goto err; - } - mutex->value = value; mutex->magic = FIO_MUTEX_MAGIC; ret = pthread_mutexattr_init(&attr); if (ret) { log_err("pthread_mutexattr_init: %s\n", strerror(ret)); - goto err; + return ret; } /* @@ -57,7 +47,7 @@ struct fio_mutex *fio_mutex_init(int value) ret = pthread_mutexattr_setpshared(&attr, PTHREAD_PROCESS_SHARED); if (ret) { log_err("pthread_mutexattr_setpshared: %s\n", strerror(ret)); - goto err; + return ret; } #endif @@ -70,17 +60,30 @@ struct fio_mutex *fio_mutex_init(int value) ret = pthread_mutex_init(&mutex->lock, &attr); if (ret) { log_err("pthread_mutex_init: %s\n", strerror(ret)); - goto err; + return ret; } pthread_condattr_destroy(&cond); pthread_mutexattr_destroy(&attr); + return 0; +} - return mutex; -err: - if (mutex) - fio_mutex_remove(mutex); +struct fio_mutex *fio_mutex_init(int value) +{ + struct fio_mutex *mutex = NULL; + + mutex = (void *) mmap(NULL, sizeof(struct fio_mutex), + PROT_READ | PROT_WRITE, + OS_MAP_ANON | MAP_SHARED, -1, 0); + if (mutex == MAP_FAILED) { + perror("mmap mutex"); + return NULL; + } + + if (!__fio_mutex_init(mutex, value)) + return mutex; + fio_mutex_remove(mutex); return NULL; } @@ -125,6 +128,22 @@ int fio_mutex_down_timeout(struct fio_mutex *mutex, unsigned int seconds) return ret; } +int fio_mutex_down_trylock(struct fio_mutex *mutex) +{ + int ret = 1; + + assert(mutex->magic == FIO_MUTEX_MAGIC); + + pthread_mutex_lock(&mutex->lock); + if (mutex->value) { + mutex->value--; + ret = 0; + } + pthread_mutex_unlock(&mutex->lock); + + return ret; +} + void fio_mutex_down(struct fio_mutex *mutex) { assert(mutex->magic == FIO_MUTEX_MAGIC); diff --git a/mutex.h b/mutex.h index 4f3486df..246afeed 100644 --- a/mutex.h +++ b/mutex.h @@ -24,10 +24,12 @@ enum { FIO_MUTEX_UNLOCKED = 1, }; +extern int __fio_mutex_init(struct fio_mutex *, int); extern struct fio_mutex *fio_mutex_init(int); extern void fio_mutex_remove(struct fio_mutex *); extern void fio_mutex_up(struct fio_mutex *); extern void fio_mutex_down(struct fio_mutex *); +extern int fio_mutex_down_trylock(struct fio_mutex *); extern int fio_mutex_down_timeout(struct fio_mutex *, unsigned int); extern void fio_rwlock_read(struct fio_rwlock *); -- 2.25.1