Reland of "[heap] Fine-grained JSArrayBuffer tracking"

Reland of "[heap] Fine-grained JSArrayBuffer tracking"

Track based on JSArrayBuffer addresses on pages instead of the attached
backing store.

Details of tracking:
- Scavenge: New space pages are processes in bulk on the main thread
- MC: Unswept pages are processed in bulk in parallel. All other pages
  are processed by the sweeper concurrently.

This reverts commit 93fe04afa9428f4f68dc5dd302cda83908be84a7.

R=hpayer@chromium.org
BUG=chromium:611688
LOG=N
CQ_EXTRA_TRYBOTS=tryserver.v8:v8_linux_arm64_gc_stress_dbg,v8_linux_gc_stress_dbg,v8_mac_gc_stress_dbg,v8_linux64_tsan_rel,v8_mac64_asan_rel

Committed: https://crrev.com/863d015e573798e263b35b31d2ab979739706533
Cr-Commit-Position: refs/heads/master@{#36513}

[Michael Lippautz, 2016-05-25 10:49:23]

Description was changed from

==========
Reland of "[heap] Fine-grained JSArrayBuffer tracking"

This reverts commit 93fe04afa9428f4f68dc5dd302cda83908be84a7.

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

to

==========
Reland of "[heap] Fine-grained JSArrayBuffer tracking"

Track based on JSArrayBuffer addresses on pages instead of the attached
backing store.

Details of tracking:
- Scavenge: New space pages are processes in bulk on the main thread
- MC: Unswept pages are processed in bulk in parallel. All other pages
  are processed by the sweeper concurrently.

This reverts commit 93fe04afa9428f4f68dc5dd302cda83908be84a7.

R=hpayer@chromium.org
BUG=chromium:611688
LOG=N
CQ_EXTRA_TRYBOTS=tryserver.v8:v8_linux_arm64_gc_stress_dbg,v8_linux_gc_stress_dbg,v8_mac_gc_stress_dbg,v8_linux64_tsan_rel,v8_mac64_asan_rel
==========

[Michael Lippautz, 2016-05-25 10:49:23]

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

[Michael Lippautz, 2016-05-25 10:50:44]

Patchset #2 (id:20001) has been deleted

[Michael Lippautz, 2016-05-25 10:52:30]

Patchset #2 (id:40001) has been deleted

[Michael Lippautz, 2016-05-25 10:53:27]

https://codereview.chromium.org/2007253002/diff/60001/src/heap/array-buffer-t...
File src/heap/array-buffer-tracker.cc (right):

https://codereview.chromium.org/2007253002/diff/60001/src/heap/array-buffer-t...
src/heap/array-buffer-tracker.cc:164: void ArrayBufferTracker::FreeDead(Page*
page) {
Called from sweeper which already holds the page lock.

[Hannes Payer (out of office), 2016-05-25 10:55:26]

lgtm

https://codereview.chromium.org/2007253002/diff/60001/src/heap/array-buffer-t...
File src/heap/array-buffer-tracker.cc (right):

https://codereview.chromium.org/2007253002/diff/60001/src/heap/array-buffer-t...
src/heap/array-buffer-tracker.cc:164: void ArrayBufferTracker::FreeDead(Page*
page) {
On 2016/05/25 10:53:26, Michael Lippautz wrote:
> Called from sweeper which already holds the page lock.

Please add that as a comment.

[Michael Lippautz, 2016-05-25 10:56:24]

https://codereview.chromium.org/2007253002/diff/60001/src/heap/array-buffer-t...
File src/heap/array-buffer-tracker.cc (right):

https://codereview.chromium.org/2007253002/diff/60001/src/heap/array-buffer-t...
src/heap/array-buffer-tracker.cc:164: void ArrayBufferTracker::FreeDead(Page*
page) {
On 2016/05/25 10:55:26, Hannes Payer wrote:
> On 2016/05/25 10:53:26, Michael Lippautz wrote:
> > Called from sweeper which already holds the page lock.
> 
> Please add that as a comment.

Done.

[Michael Lippautz, 2016-05-25 10:56:42]

The CQ bit was checked by mlippautz@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-05-25 10:56:46]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2007253002/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2007253002/80001

[commit-bot: I haz the power, 2016-05-25 11:41:52]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-05-25 11:41:53]

Dry run: This issue passed the CQ dry run.

[commit-bot: I haz the power, 2016-05-25 11:42:01]

Dry run: None

[Michael Lippautz, 2016-05-25 12:06:46]

The CQ bit was checked by mlippautz@chromium.org

[Michael Lippautz, 2016-05-25 12:06:47]

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/2007253002/#ps80001
(title: "Addressed coment")

[commit-bot: I haz the power, 2016-05-25 12:06:53]

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

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

Description was changed from

==========
Reland of "[heap] Fine-grained JSArrayBuffer tracking"

Track based on JSArrayBuffer addresses on pages instead of the attached
backing store.

Details of tracking:
- Scavenge: New space pages are processes in bulk on the main thread
- MC: Unswept pages are processed in bulk in parallel. All other pages
  are processed by the sweeper concurrently.

This reverts commit 93fe04afa9428f4f68dc5dd302cda83908be84a7.

R=hpayer@chromium.org
BUG=chromium:611688
LOG=N
CQ_EXTRA_TRYBOTS=tryserver.v8:v8_linux_arm64_gc_stress_dbg,v8_linux_gc_stress_dbg,v8_mac_gc_stress_dbg,v8_linux64_tsan_rel,v8_mac64_asan_rel
==========

to

==========
Reland of "[heap] Fine-grained JSArrayBuffer tracking"

Track based on JSArrayBuffer addresses on pages instead of the attached
backing store.

Details of tracking:
- Scavenge: New space pages are processes in bulk on the main thread
- MC: Unswept pages are processed in bulk in parallel. All other pages
  are processed by the sweeper concurrently.

This reverts commit 93fe04afa9428f4f68dc5dd302cda83908be84a7.

R=hpayer@chromium.org
BUG=chromium:611688
LOG=N
CQ_EXTRA_TRYBOTS=tryserver.v8:v8_linux_arm64_gc_stress_dbg,v8_linux_gc_stress_dbg,v8_mac_gc_stress_dbg,v8_linux64_tsan_rel,v8_mac64_asan_rel
==========

[commit-bot: I haz the power, 2016-05-25 12:09:02]

Committed patchset #3 (id:80001)

[commit-bot: I haz the power, 2016-05-25 12:11:21]

Description was changed from

==========
Reland of "[heap] Fine-grained JSArrayBuffer tracking"

Track based on JSArrayBuffer addresses on pages instead of the attached
backing store.

Details of tracking:
- Scavenge: New space pages are processes in bulk on the main thread
- MC: Unswept pages are processed in bulk in parallel. All other pages
  are processed by the sweeper concurrently.

This reverts commit 93fe04afa9428f4f68dc5dd302cda83908be84a7.

R=hpayer@chromium.org
BUG=chromium:611688
LOG=N
CQ_EXTRA_TRYBOTS=tryserver.v8:v8_linux_arm64_gc_stress_dbg,v8_linux_gc_stress_dbg,v8_mac_gc_stress_dbg,v8_linux64_tsan_rel,v8_mac64_asan_rel
==========

to

==========
Reland of "[heap] Fine-grained JSArrayBuffer tracking"

Track based on JSArrayBuffer addresses on pages instead of the attached
backing store.

Details of tracking:
- Scavenge: New space pages are processes in bulk on the main thread
- MC: Unswept pages are processed in bulk in parallel. All other pages
  are processed by the sweeper concurrently.

This reverts commit 93fe04afa9428f4f68dc5dd302cda83908be84a7.

R=hpayer@chromium.org
BUG=chromium:611688
LOG=N
CQ_EXTRA_TRYBOTS=tryserver.v8:v8_linux_arm64_gc_stress_dbg,v8_linux_gc_stress_dbg,v8_mac_gc_stress_dbg,v8_linux64_tsan_rel,v8_mac64_asan_rel

Committed: https://crrev.com/863d015e573798e263b35b31d2ab979739706533
Cr-Commit-Position: refs/heads/master@{#36513}
==========

[commit-bot: I haz the power, 2016-05-25 12:11:22]

Patchset 3 (id:??) landed as
https://crrev.com/863d015e573798e263b35b31d2ab979739706533
Cr-Commit-Position: refs/heads/master@{#36513}

[Michael Lippautz, 2016-05-25 16:29:24]

A revert of this CL (patchset #3 id:80001) has been created in
https://codereview.chromium.org/2011563003/ by mlippautz@chromium.org.

The reason for reverting is: Breaks webgl2 conformance tests
  https://bugs.chromium.org/p/chromium/issues/detail?id=614730.