• Nico Hartmann's avatar
    Revert "[heap] Rework Worklist base type" · 54ef0d87
    Nico Hartmann authored
    This reverts commit a19316d9.
    
    Reason for revert: https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Linux64%20UBSan/22670/overview
    
    Original change's description:
    > [heap] Rework Worklist base type
    >
    > Worklist uses a singly-linked list of segments to hold entries.
    > Segment size was based on a compile-time constant but already stored
    > in the segment itself.
    >
    > Rework the segments to query `malloc_usable_size()` on allocation and
    > adjust the capacity properly. For PartitionAlloc, it turns out that
    > there's ~20% more capacity available for the 64-element segments.
    >
    > This slows down actual allocation of the segments with the upside of
    > improving utilization and requiring 20% less segments.
    >
    > Change-Id: Ib8595c3fb9fb75b02e4022f6c525bb59a2df7ab7
    > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3826047
    > Commit-Queue: Anton Bikineev <bikineev@chromium.org>
    > Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
    > Reviewed-by: Anton Bikineev <bikineev@chromium.org>
    > Cr-Commit-Position: refs/heads/main@{#82432}
    
    Change-Id: I14994e11ff5ffaba70b93d977d40dd2f6e9e5d35
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3829474
    Owners-Override: Nico Hartmann <nicohartmann@chromium.org>
    Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
    Auto-Submit: Nico Hartmann <nicohartmann@chromium.org>
    Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
    Cr-Commit-Position: refs/heads/main@{#82438}
    54ef0d87
worklist-unittest.cc 10.8 KB