Fix intermittent failure in ReleaseOverReservedPages on x64.

src/mark-compact.cc

Index: src/mark-compact.cc
===================================================================
--- src/mark-compact.cc	(revision 12290)
+++ src/mark-compact.cc	(working copy)
@@ -631,7 +631,8 @@
   intptr_t over_reserved = reserved - space->SizeOfObjects();
   static const intptr_t kFreenessThreshold = 50;
 
-  if (over_reserved >= 2 * space->AreaSize()) {
+  if (over_reserved >= 2 * space->AreaSize() ||
+      (reduce_memory_footprint_ && over_reserved >= space->AreaSize())) {

[Michael Starzinger, 2012/08/10 13:03:14]

Can we merge these two conditions into the two following if statements?

if (reduce_memory_footprint_ && over_reserved >= space->AreaSize()) {
  ...
}

if (over_reserved > reserved / 3 && over_reserved >= 2 * space->AreaSize()) {
  ...
}

     // If reduction of memory footprint was requested, we are aggressive
     // about choosing pages to free.  We expect that half-empty pages
     // are easier to compact so slightly bump the limit.