• Adam Klein's avatar
    Revert "[v8] Use |AllocateAtLeast| for resizing v8 zones." · 59c070a2
    Adam Klein authored
    This reverts commit 4444874c.
    
    Reason for revert: CHECK failure under UBSan
    https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Linux64%20UBSan%20-%20builder/5103/overview
    
    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: I03c8c1ad7bb1cd20770323bffe1c42a4be47c454
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3900814
    Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
    Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
    Auto-Submit: Adam Klein <adamk@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#83237}
    59c070a2
accounting-allocator.cc 4.48 KB