Reland "[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.

BUG=chromium:611688
LOG=N
TEST=cctest/test-array-buffer-tracker/*
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/089da007bb990b5b29aab257aa836fdd3f3b8ce0
Cr-Commit-Position: refs/heads/master@{#36608}

[Michael Lippautz, 2016-05-31 07:33:15]

Description was changed from

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

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

to

==========
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.

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-31 07:34:02]

Description was changed from

==========
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.

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

==========
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.

BUG=chromium:611688
LOG=N
TEST=test/cctest/test-array-buffer/tracker/*
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-31 07:34:11]

Description was changed from

==========
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.

BUG=chromium:611688
LOG=N
TEST=test/cctest/test-array-buffer/tracker/*
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

==========
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.

BUG=chromium:611688
LOG=N
TEST=test/cctest/test-array-buffer-tracker/*
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-31 07:34:30]

Description was changed from

==========
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.

BUG=chromium:611688
LOG=N
TEST=test/cctest/test-array-buffer-tracker/*
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

==========
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.

BUG=chromium:611688
LOG=N
TEST=cctest/test-array-buffer-tracker/*
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-31 07:35:13]

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

[Michael Lippautz, 2016-05-31 07:55:48]

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

[commit-bot: I haz the power, 2016-05-31 07:55:59]

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

[Hannes Payer (out of office), 2016-05-31 08:03:51]

https://codereview.chromium.org/2024063002/diff/20001/src/heap/array-buffer-t...
File src/heap/array-buffer-tracker-inl.h (right):

https://codereview.chromium.org/2024063002/diff/20001/src/heap/array-buffer-t...
src/heap/array-buffer-tracker-inl.h:24: if
(Marking::IsBlack(Marking::MarkBitFrom(new_buffer))) {
Is Black or Grey?

[Hannes Payer (out of office), 2016-05-31 08:15:41]

LGTM

[Michael Lippautz, 2016-05-31 08:15:51]

Patchset #3 (id:40001) has been deleted

[Michael Lippautz, 2016-05-31 08:17:06]

https://codereview.chromium.org/2024063002/diff/20001/src/heap/array-buffer-t...
File src/heap/array-buffer-tracker-inl.h (right):

https://codereview.chromium.org/2024063002/diff/20001/src/heap/array-buffer-t...
src/heap/array-buffer-tracker-inl.h:24: if
(Marking::IsBlack(Marking::MarkBitFrom(new_buffer))) {
On 2016/05/31 08:03:50, Hannes Payer wrote:
> Is Black or Grey?

As discussed offline: Grey means it's on the marking dequeue but not yet marked.
The visitor will catch it later on and mark it.

[Michael Lippautz, 2016-05-31 08:17:07]

https://codereview.chromium.org/2024063002/diff/20001/src/heap/array-buffer-t...
File src/heap/array-buffer-tracker-inl.h (right):

https://codereview.chromium.org/2024063002/diff/20001/src/heap/array-buffer-t...
src/heap/array-buffer-tracker-inl.h:24: if
(Marking::IsBlack(Marking::MarkBitFrom(new_buffer))) {
On 2016/05/31 08:03:50, Hannes Payer wrote:
> Is Black or Grey?

As discussed offline: Grey means it's on the marking dequeue but not yet marked.
The visitor will catch it later on and mark it.

[Michael Lippautz, 2016-05-31 09:14:26]

The CQ bit was checked by mlippautz@chromium.org

[commit-bot: I haz the power, 2016-05-31 09:14:28]

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

[commit-bot: I haz the power, 2016-05-31 09:16:33]

Description was changed from

==========
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.

BUG=chromium:611688
LOG=N
TEST=cctest/test-array-buffer-tracker/*
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

==========
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.

BUG=chromium:611688
LOG=N
TEST=cctest/test-array-buffer-tracker/*
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-31 09:16:34]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-31 09:17:05]

Description was changed from

==========
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.

BUG=chromium:611688
LOG=N
TEST=cctest/test-array-buffer-tracker/*
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

==========
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.

BUG=chromium:611688
LOG=N
TEST=cctest/test-array-buffer-tracker/*
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/089da007bb990b5b29aab257aa836fdd3f3b8ce0
Cr-Commit-Position: refs/heads/master@{#36608}
==========

[commit-bot: I haz the power, 2016-05-31 09:17:06]

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

[Michael Lippautz, 2016-05-31 09:30:50]

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

The reason for reverting is: Breaking
 
http://build.chromium.org/p/client.v8/builders/V8%20Linux64%20-%20avx2/builds....