[heap] Simplify heuristics for incremental step size.

[heap] Simplify heuristics for incremental step size.

This patch removes the code for speeding up marking.

Now the step size depends on allocated bytes and
invoked write barriers.

We also ensure that the step size is large enough to
justify the overhead of interrupting the generated code.

BUG=chromium:616434
LOG=NO

Committed: https://crrev.com/71a793e4b1026e69d4009df0a419fe260fe1a235
Cr-Commit-Position: refs/heads/master@{#39067}

[ulan, 2016-08-31 16:55:47]

Description was changed from

==========
Marking speed

BUG=
==========

to

==========
[heap] Simplify heuristics of incremental step size.

This patch removes the code for speeding up marking.

Now we ensure that an incremental step takes at least 1ms
to justify overhead of interrupting the generated code.

Besides this lower bound, the step size depends only
on allocated bytes and invoked write barriers.

BUG=chromium:616434
LOG=NO
==========

[ulan, 2016-08-31 16:55:48]

ulan@chromium.org changed reviewers:
+ mlippautz@chromium.org

[ulan, 2016-08-31 16:56:57]

Description was changed from

==========
[heap] Simplify heuristics of incremental step size.

This patch removes the code for speeding up marking.

Now we ensure that an incremental step takes at least 1ms
to justify overhead of interrupting the generated code.

Besides this lower bound, the step size depends only
on allocated bytes and invoked write barriers.

BUG=chromium:616434
LOG=NO
==========

to

==========
[heap] Simplify heuristics for incremental step size.

This patch removes the code for speeding up marking.

Now we ensure that an incremental step takes at least 1ms
to justify overhead of interrupting the generated code.

Besides this lower bound, the step size depends only
on allocated bytes and invoked write barriers.

BUG=chromium:616434
LOG=NO
==========

[ulan, 2016-08-31 16:58:12]

Description was changed from

==========
[heap] Simplify heuristics for incremental step size.

This patch removes the code for speeding up marking.

Now we ensure that an incremental step takes at least 1ms
to justify overhead of interrupting the generated code.

Besides this lower bound, the step size depends only
on allocated bytes and invoked write barriers.

BUG=chromium:616434
LOG=NO
==========

to

==========
[heap] Simplify heuristics for incremental step size.

This patch removes the code for speeding up marking.

Now the step size depends only on allocated bytes and
invoked write barriers.

We also ensure that the step size is large enough to
justify the overhead of interrupt the generated code.

BUG=chromium:616434
LOG=NO
==========

[ulan, 2016-08-31 16:58:40]

Description was changed from

==========
[heap] Simplify heuristics for incremental step size.

This patch removes the code for speeding up marking.

Now the step size depends only on allocated bytes and
invoked write barriers.

We also ensure that the step size is large enough to
justify the overhead of interrupt the generated code.

BUG=chromium:616434
LOG=NO
==========

to

==========
[heap] Simplify heuristics for incremental step size.

This patch removes the code for speeding up marking.

Now the step size depends on allocated bytes and
invoked write barriers.

We also ensure that the step size is large enough to
justify the overhead of interrupt the generated code.

BUG=chromium:616434
LOG=NO
==========

[ulan, 2016-08-31 17:03:15]

Patchset #1 (id:1) has been deleted

[ulan, 2016-08-31 17:03:22]

Patchset #1 (id:20001) has been deleted

[ulan, 2016-08-31 17:03:29]

Patchset #1 (id:40001) has been deleted

[ulan, 2016-08-31 17:03:39]

Patchset #1 (id:60001) has been deleted

[ulan, 2016-08-31 17:03:45]

Patchset #1 (id:80001) has been deleted

[ulan, 2016-08-31 17:04:50]

ptal

[ulan, 2016-08-31 17:05:28]

Description was changed from

==========
[heap] Simplify heuristics for incremental step size.

This patch removes the code for speeding up marking.

Now the step size depends on allocated bytes and
invoked write barriers.

We also ensure that the step size is large enough to
justify the overhead of interrupt the generated code.

BUG=chromium:616434
LOG=NO
==========

to

==========
[heap] Simplify heuristics for incremental step size.

This patch removes the code for speeding up marking.

Now the step size depends on allocated bytes and
invoked write barriers.

We also ensure that the step size is large enough to
justify the overhead of interrupting the generated code.

BUG=chromium:616434
LOG=NO
==========

[Michael Lippautz, 2016-08-31 17:10:17]

lgtm, fingers crossed that it doesn't regress :)

[ulan, 2016-09-01 08:05:21]

The CQ bit was checked by ulan@chromium.org

[commit-bot: I haz the power, 2016-09-01 08:05:25]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-01 08:32:01]

Description was changed from

==========
[heap] Simplify heuristics for incremental step size.

This patch removes the code for speeding up marking.

Now the step size depends on allocated bytes and
invoked write barriers.

We also ensure that the step size is large enough to
justify the overhead of interrupting the generated code.

BUG=chromium:616434
LOG=NO
==========

to

==========
[heap] Simplify heuristics for incremental step size.

This patch removes the code for speeding up marking.

Now the step size depends on allocated bytes and
invoked write barriers.

We also ensure that the step size is large enough to
justify the overhead of interrupting the generated code.

BUG=chromium:616434
LOG=NO
==========

[commit-bot: I haz the power, 2016-09-01 08:32:02]

Committed patchset #5 (id:180001)

[commit-bot: I haz the power, 2016-09-01 08:32:42]

Description was changed from

==========
[heap] Simplify heuristics for incremental step size.

This patch removes the code for speeding up marking.

Now the step size depends on allocated bytes and
invoked write barriers.

We also ensure that the step size is large enough to
justify the overhead of interrupting the generated code.

BUG=chromium:616434
LOG=NO
==========

to

==========
[heap] Simplify heuristics for incremental step size.

This patch removes the code for speeding up marking.

Now the step size depends on allocated bytes and
invoked write barriers.

We also ensure that the step size is large enough to
justify the overhead of interrupting the generated code.

BUG=chromium:616434
LOG=NO

Committed: https://crrev.com/71a793e4b1026e69d4009df0a419fe260fe1a235
Cr-Commit-Position: refs/heads/master@{#39067}
==========

[commit-bot: I haz the power, 2016-09-01 08:32:42]

Patchset 5 (id:??) landed as
https://crrev.com/71a793e4b1026e69d4009df0a419fe260fe1a235
Cr-Commit-Position: refs/heads/master@{#39067}

[ulan, 2016-09-01 17:28:30]

A revert of this CL (patchset #5 id:180001) has been created in
https://codereview.chromium.org/2304613002/ by ulan@chromium.org.

The reason for reverting is: Regressed GC times in v8.infinite_scroll_tbmv2
benchmarks..