Backed out changeset 5f59c9c92b8b (bug 1288579) for bustage.
authorRyan VanderMeulen <ryanvm@gmail.com>
Thu, 01 Sep 2016 22:16:08 -0400
changeset 348004 99c0c53f5ea3ca670d75e420d165de97e60d6db1
parent 348003 d76058fbf56fd00bc0e69136a1f8ae73047efe74
child 348005 6ea5133aab19860080b3d8b2221c2d5b5c17a465
push id6389
push userraliiev@mozilla.com
push dateMon, 19 Sep 2016 13:38:22 +0000
treeherdermozilla-beta@01d67bfe6c81 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1288579
milestone50.0a2
backs out5f59c9c92b8b9ea206975b7b37db483e85513a27
Backed out changeset 5f59c9c92b8b (bug 1288579) for bustage.
js/src/gc/Allocator.cpp
--- a/js/src/gc/Allocator.cpp
+++ b/js/src/gc/Allocator.cpp
@@ -328,20 +328,18 @@ GCRuntime::refillFreeListOffMainThread(E
     JSRuntime* rt = zone->runtimeFromAnyThread();
 
     AutoMaybeStartBackgroundAllocation maybeStartBGAlloc;
 
     // If we're off the main thread, we try to allocate once and return
     // whatever value we get. We need to first ensure the main thread is not in
     // a GC session.
     AutoLockHelperThreadState lock;
-    while (rt->isHeapBusy()) {
+    while (rt->isHeapBusy())
         HelperThreadState().wait(lock, GlobalHelperThreadState::PRODUCER);
-        HelperThreadState().notifyOne(GlobalHelperThreadState::PRODUCER, lock);
-    }
 
     return arenas->allocateFromArena(zone, thingKind, maybeStartBGAlloc);
 }
 
 /* static */ TenuredCell*
 GCRuntime::refillFreeListInGC(Zone* zone, AllocKind thingKind)
 {
     /*