currentThread is too slow!
authorlevin@chromium.org <levin@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 15 Jul 2011 18:49:24 +0000 (18:49 +0000)
committerlevin@chromium.org <levin@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 15 Jul 2011 18:49:24 +0000 (18:49 +0000)
https://bugs.webkit.org/show_bug.cgi?id=64577

Reviewed by Darin Adler and Dmitry Titov.

The problem is that currentThread results in a pthread_once call which always takes a lock.
With this change, currentThread is 10% faster than isMainThread in release mode and only
5% slower than isMainThread in debug.

* wtf/ThreadIdentifierDataPthreads.cpp:
(WTF::ThreadIdentifierData::initializeOnce): Remove the pthread once stuff
which is no longer needed because this is called from initializeThreading().
(WTF::ThreadIdentifierData::identifier): Remove the initializeKeyOnce call because
intialization of the pthread key should already be done.
(WTF::ThreadIdentifierData::initialize): Ditto.
* wtf/ThreadIdentifierDataPthreads.h:
* wtf/ThreadingPthreads.cpp:
(WTF::initializeThreading): Acquire the pthread key here.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@91082 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/wtf/ThreadIdentifierDataPthreads.cpp
Source/JavaScriptCore/wtf/ThreadIdentifierDataPthreads.h
Source/JavaScriptCore/wtf/ThreadingPthreads.cpp

index 7a336f1f16e39b7e47d641d8019867dfdefcf092..f425ff42edd648edb8b8bbc3ab23ace860d7403e 100644 (file)
@@ -1,3 +1,24 @@
+2011-07-14  David Levin  <levin@chromium.org>
+
+        currentThread is too slow!
+        https://bugs.webkit.org/show_bug.cgi?id=64577
+
+        Reviewed by Darin Adler and Dmitry Titov.
+
+        The problem is that currentThread results in a pthread_once call which always takes a lock.
+        With this change, currentThread is 10% faster than isMainThread in release mode and only
+        5% slower than isMainThread in debug.
+
+        * wtf/ThreadIdentifierDataPthreads.cpp:
+        (WTF::ThreadIdentifierData::initializeOnce): Remove the pthread once stuff
+        which is no longer needed because this is called from initializeThreading().
+        (WTF::ThreadIdentifierData::identifier): Remove the initializeKeyOnce call because
+        intialization of the pthread key should already be done.
+        (WTF::ThreadIdentifierData::initialize): Ditto.
+        * wtf/ThreadIdentifierDataPthreads.h:
+        * wtf/ThreadingPthreads.cpp:
+        (WTF::initializeThreading): Acquire the pthread key here.
+
 2011-07-14  Filip Pizlo  <fpizlo@apple.com>
 
         DFG JIT does not optimize Branch as well as it could.
index dcbc96ffc44b03260a577f2d9559eacfdfc34324..c44971dca60f4b22ffb6249fcff16feb18ff1e23 100644 (file)
@@ -38,8 +38,7 @@
 
 namespace WTF {
 
-pthread_key_t ThreadIdentifierData::m_key;
-static pthread_once_t onceControl = PTHREAD_ONCE_INIT;
+pthread_key_t ThreadIdentifierData::m_key = PTHREAD_KEYS_MAX;
 
 void clearPthreadHandleForIdentifier(ThreadIdentifier);
 
@@ -48,9 +47,15 @@ ThreadIdentifierData::~ThreadIdentifierData()
     clearPthreadHandleForIdentifier(m_identifier);
 }
 
+void ThreadIdentifierData::initializeOnce()
+{
+    if (pthread_key_create(&m_key, destruct))
+        CRASH();
+}
+
 ThreadIdentifier ThreadIdentifierData::identifier()
 {
-    initializeKeyOnce();
+    ASSERT(m_key != PTHREAD_KEYS_MAX);
     ThreadIdentifierData* threadIdentifierData = static_cast<ThreadIdentifierData*>(pthread_getspecific(m_key));
 
     return threadIdentifierData ? threadIdentifierData->m_identifier : 0;
@@ -59,8 +64,6 @@ ThreadIdentifier ThreadIdentifierData::identifier()
 void ThreadIdentifierData::initialize(ThreadIdentifier id)
 {
     ASSERT(!identifier());
-
-    initializeKeyOnce();
     pthread_setspecific(m_key, new ThreadIdentifierData(id));
 }
 
@@ -79,18 +82,6 @@ void ThreadIdentifierData::destruct(void* data)
     pthread_setspecific(m_key, threadIdentifierData);
 }
 
-void ThreadIdentifierData::initializeKeyOnceHelper()
-{
-    if (pthread_key_create(&m_key, destruct))
-        CRASH();
-}
-
-void ThreadIdentifierData::initializeKeyOnce()
-{
-    if (pthread_once(&onceControl, initializeKeyOnceHelper))
-        CRASH();
-}
-
 } // namespace WTF
 
 #endif // USE(PTHREADS)
index ed1ba2cc0285af2b196e66c77bcb9cd3b62cb18e..84349a0cddaaec860bcbe4bc0a725bd275e72ebd 100644 (file)
@@ -42,6 +42,10 @@ class ThreadIdentifierData {
 public:
     ~ThreadIdentifierData();
 
+    // One time initialization for this class as a whole.
+    // This method must be called before initialize() and it is not thread-safe.
+    static void initializeOnce();
+
     // Creates and puts an instance of ThreadIdentifierData into thread-specific storage.
     static void initialize(ThreadIdentifier identifier);
 
@@ -62,9 +66,6 @@ private:
     // ThreadIdentifier from the threadMap, completing the cleanup.
     static void destruct(void* data);
 
-    static void initializeKeyOnceHelper();
-    static void initializeKeyOnce();
-
     ThreadIdentifier m_identifier;
     bool m_isDestroyedOnce;
     static pthread_key_t m_key;
index 916aa3656dbac7f383f63e743ffeb6e2810ddec5..d5ad46c4bb6fe7caccdb1e11005b381597f73597 100644 (file)
@@ -81,6 +81,7 @@ void initializeThreading()
     atomicallyInitializedStaticMutex = new Mutex;
     threadMapMutex();
     initializeRandomNumberGenerator();
+    ThreadIdentifierData::initializeOnce();
 }
 
 void lockAtomicallyInitializedStaticMutex()