Reviewers: Michael Starzinger,

Message:
Committed patchset #1 manually as r21425 (presubmit successful).

Description:
Increase external allocation limit.

BUG=
[email protected]

Committed: https://code.google.com/p/v8/source/detail?r=21425

Please review this at https://codereview.chromium.org/291653005/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files (+2, -3 lines):
  M src/heap.cc


Index: src/heap.cc
diff --git a/src/heap.cc b/src/heap.cc
index b672bbffbb811966d292580abca673257041dd9b..ff47e30a1a7546a9aa61a50d9b5643f95836b440 100644
--- a/src/heap.cc
+++ b/src/heap.cc
@@ -5052,9 +5052,8 @@ bool Heap::ConfigureHeap(int max_semi_space_size,
initial_semispace_size_ = Min(initial_semispace_size_, max_semi_space_size_);

// The external allocation limit should be below 256 MB on all architectures - // to avoid unnecessary low memory notifications, as that is the threshold
-  // for some embedders.
-  external_allocation_limit_ = 12 * max_semi_space_size_;
+  // to avoid that resource-constrained embedders run low on memory.
+  external_allocation_limit_ = 256 * MB;
   ASSERT(external_allocation_limit_ <= 256 * MB);

// The old generation is paged and needs at least one page for each space.


--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to