https://bugs.webkit.org/show_bug.cgi?id=81963 WebProcess can get stuck in GC during...
authorslewis@apple.com <slewis@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 24 Mar 2012 05:21:26 +0000 (05:21 +0000)
committerslewis@apple.com <slewis@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 24 Mar 2012 05:21:26 +0000 (05:21 +0000)
<rdar://problem/11094830> WebProcess appears to get stuck in its GC handler (81963).
Remove the call to garbage collect  in low memory signal handler.  Did some testing with hitting the low memory handler
during Membuster and we would get back at most 100k - 200k.  That isn't enough to help the system, and in
that state the GC collection can take a substantial amount of time.

Reviewed by Geoff Garen.

Performance Change, no change in behavior.

* platform/mac/MemoryPressureHandlerMac.mm:
(WebCore::MemoryPressureHandler::releaseMemory):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@111977 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/platform/mac/MemoryPressureHandlerMac.mm

index b241f18f021fcbf7fc637e6d3ff56d707c8f15d9..71fddd57d36fecf16ad9364f5b8ecbab5a607049 100644 (file)
@@ -1,3 +1,18 @@
+2012-03-23  Stephanie Lewis  <slewis@apple.com>
+
+        https://bugs.webkit.org/show_bug.cgi?id=81963 WebProcess can get stuck in GC during many low memory signals.
+        <rdar://problem/11094830> WebProcess appears to get stuck in its GC handler (81963).
+        Remove the call to garbage collect  in low memory signal handler.  Did some testing with hitting the low memory handler
+        during Membuster and we would get back at most 100k - 200k.  That isn't enough to help the system, and in 
+        that state the GC collection can take a substantial amount of time.
+
+        Reviewed by Geoff Garen.
+
+        Performance Change, no change in behavior.
+
+        * platform/mac/MemoryPressureHandlerMac.mm:
+        (WebCore::MemoryPressureHandler::releaseMemory):
+
 2012-03-23  W. James MacLean  <wjmaclean@chromium.org>
 
         [chromium] CCLayerTreeHostImpl::scrollBegin() should return ScrollFailed for CCInputHandlerClient::Gesture type when wheel handlers found.
index 0e410d94fc44f6307700a68af5db485f0f95d0ec..1727bac449614f7276411df7d7893ab4b6b3f26e 100644 (file)
@@ -135,8 +135,6 @@ void MemoryPressureHandler::releaseMemory(bool critical)
 
     memoryCache()->pruneToPercentage(critical ? 0 : 0.5f);
 
-    gcController().garbageCollectNow();
-
     WTF::releaseFastMallocFreeMemory();
 }
 #endif