Title: [173909] trunk/Source/WTF
Revision
173909
Author
gga...@apple.com
Date
2014-09-23 22:10:18 -0700 (Tue, 23 Sep 2014)

Log Message

2014-09-23  Geoffrey Garen  <gga...@apple.com>

        Rolled out r173346.

            bmalloc should honor the FastMalloc statistics API
            https://bugs.webkit.org/show_bug.cgi?id=136592

        This didn't really work. Because we allow ranges with and without
        physical pages to merge, and we allow double-committing and
        double-decommitting, we can't rely on commit actions to track memory
        footprint.

        * wtf/FastMalloc.cpp:
        (WTF::fastMallocStatistics):

Modified Paths

Diff

Modified: trunk/Source/WTF/ChangeLog (173908 => 173909)


--- trunk/Source/WTF/ChangeLog	2014-09-24 05:07:19 UTC (rev 173908)
+++ trunk/Source/WTF/ChangeLog	2014-09-24 05:10:18 UTC (rev 173909)
@@ -1,3 +1,18 @@
+2014-09-23  Geoffrey Garen  <gga...@apple.com>
+
+        Rolled out r173346.
+
+            bmalloc should honor the FastMalloc statistics API
+            https://bugs.webkit.org/show_bug.cgi?id=136592
+
+        This didn't really work. Because we allow ranges with and without
+        physical pages to merge, and we allow double-committing and
+        double-decommitting, we can't rely on commit actions to track memory
+        footprint.
+
+        * wtf/FastMalloc.cpp:
+        (WTF::fastMallocStatistics):
+
 2014-09-23  Joseph Pecoraro  <pecor...@apple.com>
 
         Web Inspector: Should be able to attach a debugger to a JSContext before anything is executed

Modified: trunk/Source/WTF/wtf/FastMalloc.cpp (173908 => 173909)


--- trunk/Source/WTF/wtf/FastMalloc.cpp	2014-09-24 05:07:19 UTC (rev 173908)
+++ trunk/Source/WTF/wtf/FastMalloc.cpp	2014-09-24 05:10:18 UTC (rev 173909)
@@ -477,11 +477,7 @@
 
 FastMallocStatistics fastMallocStatistics()
 {
-    FastMallocStatistics statistics;
-    statistics.committedVMBytes = bmalloc::api::heapSize();
-    statistics.reservedVMBytes = bmalloc::api::heapCapacity();
-    statistics.freeListBytes = 0; // bmalloc doesn't really have free lists.
-
+    FastMallocStatistics statistics = { 0, 0, 0 };
     return statistics;
 }
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to