Changeset 130643 in webkit

Timestamp:

Oct 8, 2012, 8:11:21 AM (14 years ago)

Author:

kling@webkit.org

Message:

Lower minimum table size of WTF::HashTable to reduce memory usage.
<​http://webkit.org/b/98406>
<rdar://problem/12432140>

Reviewed by Anders Carlsson.

Lower the default minimumTableSize for WTF hash tables from 64 to 8 entries.
This reduces WebProcess memory consumption by ~16MB on Membuster3 (a 6% progression!)

No significant movement on PLT or JSC benchmarks on my machine. If there's a perf regression somewhere
from this, we can tweak individual tables to have a larger minimumTableSize.

• wtf/HashTraits.h:

Location:

trunk/Source/WTF

Files:

• ChangeLog (modified) (1 diff)
• wtf/HashTraits.h (modified) (1 diff)

trunk/Source/WTF/ChangeLog

@@ +1 @@
+2012-10-08  Andreas Kling  <kling@webkit.org>
+
+        Lower minimum table size of WTF::HashTable to reduce memory usage.
+        <http://webkit.org/b/98406>
+        <rdar://problem/12432140>
+
+        Reviewed by Anders Carlsson.
+
+        Lower the default minimumTableSize for WTF hash tables from 64 to 8 entries.
+        This reduces WebProcess memory consumption by ~16MB on Membuster3 (a 6% progression!)
+
+        No significant movement on PLT or JSC benchmarks on my machine. If there's a perf regression somewhere
+        from this, we can tweak individual tables to have a larger minimumTableSize.
+
+        * wtf/HashTraits.h:
+
 2012-10-08  Andreas Kling  <kling@webkit.org>
 

trunk/Source/WTF/wtf/HashTraits.h

@@ -51 +51 @@
         static const bool needsDestruction = true;
 
-        static const int minimumTableSize = 64;
+        // The starting table size. Can be overridden when we know beforehand that
+        // a hash table will have at least N entries.
+        static const int minimumTableSize = 8;
     };