summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorwolfbeast <mcwerewolf@gmail.com>2018-02-19 15:37:37 +0100
committerwolfbeast <mcwerewolf@gmail.com>2018-02-19 15:37:37 +0100
commitce1bd4f244710fcec718e164c6d9b1621c8ec810 (patch)
tree83ca82c69f34d05f18f388a9b043407aed22cfe6
parent616475a0c1109460fe42045ca3380f4d8f0752cd (diff)
downloadUXP-ce1bd4f244710fcec718e164c6d9b1621c8ec810.tar
UXP-ce1bd4f244710fcec718e164c6d9b1621c8ec810.tar.gz
UXP-ce1bd4f244710fcec718e164c6d9b1621c8ec810.tar.lz
UXP-ce1bd4f244710fcec718e164c6d9b1621c8ec810.tar.xz
UXP-ce1bd4f244710fcec718e164c6d9b1621c8ec810.zip
Use thread_local calls on Win and Mac.
This resolves #24.
-rw-r--r--js/src/jsutil.cpp2
-rw-r--r--mfbt/ThreadLocal.h38
2 files changed, 7 insertions, 33 deletions
diff --git a/js/src/jsutil.cpp b/js/src/jsutil.cpp
index bb9f33df9..705d21975 100644
--- a/js/src/jsutil.cpp
+++ b/js/src/jsutil.cpp
@@ -39,7 +39,7 @@ mozilla::Atomic<AutoEnterOOMUnsafeRegion*> AutoEnterOOMUnsafeRegion::owner_;
namespace oom {
JS_PUBLIC_DATA(uint32_t) targetThread = 0;
-JS_PUBLIC_DATA(MOZ_THREAD_LOCAL(uint32_t)) threadType;
+MOZ_THREAD_LOCAL(uint32_t) threadType;
JS_PUBLIC_DATA(uint64_t) maxAllocations = UINT64_MAX;
JS_PUBLIC_DATA(uint64_t) counter = 0;
JS_PUBLIC_DATA(bool) failAlways = true;
diff --git a/mfbt/ThreadLocal.h b/mfbt/ThreadLocal.h
index 880e67735..391e748ad 100644
--- a/mfbt/ThreadLocal.h
+++ b/mfbt/ThreadLocal.h
@@ -9,20 +9,7 @@
#ifndef mozilla_ThreadLocal_h
#define mozilla_ThreadLocal_h
-#if defined(XP_WIN)
-// This file will get included in any file that wants to add a profiler mark.
-// In order to not bring <windows.h> together we could include windef.h and
-// winbase.h which are sufficient to get the prototypes for the Tls* functions.
-// # include <windef.h>
-// # include <winbase.h>
-// Unfortunately, even including these headers causes us to add a bunch of ugly
-// stuff to our namespace e.g #define CreateEvent CreateEventW
-extern "C" {
-__declspec(dllimport) void* __stdcall TlsGetValue(unsigned long);
-__declspec(dllimport) int __stdcall TlsSetValue(unsigned long, void*);
-__declspec(dllimport) unsigned long __stdcall TlsAlloc();
-}
-#else
+#if !defined(XP_WIN)
# include <pthread.h>
# include <signal.h>
#endif
@@ -44,7 +31,7 @@ typedef sig_atomic_t sig_safe_t;
namespace detail {
-#if defined(HAVE_THREAD_TLS_KEYWORD)
+#if defined(HAVE_THREAD_TLS_KEYWORD) || defined(XP_WIN) || defined(XP_MACOSX)
#define MOZ_HAS_THREAD_LOCAL
#endif
@@ -91,11 +78,7 @@ template<typename T>
class ThreadLocal
{
#ifndef MOZ_HAS_THREAD_LOCAL
-#if defined(XP_WIN)
- typedef unsigned long key_t;
-#else
typedef pthread_key_t key_t;
-#endif
// Integral types narrower than void* must be extended to avoid
// warnings from valgrind on some platforms. This helper type
@@ -161,12 +144,7 @@ ThreadLocal<T>::init()
return true;
#else
if (!initialized()) {
-#ifdef XP_WIN
- mKey = TlsAlloc();
- mInited = mKey != 0xFFFFFFFFUL; // TLS_OUT_OF_INDEXES
-#else
mInited = !pthread_key_create(&mKey, nullptr);
-#endif
}
return mInited;
#endif
@@ -181,11 +159,7 @@ ThreadLocal<T>::get() const
#else
MOZ_ASSERT(initialized());
void* h;
-#ifdef XP_WIN
- h = TlsGetValue(mKey);
-#else
h = pthread_getspecific(mKey);
-#endif
return static_cast<T>(reinterpret_cast<typename Helper<T>::Type>(h));
#endif
}
@@ -199,11 +173,7 @@ ThreadLocal<T>::set(const T aValue)
#else
MOZ_ASSERT(initialized());
void* h = reinterpret_cast<void*>(static_cast<typename Helper<T>::Type>(aValue));
-#ifdef XP_WIN
- bool succeeded = TlsSetValue(mKey, h);
-#else
bool succeeded = !pthread_setspecific(mKey, h);
-#endif
if (!succeeded) {
MOZ_CRASH();
}
@@ -211,7 +181,11 @@ ThreadLocal<T>::set(const T aValue)
}
#ifdef MOZ_HAS_THREAD_LOCAL
+#if defined(XP_WIN) || defined(XP_MACOSX)
+#define MOZ_THREAD_LOCAL(TYPE) thread_local mozilla::detail::ThreadLocal<TYPE>
+#else
#define MOZ_THREAD_LOCAL(TYPE) __thread mozilla::detail::ThreadLocal<TYPE>
+#endif
#else
#define MOZ_THREAD_LOCAL(TYPE) mozilla::detail::ThreadLocal<TYPE>
#endif