• Leszek Swirski's avatar
    Revert "Reland "[v8] Use |AllocateAtLeast| for resizing v8 zones."" · 5110b3c6
    Leszek Swirski authored
    This reverts commit 2a7f86ed.
    
    Reason for revert: Blocking the roll (https://ci.chromium.org/ui/p/chromium/builders/try/mac-rel/b8802346196479466065/test-results?q=ExactID%3Aninja%3A%2F%2Fgin%3Agin_unittests%2FGinShellTest.HelloWorld+VHash%3A46648b293475ce2a&clean=)
    
    Original change's description:
    > Reland "[v8] Use |AllocateAtLeast| for resizing v8 zones."
    >
    > This is a reland of commit 4444874c
    > This fixes a failure on the UBSan bots caused by assuming |Realloc| is always in-place if the new size is <= the value of |malloc_usable_size|.
    >
    > Original change's description:
    > > [v8] Use |AllocateAtLeast| for resizing v8 zones.
    > >
    > > This is part of an ongoing effort to reduce fragmentation in Chrome.  Partition alloc shows v8 zones are a large user of memory in Renderer processes, and that there is fragmentation from these allocations. This CL will reduce this fragmentation by allowing v8 to use all allocated memory for its zones.
    > >
    > > Bug: v8:13193, chromium:1238858
    > > Change-Id: Ibeac8bdba9d0e7ff66b14a3dde10e7c87d3cf953
    > > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3889361
    > > Reviewed-by: Adam Klein <adamk@chromium.org>
    > > Commit-Queue: Thiabaud Engelbrecht <thiabaud@google.com>
    > > Reviewed-by: Michael Lippautz <mlippautz@chromium.org>
    > > Cr-Commit-Position: refs/heads/main@{#83235}
    >
    > Bug: v8:13193, chromium:1238858
    > Change-Id: I923bcbce8403dd7d84642340fd7202087b8a4440
    > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3910268
    > Reviewed-by: Adam Klein <adamk@chromium.org>
    > Commit-Queue: Thiabaud Engelbrecht <thiabaud@google.com>
    > Reviewed-by: Michael Lippautz <mlippautz@chromium.org>
    > Cr-Commit-Position: refs/heads/main@{#83372}
    
    Bug: v8:13193, chromium:1238858
    Change-Id: I32f9c4701a29c0516e3f587f1c6028e4d4158466
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3909358
    Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
    Auto-Submit: Leszek Swirski <leszeks@chromium.org>
    Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
    Owners-Override: Leszek Swirski <leszeks@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#83378}
    5110b3c6
allocation.cc 16.3 KB