Revision: 19930
Author: [email protected]
Date: Fri Mar 14 10:51:16 2014 UTC
Log: Synchronize thread creation on posix platforms.
BUG=
[email protected]
Review URL: https://codereview.chromium.org/200173002
http://code.google.com/p/v8/source/detail?r=19930
Modified:
/branches/bleeding_edge/src/platform-posix.cc
=======================================
--- /branches/bleeding_edge/src/platform-posix.cc Tue Mar 11 15:46:56 2014
UTC
+++ /branches/bleeding_edge/src/platform-posix.cc Fri Mar 14 10:51:16 2014
UTC
@@ -581,6 +581,10 @@
Thread::~Thread() {
delete data_;
}
+
+
+// Synchronizes thread creation
+static Mutex thread_creation_mutex_;
static void SetThreadName(const char* name) {
@@ -612,10 +616,10 @@
static void* ThreadEntry(void* arg) {
Thread* thread = reinterpret_cast<Thread*>(arg);
- // This is also initialized by the first argument to pthread_create()
but we
- // don't know which thread will run first (the original thread or the new
- // one) so we initialize it here too.
- thread->data()->thread_ = pthread_self();
+ // We take the lock here to make sure that pthread_create finished first
since
+ // we don't know which thread will run first (the original thread or the
new
+ // one).
+ { LockGuard<Mutex> lock_guard(&thread_creation_mutex_); }
SetThreadName(thread->name());
ASSERT(thread->data()->thread_ != kNoThread);
thread->NotifyStartedAndRun();
@@ -642,7 +646,10 @@
ASSERT_EQ(0, result);
}
#endif
- result = pthread_create(&data_->thread_, &attr, ThreadEntry, this);
+ {
+ LockGuard<Mutex> lock_guard(&thread_creation_mutex_);
+ result = pthread_create(&data_->thread_, &attr, ThreadEntry, this);
+ }
ASSERT_EQ(0, result);
result = pthread_attr_destroy(&attr);
ASSERT_EQ(0, result);
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
