Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(719)

Issue 11413183: Fix incremental marking to be off during heap snapshots. (Closed)

Created:
8 years ago by Michael Starzinger
Modified:
8 years ago
Reviewers:
Toon Verwaest
CC:
v8-dev
Visibility:
Public.

Description

Fix incremental marking to be off during heap snapshots. The heap snapshot generator uses the UnreachableObjectsFilter which in turn messes with marking bits. This requires incremental marking to be turned off while the snapshot is being generated. R=verwaest@chromium.org TEST=cctest/test-heap-profiler --stress-compaction Committed: https://code.google.com/p/v8/source/detail?r=13066

Patch Set 1 #

Total comments: 2

Patch Set 2 : Addressed comments by Toon Verwaest. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -5 lines) Patch
M src/heap.cc View 1 2 chunks +8 lines, -5 lines 0 comments Download
M src/mark-compact.h View 1 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Michael Starzinger
8 years ago (2012-11-27 11:32:28 UTC) #1
Toon Verwaest
lgtm with nit https://codereview.chromium.org/11413183/diff/1/src/heap.cc File src/heap.cc (right): https://codereview.chromium.org/11413183/diff/1/src/heap.cc#newcode662 src/heap.cc:662: if (!mark_compact_collector()->abort_incremental_marking_ && Please make a ...
8 years ago (2012-11-27 12:07:26 UTC) #2
Michael Starzinger
8 years ago (2012-11-27 13:18:10 UTC) #3
Addressed comments. Landing.

https://codereview.chromium.org/11413183/diff/1/src/heap.cc
File src/heap.cc (right):

https://codereview.chromium.org/11413183/diff/1/src/heap.cc#newcode662
src/heap.cc:662: if (!mark_compact_collector()->abort_incremental_marking_ &&
On 2012/11/27 12:07:26, Toon Verwaest wrote:
> Please make a protected accessor for abort_incremental_marking_;, and maybe
give
> it a name that describes that incremental marking should not be turned on
again.

Done.

Powered by Google App Engine
This is Rietveld 408576698