Make |Heap::findGCInfo()| threadsafe.

Source/platform/heap/Heap.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 1582 matching lines @@
 #else
     if (!s_heapDoesNotContainCache->lookup(address))
         s_heapDoesNotContainCache->addEntry(address, true);
 #endif
     return 0;
 }
 
 #if ENABLE(GC_TRACING)
 const GCInfo* Heap::findGCInfo(Address address)
 {
+    bool needLockForIteration = !ThreadState::current()->isAtSafePoint();

[haraken, 2014/06/05 09:11:23]

I guess what matters is not whether we're at a safe point but whether we're in a
GC. If so, we should check ThreadState::isAnyThreadInGC() instead of
isAtSafePoint().

[kouhei (in TOK), 2014/06/05 09:26:22]

Done.

+    if (needLockForIteration)
+        ThreadState::threadAttachMutex().lock();
+
     ThreadState::AttachedThreadStateSet& threads = ThreadState::attachedThreads();
     for (ThreadState::AttachedThreadStateSet::iterator it = threads.begin(), end = threads.end(); it != end; ++it) {
         if (const GCInfo* gcInfo = (*it)->findGCInfo(address)) {
+            if (needLockForIteration)
+                ThreadState::threadAttachMutex().unlock();
             return gcInfo;
         }
     }
+    if (needLockForIteration)
+        ThreadState::threadAttachMutex().unlock();
     return 0;
 }
 
 void Heap::dumpPathToObjectOnNextGC(void* p)
 {
     static_cast<MarkingVisitor*>(s_markingVisitor)->dumpPathToObjectOnNextGC(p);
 }
 #endif
 
 void Heap::pushTraceCallback(void* object, TraceCallback callback)
@@ skipping 158 matching lines @@
 template class ThreadHeap<FinalizedHeapObjectHeader>;
 template class ThreadHeap<HeapObjectHeader>;
 
 Visitor* Heap::s_markingVisitor;
 CallbackStack* Heap::s_markingStack;
 CallbackStack* Heap::s_weakCallbackStack;
 HeapDoesNotContainCache* Heap::s_heapDoesNotContainCache;
 bool Heap::s_shutdownCalled = false;
 bool Heap::s_lastGCWasConservative = false;
 }