Changeset 110902 in webkit for trunk/Source/JavaScriptCore/heap/CopiedSpace.h

Timestamp:

Mar 15, 2012, 4:04:39 PM (13 years ago)

Author:

[email protected]

Message:

CopiedSpace::tryAllocateOversize assumes system page size
​https://bugs.webkit.org/show_bug.cgi?id=80615

Patch by Myles Maxfield <​[email protected]> on 2012-03-15
Reviewed by Geoffrey Garen.

• heap/CopiedSpace.cpp:

(JSC::CopiedSpace::tryAllocateOversize):

• heap/CopiedSpace.h:

(CopiedSpace):

• heap/CopiedSpaceInlineMethods.h:

(JSC::CopiedSpace::oversizeBlockFor):

• wtf/BumpPointerAllocator.h:

(WTF::BumpPointerPool::create):

• wtf/StdLibExtras.h:

(WTF::roundUpToMultipleOf):

File:

• trunk/Source/JavaScriptCore/heap/CopiedSpace.h (modified) (2 diffs)

trunk/Source/JavaScriptCore/heap/CopiedSpace.h

@@ -36 +36 @@
 #include <wtf/OSAllocator.h>
 #include <wtf/PageAllocationAligned.h>
+#include <wtf/PageBlock.h>
 #include <wtf/StdLibExtras.h>
 #include <wtf/ThreadingPrimitives.h>
@@ -119 +120 @@
 
     static const size_t s_maxAllocationSize = 32 * KB;
-    static const size_t s_pageSize = 4 * KB;
-    static const size_t s_pageMask = ~(s_pageSize - 1);
     static const size_t s_initialBlockNum = 16;
     static const size_t s_blockMask = ~(HeapBlock::s_blockSize - 1);