Changeset 20351 in webkit for trunk/JavaScriptCore/wtf

Timestamp:

Mar 20, 2007, 4:57:01 PM (18 years ago)

Author:

mjs

Message:

Reviewed by Geoff and Adam.

• make USE(MULTIPLE_THREADS) support more portable ​http://bugs.webkit.org/show_bug.cgi?id=13069

• fixed a threadsafety bug discovered by testing this

• enhanced threadsafety assertions in collector

• API/JSCallbackObject.cpp: (KJS::JSCallbackObject::~JSCallbackObject): This destructor can't DropAllLocks around the finalize callback, because it gets called from garbage collection and we can't let other threads collect!

• JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:
• kjs/JSLock.cpp: (KJS::JSLock::currentThreadIsHoldingLock): Added new function to allow stronger assertions than just that the lock is held by some thread (you can now assert that the current thread is holding it, given the new JSLock design).
• kjs/JSLock.h:
• kjs/collector.cpp: Refactored for portability plus added some stronger assertions. (KJS::Collector::allocate): (KJS::currentThreadStackBase): (KJS::Collector::registerAsMainThread): (KJS::onMainThread): (KJS::PlatformThread::PlatformThread): (KJS::getCurrentPlatformThread): (KJS::Collector::Thread::Thread): (KJS::destroyRegisteredThread): (KJS::Collector::registerThread): (KJS::Collector::markCurrentThreadConservatively): (KJS::suspendThread): (KJS::resumeThread): (KJS::getPlatformThreadRegisters): (KJS::otherThreadStackPointer): (KJS::otherThreadStackBase): (KJS::Collector::markOtherThreadConservatively): (KJS::Collector::markStackObjectsConservatively): (KJS::Collector::protect): (KJS::Collector::unprotect): (KJS::Collector::collectOnMainThreadOnly): (KJS::Collector::markMainThreadOnlyObjects): (KJS::Collector::collect):
• kjs/collector.h:
• wtf/FastMalloc.cpp: (WTF::fastMallocSetIsMultiThreaded):
• wtf/FastMallocInternal.h:
• wtf/Platform.h:

Location:

trunk/JavaScriptCore/wtf

Files:

• FastMalloc.cpp (modified) (2 diffs)
• FastMallocInternal.h (modified) (2 diffs)
• Platform.h (modified) (1 diff)

trunk/JavaScriptCore/wtf/FastMalloc.cpp

@@ -67 +67 @@
 
 #include "Assertions.h"
+#if USE(MULTIPLE_THREADS)
+#include <pthread.h>
+#endif
 
 #ifndef USE_SYSTEM_MALLOC
@@ -160 +163 @@
 }
 
-#if !PLATFORM(WIN_OS)
 void fastMallocSetIsMultiThreaded() 
 {
 }
-#endif
 
 } // namespace WTF

trunk/JavaScriptCore/wtf/FastMallocInternal.h

@@ -24 +24 @@
 #define WTF_FastMallocInternal_h
 
-#if !PLATFORM(WIN_OS)
-
-#include <pthread.h>
-
 namespace WTF {
     // Clients must call this function before allocating memory on a secondary thread.
@@ -33 +29 @@
 }
 
-#endif
-
 #endif //  WTF_FastMallocInternal_h

trunk/JavaScriptCore/wtf/Platform.h

@@ -173 +173 @@
 
 /* multiple threads only supported on Mac for now */
-#if PLATFORM(MAC)
+#if PLATFORM(MAC) || PLATFORM(WIN)
 #define WTF_USE_MULTIPLE_THREADS 1
 #endif