• Ulan Degenbaev's avatar
    Revert "Reland x5 [arraybuffer] Rearchitect backing store ownership" · 9da34831
    Ulan Degenbaev authored
    This reverts commit 62e16830.
    
    Reason for revert: it will be relanded after branch
    
    Original change's description:
    > Reland x5 [arraybuffer] Rearchitect backing store ownership
    > 
    > This reverts commit 8fdb2387.
    > 
    > Original change's description:
    > > "Reland x4 [arraybuffer] Rearchitect backing store ownership"
    > >
    > > This is a reland of bc33f5ae
    > >
    > > Contributed by titzer@chromium.org
    > >
    > > Original change's description:
    > > > [arraybuffer] Rearchitect backing store ownership
    > > >
    > > > This CL completely rearchitects the ownership of array buffer backing stores,
    > > > consolidating ownership into a {BackingStore} C++ object that is tracked
    > > > throughout V8 using unique_ptr and shared_ptr where appropriate.
    > > >
    > > > Overall, lifetime management is simpler and more explicit. The numerous
    > > > ways that array buffers were initialized have been streamlined to one
    > > > Attach() method on JSArrayBuffer. The array buffer tracker in the
    > > > GC implementation now manages std::shared_ptr<BackingStore> pointers,
    > > > and the construction and destruction of the BackingStore object itself
    > > > handles the underlying page or embedder-allocated memory.
    > > >
    > > > The embedder API remains unchanged for now. We use the
    > > > v8::ArrayBuffer::Contents struct to hide an additional shared_ptr to
    > > > keep the backing store alive properly, even in the case of aliases
    > > > from live heap objects. Thus the embedder has a lower chance of making
    > > > a mistake. Long-term, we should move the embedder to a model where they
    > > > manage backing stores using shared_ptr to an opaque backing store object.
    > >
    > > TBR=yangguo@chromium.org
    > >
    > > BUG=v8:9380,v8:9221,chromium:986318
    > >
    > > Change-Id: If671a4a9ca0476e8f084efae46e0d2bf99ed99ef
    > > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1731005
    > > Commit-Queue: Ulan Degenbaev <ulan@chromium.org>
    > > Reviewed-by: Clemens Hammacher <clemensh@chromium.org>
    > > Reviewed-by: Michael Starzinger <mstarzinger@chromium.org>
    > > Cr-Commit-Position: refs/heads/master@{#63041}
    > 
    > TBR=yangguo@chromium.org,clemensh@chromium.org,mstarzinger@chromium.org
    > 
    > Change-Id: Iba55c7ab71e5642b5cb6aeb699d6fc9cf9061486
    > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1771795
    > Reviewed-by: Michael Lippautz <mlippautz@chromium.org>
    > Commit-Queue: Ulan Degenbaev <ulan@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#63461}
    
    TBR=ulan@chromium.org,mlippautz@chromium.org
    
    Change-Id: Id8f67a68ab398032eb2975b1b24ee125394d9c4b
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1776095Reviewed-by: 's avatarUlan Degenbaev <ulan@chromium.org>
    Commit-Queue: Ulan Degenbaev <ulan@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#63471}
    9da34831
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...