[heap] Use live memory as heuristic for spawning compaction tasks

[heap] Use live memory as heuristic for spawning compaction tasks

R=hpayer@chromium.org
BUG=chromium:524425
LOG=N

Committed: https://crrev.com/88b764d7f74008fe5f30d59e3881e374d538b72e
Cr-Commit-Position: refs/heads/master@{#31725}

[Hannes Payer (out of office), 2015-11-02 23:44:38]

lgtm

https://codereview.chromium.org/1410163005/diff/1/src/heap/mark-compact.cc
File src/heap/mark-compact.cc (right):

https://codereview.chromium.org/1410163005/diff/1/src/heap/mark-compact.cc#ne...
src/heap/mark-compact.cc:3316: const int kLiveMemoryPerCompactionTask = 2 *
Page::kPageSize;
We could get a better bound here if we would use online profiling to determine
compaction throughput. Maybe leave a TODO here and investigate (or fix) this
later.

[Michael Lippautz, 2015-11-02 23:48:48]

https://codereview.chromium.org/1410163005/diff/1/src/heap/mark-compact.cc
File src/heap/mark-compact.cc (right):

https://codereview.chromium.org/1410163005/diff/1/src/heap/mark-compact.cc#ne...
src/heap/mark-compact.cc:3316: const int kLiveMemoryPerCompactionTask = 2 *
Page::kPageSize;
On 2015/11/02 23:44:38, Hannes Payer wrote:
> We could get a better bound here if we would use online profiling to determine
> compaction throughput. Maybe leave a TODO here and investigate (or fix) this
> later.

Done.

[Michael Lippautz, 2015-11-02 23:48:55]

The CQ bit was checked by mlippautz@chromium.org

[Michael Lippautz, 2015-11-02 23:48:56]

The patchset sent to the CQ was uploaded after l-g-t-m from hpayer@chromium.org
Link to the patchset: https://codereview.chromium.org/1410163005/#ps20001
(title: "Added comment")

[commit-bot: I haz the power, 2015-11-02 23:49:01]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1410163005/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1410163005/20001

[commit-bot: I haz the power, 2015-11-03 00:04:14]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-11-03 00:04:58]

Patchset 2 (id:??) landed as
https://crrev.com/88b764d7f74008fe5f30d59e3881e374d538b72e
Cr-Commit-Position: refs/heads/master@{#31725}