Commit 58912650 authored by Leon Bettscheider's avatar Leon Bettscheider Committed by V8 LUCI CQ

[heap] IncrementalMarkingJob bailout for MinorMC

It is possible for IncrementalMarkingJob to be scheduled while MajorMC
is running, but it only gets to run after MajorMC finished.

If concurrent MinorMC is run in the meantime, RunInternal should not
invoke Step, which is currently only supported for MajorMC. This CL adds
a bailout for this case.

Bug: v8:13012
Change-Id: I3012cac3de5195a9f1b85f1ac18b02cef67b004b
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3867516
Commit-Queue: Leon Bettscheider <bettscheider@google.com>
Reviewed-by: 's avatarOmer Katz <omerkatz@chromium.org>
Cr-Commit-Position: refs/heads/main@{#82903}
parent eed7eaff
......@@ -94,7 +94,7 @@ void IncrementalMarkingJob::Task::RunInternal() {
job_->is_task_pending_ = false;
}
if (incremental_marking->IsMarking()) {
if (incremental_marking->IsMajorMarking()) {
// All objects are initialized at that point.
heap->new_space()->MarkLabStartInitialized();
heap->new_lo_space()->ResetPendingObject();
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment