aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2013-01-07 10:32:01 +0100
committerMax Kellermann <max@duempel.org>2013-01-07 10:33:37 +0100
commita68302e50b835146cdb526a5850b9b19cb36a0ad (patch)
tree1b6832c3022a78f5b328b73c9c6dbf8370b94f50
parent333a08ebf9ceb86d5f9354f96321c4c0f704374e (diff)
downloadmpd-a68302e50b835146cdb526a5850b9b19cb36a0ad.tar.gz
mpd-a68302e50b835146cdb526a5850b9b19cb36a0ad.tar.xz
mpd-a68302e50b835146cdb526a5850b9b19cb36a0ad.zip
thread/Mutex: don't use std::mutex
Use a custom pthread_mutex_t wrapper because std::mutex adds overhead.
Diffstat (limited to '')
-rw-r--r--Makefile.am1
-rw-r--r--src/thread/Mutex.hxx16
-rw-r--r--src/thread/PosixMutex.hxx60
3 files changed, 69 insertions, 8 deletions
diff --git a/Makefile.am b/Makefile.am
index aa3234bf5..4d1e5620d 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -167,6 +167,7 @@ src_mpd_SOURCES = \
$(OUTPUT_API_SRC) \
$(MIXER_API_SRC) \
src/thread/Mutex.hxx \
+ src/thread/PosixMutex.hxx \
src/thread/CriticalSection.hxx \
src/glib_socket.h \
src/clock.c src/clock.h \
diff --git a/src/thread/Mutex.hxx b/src/thread/Mutex.hxx
index df8d140c6..675af74b1 100644
--- a/src/thread/Mutex.hxx
+++ b/src/thread/Mutex.hxx
@@ -27,6 +27,14 @@
#include "CriticalSection.hxx"
typedef CriticalSection Mutex;
+#else
+
+#include "PosixMutex.hxx"
+
+typedef PosixMutex Mutex;
+
+#endif
+
class ScopeLock {
Mutex &mutex;
@@ -43,12 +51,4 @@ public:
ScopeLock &operator=(const ScopeLock &other) = delete;
};
-#else
-
-#include <mutex>
-typedef std::mutex Mutex;
-typedef std::lock_guard<std::mutex> ScopeLock;
-
-#endif
-
#endif
diff --git a/src/thread/PosixMutex.hxx b/src/thread/PosixMutex.hxx
new file mode 100644
index 000000000..1862a3e2d
--- /dev/null
+++ b/src/thread/PosixMutex.hxx
@@ -0,0 +1,60 @@
+/*
+ * Copyright (C) 2009-2013 Max Kellermann <max@duempel.org>
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ *
+ * - Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ *
+ * - Redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the
+ * distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS
+ * FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE
+ * FOUNDATION OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT,
+ * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
+ * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
+ * SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
+ * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT,
+ * STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED
+ * OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#ifndef MPD_THREAD_POSIX_MUTEX_HXX
+#define MPD_THREAD_POSIX_MUTEX_HXX
+
+#include <pthread.h>
+
+/**
+ * Low-level wrapper for a pthread_mutex_t.
+ */
+class PosixMutex {
+ pthread_mutex_t mutex;
+
+public:
+ constexpr PosixMutex():mutex(PTHREAD_MUTEX_INITIALIZER) {}
+
+ PosixMutex(const PosixMutex &other) = delete;
+ PosixMutex &operator=(const PosixMutex &other) = delete;
+
+ void lock() {
+ pthread_mutex_lock(&mutex);
+ }
+
+ bool try_lock() {
+ return pthread_mutex_trylock(&mutex) == 0;
+ }
+
+ void unlock() {
+ pthread_mutex_unlock(&mutex);
+ }
+};
+
+#endif