• Clemens Backes's avatar
    Fail earlier on FreePages · 205fb295
    Clemens Backes authored
    {FreePages} is never expected to fail, and each caller wraps the call in
    a CHECK macro. In order to learn more about failures, this CL moves the
    CHECK inside of {::FreePages}, to fail whenever the {PageAllocator}
    fails to free pages.
    
    As a next step, I'll audit our {PageAllocator} implementations to ensure
    that none of them return {false} for {FreePages}. Note that this is
    already the case for the gin platform (chromium).
    
    R=mlippautz@chromium.org
    
    Bug: v8:12656, chromium:1299735
    Change-Id: Ib61be6cc8da0110ead2db1ad005728bd061e0243
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3484321Reviewed-by: 's avatarMichael Lippautz <mlippautz@chromium.org>
    Reviewed-by: 's avatarLeszek Swirski <leszeks@chromium.org>
    Commit-Queue: Clemens Backes <clemensb@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#79248}
    205fb295
Name
Last commit
Last update
..
DIR_METADATA Loading commit data...
OWNERS Loading commit data...
address-map.cc Loading commit data...
address-map.h Loading commit data...
allocation.cc Loading commit data...
allocation.h Loading commit data...
bit-vector.cc Loading commit data...
bit-vector.h Loading commit data...
boxed-float.h Loading commit data...
detachable-vector.cc Loading commit data...
detachable-vector.h Loading commit data...
identity-map.cc Loading commit data...
identity-map.h Loading commit data...
locked-queue-inl.h Loading commit data...
locked-queue.h Loading commit data...
memcopy.cc Loading commit data...
memcopy.h Loading commit data...
ostreams.cc Loading commit data...
ostreams.h Loading commit data...
pointer-with-payload.h Loading commit data...
scoped-list.h Loading commit data...
utils-inl.h Loading commit data...
utils.cc Loading commit data...
utils.h Loading commit data...
v8dll-main.cc Loading commit data...
version.cc Loading commit data...
version.h Loading commit data...