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

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/b5704bc1e6ba74c9ca975f5d58f5b2dc60261457
Cr-Commit-Position: refs/heads/master@{#36495}

[Michael Lippautz, 2016-05-24 09:52:37]

Patchset #3 (id:40001) has been deleted

[Michael Lippautz, 2016-05-24 09:53:34]

Patchset #3 (id:60001) has been deleted

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

Patchset #3 (id:80001) has been deleted

[Michael Lippautz, 2016-05-24 09:59:56]

Patchset #3 (id:100001) has been deleted

[Michael Lippautz, 2016-05-24 10:00:40]

Description was changed from

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

This reverts commit d2dff0320b75bf6054e46c379b192619d4884caf.

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

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-24 10:00:56]

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

[commit-bot: I haz the power, 2016-05-24 10:01:00]

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

[commit-bot: I haz the power, 2016-05-24 10:29:44]

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

[commit-bot: I haz the power, 2016-05-24 10:29:45]

Dry run: Try jobs failed on following builders:
  v8_linux64_asan_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_asan_rel_ng/buil...)
  v8_linux64_asan_rel_ng_triggered on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_asan_rel_ng_trig...)

[Michael Lippautz, 2016-05-24 10:36:17]

Patchset #3 (id:120001) has been deleted

[Michael Lippautz, 2016-05-24 10:36:50]

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

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

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-24 10:37:06]

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

[Michael Lippautz, 2016-05-24 10:37:07]

PTAL

[Michael Lippautz, 2016-05-24 10:37:12]

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

[commit-bot: I haz the power, 2016-05-24 10:37:21]

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

[commit-bot: I haz the power, 2016-05-24 12:25:51]

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

[commit-bot: I haz the power, 2016-05-24 12:25:52]

Dry run: Try jobs failed on following builders:
  v8_linux_dbg_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg_ng/builds/6248)
  v8_linux_dbg_ng_triggered on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg_ng_triggered/b...)

[Michael Lippautz, 2016-05-24 13:27:56]

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

[commit-bot: I haz the power, 2016-05-24 13:27:58]

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

[commit-bot: I haz the power, 2016-05-24 13:32:59]

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

[commit-bot: I haz the power, 2016-05-24 13:33:00]

Dry run: Try jobs failed on following builders:
  v8_linux_mips64el_compile_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mips64el_compile_r...)
  v8_linux_nodcheck_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_nodcheck_rel_ng/bu...)

[Michael Lippautz, 2016-05-24 13:36:32]

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

[commit-bot: I haz the power, 2016-05-24 13:36:40]

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

[Michael Lippautz, 2016-05-24 14:14:19]

Patchset #4 (id:160001) has been deleted

[commit-bot: I haz the power, 2016-05-24 14:18:05]

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

[commit-bot: I haz the power, 2016-05-24 14:18:07]

Dry run: This issue passed the CQ dry run.

[Hannes Payer (out of office), 2016-05-25 05:22:07]

lgtm

https://codereview.chromium.org/2005723005/diff/200001/src/heap/spaces.h
File src/heap/spaces.h (right):

https://codereview.chromium.org/2005723005/diff/200001/src/heap/spaces.h#newc...
src/heap/spaces.h:472: enum TrackerAccessMode { kDontCreate, kCreateIfNotPresent
};
Can we call it ArrayBufferTrackerAccessMode to associate it with array buffers?

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

https://codereview.chromium.org/2005723005/diff/200001/src/heap/spaces.h
File src/heap/spaces.h (right):

https://codereview.chromium.org/2005723005/diff/200001/src/heap/spaces.h#newc...
src/heap/spaces.h:472: enum TrackerAccessMode { kDontCreate, kCreateIfNotPresent
};
On 2016/05/25 05:22:07, Hannes Payer wrote:
> Can we call it ArrayBufferTrackerAccessMode to associate it with array
buffers?

Done.

[Michael Lippautz, 2016-05-25 06:39:35]

The CQ bit was checked by mlippautz@chromium.org

[Michael Lippautz, 2016-05-25 06:39:35]

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/2005723005/#ps220001
(title: "Addressed comment")

[commit-bot: I haz the power, 2016-05-25 06:39:40]

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

[commit-bot: I haz the power, 2016-05-25 07:26:40]

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

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

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 07:26:41]

Committed patchset #6 (id:220001)

[commit-bot: I haz the power, 2016-05-25 07:29:15]

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

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

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/b5704bc1e6ba74c9ca975f5d58f5b2dc60261457
Cr-Commit-Position: refs/heads/master@{#36495}
==========

[commit-bot: I haz the power, 2016-05-25 07:29:16]

Patchset 6 (id:??) landed as
https://crrev.com/b5704bc1e6ba74c9ca975f5d58f5b2dc60261457
Cr-Commit-Position: refs/heads/master@{#36495}

[Michael Lippautz, 2016-05-25 09:37:25]

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

The reason for reverting is: Breaking TSAN
 
https://uberchromegw.corp.google.com/i/client.v8/builders/V8%20Linux64%20TSAN....