Force relinking of paged space if first attempt to recommit from space fails.

Force relinking of paged space if first attempt to recommit from space fails.

That could improve chances for commit success as currently,
if we moved free pages out of order, we cannot shrink spaces.
However, when we experience problems commiting from space back, we should
use most of resources at our disposal.

Also get rid of currently unused parameter to DeallocateFunction.

Committed: http://code.google.com/p/v8/source/detail?r=5372

[antonm, 2010-08-27 12:43:15]

Slava,

may you have a look?

Maybe newly added function should be rather named EnsureInChunkOrder.

And maybe we should deallocate functions out of mark compactor and into spaces:
after all it is a space who knows how to deallocate a stuff from it.

[Vyacheslav Egorov (Chromium), 2010-08-27 14:14:46]

LGTM with comment addressed.

Thanks for cleaning debris left after me!

http://codereview.chromium.org/3260001/diff/1/4
File src/mark-compact.h (right):

http://codereview.chromium.org/3260001/diff/1/4#newcode122
src/mark-compact.h:122: static void DeallocateOldPointerBlock(Address start,
Maybe it worth turning this into virtual members of PagedSpace?

There is nothing MarkCompactCollector specific in them.

[Vyacheslav Egorov (Chromium), 2010-08-27 14:16:29]

http://codereview.chromium.org/3260001/diff/1/2
File src/heap.cc (right):

http://codereview.chromium.org/3260001/diff/1/2#newcode544
src/heap.cc:544: Heap::old_pointer_space()->RelinkPageListInChunkOrder(
After turning block deallocation into virtual method. This might be turned into
simple loop over paged spaces.