• Dominik Inführ's avatar
    Revert "[test] Avoid disabling of RCS during runtime" · 0feba98a
    Dominik Inführ authored
    This reverts commit 11045926.
    
    Reason for revert: Uncovered TSAN failure
    
    Original change's description:
    > [test] Avoid disabling of RCS during runtime
    >
    > Test was flaky since disabling of RCS at runtime is unsafe. Some code
    > (e.g. TRACE_GC) is run only in case RCS is enabled and such code paths
    > might also DCHECK that RCS is enabled.
    > A background thread (sweeping in this case) could've already
    > entered such a code path right before RCS is disabled. In this case the
    > guard at the entry still saw that RCS is enabled but subsequent DCHECKs
    > that ensure that RCS is enabled might fail.
    >
    > We could fix this by completing the sweeper tasks before disabling RCS
    > but there might be other similar tasks which might be run at that point
    > in the future (e.g. memory unmapper).
    >
    > Disabling the flag doesn't even seem to be needed for the initial
    > regression test, so it seems simpler to just not disable RCS anymore.
    >
    > Bug: v8:12026
    > Change-Id: I878f7dd9a7a4abb6a501f7a7651a1240ef2082a6
    > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3162043
    > Reviewed-by: Michael Lippautz <mlippautz@chromium.org>
    > Commit-Queue: Dominik Inführ <dinfuehr@chromium.org>
    > Cr-Commit-Position: refs/heads/main@{#76845}
    
    Bug: v8:12026
    Change-Id: I154817fc5ddeab68c69c745d08f3d77c48896dc4
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3162049
    Auto-Submit: Dominik Inführ <dinfuehr@chromium.org>
    Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
    Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
    Cr-Commit-Position: refs/heads/main@{#76848}
    0feba98a
Name
Last commit
Last update
..
heap-tester.h Loading commit data...
heap-utils.cc Loading commit data...
heap-utils.h Loading commit data...
test-alloc.cc Loading commit data...
test-array-buffer-tracker.cc Loading commit data...
test-compaction.cc Loading commit data...
test-concurrent-allocation.cc Loading commit data...
test-concurrent-marking.cc Loading commit data...
test-embedder-tracing.cc Loading commit data...
test-external-string-tracker.cc Loading commit data...
test-heap.cc Loading commit data...
test-incremental-marking.cc Loading commit data...
test-invalidated-slots.cc Loading commit data...
test-iterators.cc Loading commit data...
test-lab.cc Loading commit data...
test-mark-compact.cc Loading commit data...
test-memory-measurement.cc Loading commit data...
test-page-promotion.cc Loading commit data...
test-shared-heap.cc Loading commit data...
test-spaces.cc Loading commit data...
test-unmapper.cc Loading commit data...
test-weak-references.cc Loading commit data...
test-write-barrier.cc Loading commit data...