Changeset 295703 in webkit for trunk/Source/JavaScriptCore/heap/Heap.h

Timestamp:

Jun 21, 2022, 4:57:05 PM (3 years ago)

Author:

[email protected]

Message:

Unreviewed, revert "The extraMemorySize() get wrong when transferring ArrayBuffer from Worker VM"
​https://bugs.webkit.org/show_bug.cgi?id=241826
rdar://95384643

This reverts commit 71960bed2a3ee0917367bc4144911a9e8168deea.

m_extraMemorySize must be monotonically increasing during GC cycles until
full-collection happens. And after the full-collection, it is adjusted.
We already adjusted it in sweep of m_arrayBuffer, so, we should not reduce
that number. This is used for GC invocation scheduling. So, if we would like to
have a number which more precisely reflecting the current status,
then we should have yet another one. And we can still use extraMemorySize
since it will be adjusted after the full-collection. So we can consider
that transferred array-buffer is collected at the full-collection.

Canonical link: ​https://commits.webkit.org/251708@main

File:

• trunk/Source/JavaScriptCore/heap/Heap.h (modified) (1 diff)

trunk/Source/JavaScriptCore/heap/Heap.h

@@ -440 +440 @@
     
     void addReference(JSCell*, ArrayBuffer*);
-    void reduceArrayBufferSize(size_t bytes);
     
     bool isDeferred() const { return !!m_deferralDepth; }