• Dominik Inführ's avatar
    Revert "Use list of invalidated objects for old-to-new refs" · 9a9ba762
    Dominik Inführ authored
    This reverts commit 60843b42.
    
    Reason for revert: TSAN detected issue between Scavenge workers.
    One task could invoke RefillFreeList(), while the other task iterates the remembered set of a swept page.
    
    Original change's description:
    > Use list of invalidated objects for old-to-new refs
    > 
    > Instead of inserting "deletion" entries into the store buffer, keep a
    > list of invalidated objects to filter out invalid old-to-new slots.
    > 
    > The first CL https://crrev.com/c/1704109 got reverted because both the
    > sweeper and the main task were modifying the invalidated slots data
    > structure concurrently. This CL changes this, such that the sweeper
    > only modifies the invalidated slots during the final atomic pause when
    > the main thread is not running. The sweeper does not need to clean this
    > data structure after the pause, since the "update pointers" phase
    > already removed all invalidated slots.
    > 
    > The second CL https://crrev.com/c/1733081 got reverted because the
    > sweeper might find more free space than the full GC before it. If an
    > object shrinks after the pause but before the sweep, the invalidated
    > object might span free memory and potentially new allocated objects.
    > Therefore shrink invalidated objects when processing swept pages on
    > the main thread. Also clean recorded slots in the gap.
    > 
    > TBR=petermarshall@chromium.org
    > 
    > Bug: v8:9454
    > Change-Id: I80d1fa3bbc24e97f7c97a373aaad66f105456f12
    > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1751795
    > Commit-Queue: Dominik Inführ <dinfuehr@chromium.org>
    > Reviewed-by: Ulan Degenbaev <ulan@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#63239}
    
    TBR=ulan@chromium.org,hpayer@chromium.org,dinfuehr@chromium.org
    
    Change-Id: I9c6a371ebe36a1873acbe0d6c6a75dd2f5a55f4e
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: v8:9454
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1760817Reviewed-by: 's avatarDominik Inführ <dinfuehr@chromium.org>
    Commit-Queue: Dominik Inführ <dinfuehr@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#63253}
    9a9ba762
Name
Last commit
Last update
..
OWNERS Loading commit data...
array-buffer-collector.cc Loading commit data...
array-buffer-collector.h Loading commit data...
array-buffer-tracker-inl.h Loading commit data...
array-buffer-tracker.cc Loading commit data...
array-buffer-tracker.h Loading commit data...
barrier.h Loading commit data...
basic-memory-chunk.cc Loading commit data...
basic-memory-chunk.h Loading commit data...
code-stats.cc Loading commit data...
code-stats.h Loading commit data...
combined-heap.cc Loading commit data...
combined-heap.h Loading commit data...
concurrent-marking.cc Loading commit data...
concurrent-marking.h Loading commit data...
embedder-tracing.cc Loading commit data...
embedder-tracing.h Loading commit data...
factory-inl.h Loading commit data...
factory.cc Loading commit data...
factory.h Loading commit data...
gc-idle-time-handler.cc Loading commit data...
gc-idle-time-handler.h Loading commit data...
gc-tracer.cc Loading commit data...
gc-tracer.h Loading commit data...
heap-controller.cc Loading commit data...
heap-controller.h Loading commit data...
heap-inl.h Loading commit data...
heap-write-barrier-inl.h Loading commit data...
heap-write-barrier.h Loading commit data...
heap.cc Loading commit data...
heap.h Loading commit data...
incremental-marking-inl.h Loading commit data...
incremental-marking-job.cc Loading commit data...
incremental-marking-job.h Loading commit data...
incremental-marking.cc Loading commit data...
incremental-marking.h Loading commit data...
invalidated-slots-inl.h Loading commit data...
invalidated-slots.cc Loading commit data...
invalidated-slots.h Loading commit data...
item-parallel-job.cc Loading commit data...
item-parallel-job.h Loading commit data...
local-allocator-inl.h Loading commit data...
local-allocator.h Loading commit data...
mark-compact-inl.h Loading commit data...
mark-compact.cc Loading commit data...
mark-compact.h Loading commit data...
marking.cc Loading commit data...
marking.h Loading commit data...
memory-reducer.cc Loading commit data...
memory-reducer.h Loading commit data...
object-stats.cc Loading commit data...
object-stats.h Loading commit data...
objects-visiting-inl.h Loading commit data...
objects-visiting.cc Loading commit data...
objects-visiting.h Loading commit data...
read-only-heap-inl.h Loading commit data...
read-only-heap.cc Loading commit data...
read-only-heap.h Loading commit data...
remembered-set.h Loading commit data...
scavenge-job.cc Loading commit data...
scavenge-job.h Loading commit data...
scavenger-inl.h Loading commit data...
scavenger.cc Loading commit data...
scavenger.h Loading commit data...
setup-heap-internal.cc Loading commit data...
slot-set.cc Loading commit data...
slot-set.h Loading commit data...
spaces-inl.h Loading commit data...
spaces.cc Loading commit data...
spaces.h Loading commit data...
store-buffer-inl.h Loading commit data...
store-buffer.cc Loading commit data...
store-buffer.h Loading commit data...
stress-marking-observer.cc Loading commit data...
stress-marking-observer.h Loading commit data...
stress-scavenge-observer.cc Loading commit data...
stress-scavenge-observer.h Loading commit data...
sweeper.cc Loading commit data...
sweeper.h Loading commit data...
worklist.h Loading commit data...