They return the errno, not 0/-1. Thanks to
Fabio Checconi <fabio@gandalf.sssup.it>
for noticing this.
Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
#include <pthread.h>
#include <sys/mman.h>
#include <pthread.h>
#include <sys/mman.h>
#include "mutex.h"
#include "arch/arch.h"
#include "mutex.h"
#include "arch/arch.h"
struct fio_mutex *mutex = NULL;
pthread_mutexattr_t attr;
pthread_condattr_t cond;
struct fio_mutex *mutex = NULL;
pthread_mutexattr_t attr;
pthread_condattr_t cond;
fd = mkstemp(mutex_name);
if (fd < 0) {
fd = mkstemp(mutex_name);
if (fd < 0) {
mutex->mutex_fd = fd;
mutex->value = value;
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));
- if (pthread_mutexattr_setpshared(&attr, PTHREAD_PROCESS_SHARED) < 0) {
- perror("pthread_mutexattr_setpshared");
+ ret = pthread_mutexattr_setpshared(&attr, PTHREAD_PROCESS_SHARED);
+ if (ret) {
+ log_err("pthread_mutexattr_setpshared: %s\n", strerror(ret));
pthread_condattr_setpshared(&cond, PTHREAD_PROCESS_SHARED);
pthread_cond_init(&mutex->cond, &cond);
pthread_condattr_setpshared(&cond, PTHREAD_PROCESS_SHARED);
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));