Revision: 5802
Author: [email protected]
Date: Wed Nov 10 02:14:13 2010
Log: Another fix after the executable memory limit introduction.
The max executable size must be less than or equal to the max old
space size. It is an extra limit in addition to the old space size.
Review URL: http://codereview.chromium.org/4730002
http://code.google.com/p/v8/source/detail?r=5802
Modified:
/branches/bleeding_edge/src/heap.cc
=======================================
--- /branches/bleeding_edge/src/heap.cc Wed Nov 10 01:20:08 2010
+++ /branches/bleeding_edge/src/heap.cc Wed Nov 10 02:14:13 2010
@@ -4348,6 +4348,12 @@
if (max_executable_size > 0) {
max_executable_size_ = RoundUp(max_executable_size, Page::kPageSize);
}
+
+ // The max executable size must be less than or equal to the max old
+ // generation size.
+ if (max_executable_size_ > max_old_generation_size_) {
+ max_executable_size_ = max_old_generation_size_;
+ }
// The new space size must be a power of two to support single-bit
testing
// for containment.
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
