Changeset 215671 in webkit for trunk/Source/JavaScriptCore/runtime/VMTraps.cpp

Timestamp:

Apr 23, 2017, 2:12:44 AM (8 years ago)

Author:

Yusuke Suzuki

Message:

[JSC] Use DoublyLinkedList for MachineThread
​https://bugs.webkit.org/show_bug.cgi?id=171171

Reviewed by Mark Lam.

MachineThread can use WTF::DoublyLinkedList to simplify
its implementation. We should not use Vector<> etc. since
we do not want to call allocations during suspending and
resuming threads.

• heap/MachineStackMarker.cpp:

(JSC::MachineThreads::MachineThreads):
(JSC::MachineThreads::~MachineThreads):
(JSC::MachineThreads::addCurrentThread):
(JSC::MachineThreads::removeThreadIfFound):
(JSC::MachineThreads::MachineThread::MachineThread):
(JSC::MachineThreads::tryCopyOtherThreadStacks):

• heap/MachineStackMarker.h:

(JSC::MachineThreads::threadsListHead):

• runtime/SamplingProfiler.cpp:

(JSC::FrameWalker::isValidFramePointer):

• runtime/VMTraps.cpp:

(JSC::findActiveVMAndStackBounds):

File:

• trunk/Source/JavaScriptCore/runtime/VMTraps.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/runtime/VMTraps.cpp

@@ -111 +111 @@
         }
 
-        for (MachineThreads::MachineThread* thread = machineThreads.threadsListHead(machineThreadsLocker); thread; thread = thread->next) {
+        const auto& threadList = machineThreads.threadsListHead(machineThreadsLocker);
+        for (MachineThreads::MachineThread* thread = threadList.head(); thread; thread = thread->next()) {
             RELEASE_ASSERT(thread->stackBase());
             RELEASE_ASSERT(thread->stackEnd());