Changeset 34886 in webkit for trunk/JavaScriptCore/VM/RegisterFile.cpp

Timestamp:

Jun 30, 2008, 4:21:43 AM (17 years ago)

Author:

[email protected]

Message:

Reviewed by Cameron Zwarich.

Make RegisterFile really unmap memory on destruction.

This fixes run-webkit-tests --threaded, which ran out of address space in a few seconds.

• VM/RegisterFile.cpp: (KJS::RegisterFile::~RegisterFile): Unmap all the memory, not just 1/4 of it.

• kjs/JSGlobalObject.h: Don't include RegisterFile.h, so that changes to it don't make half of WebCore rebuild.

• VM/Machine.h: Don't forward declare RegisterFile, as RegisterFile.h is included already.

• VM/RegisterFile.h: (KJS::RegisterFile::RegisterFile): Assert that the allocation succeeded.

File:

• trunk/JavaScriptCore/VM/RegisterFile.cpp (modified) (1 diff)

trunk/JavaScriptCore/VM/RegisterFile.cpp

@@ -35 +35 @@
 {
 #if HAVE(MMAP)
-    munmap(m_buffer, m_capacity + m_maxGlobals);
+    munmap(m_buffer, (m_capacity + m_maxGlobals) * sizeof(Register));
 #elif HAVE(VIRTUALALLOC)
     // FIXME: Use VirtualFree.