• Mircea Trofin's avatar
    Revert "[wasm] A simple allocator datastructure for off-the heap" · ee5c31f3
    Mircea Trofin authored
    This reverts commit 110d9ab0.
    
    Reason for revert: https://build.chromium.org/p/client.v8/builders/V8%20Linux64%20-%20debug%20builder/builds/26607
    
    Surprising we're seeing a failure on Linux 64 *after* CQ. Is the compiler there different?
    
    Original change's description:
    > [wasm] A simple allocator datastructure for off-the heap
    > 
    > We'll use this allocator in a follow-up CL to:
    > - allocate speculative sizes of memory for a module that's being
    > compiled (e.g. 2*size of wasm code).
    > - each module will own such a sub-pool, and then use it to allocate
    > contiguous chunks of memory for code.
    > 
    > The underlying assumptions for the chosen allocation strategy is that:
    > - the allocation granularity for pools is 1 page, so that no one page
    > is owned by more than one wasm module
    > - typical pool sizes (given module sizes) are multiple pages.
    > - modules and module instances are typically few and long lived. Typically,
    > we expect one module and one instance. 
    > 
    > This means we shouldn't expect fragmentations that lead to code being
    > non-allocatable, or prohibitively many ranges.
    > 
    > The data structure just manages ranges of addresses. Virtual memory management
    > will be separate, as part of the responsibility of a "WasmHeap"
    > that will be introduced in the future. So will concurrency control.
    > 
    > Bug: 
    > Change-Id: Id99f46d10c25553b013054d994760f3c2a737c39
    > Reviewed-on: https://chromium-review.googlesource.com/669296
    > Commit-Queue: Mircea Trofin <mtrofin@chromium.org>
    > Reviewed-by: Eric Holk <eholk@chromium.org>
    > Reviewed-by: Brad Nelson <bradnelson@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#48053}
    
    TBR=bradnelson@chromium.org,mtrofin@chromium.org,eholk@chromium.org
    
    Change-Id: Id82fa341b77624e4971f24c4757a9a666a65930c
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Reviewed-on: https://chromium-review.googlesource.com/670141Reviewed-by: 's avatarMircea Trofin <mtrofin@chromium.org>
    Commit-Queue: Mircea Trofin <mtrofin@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#48054}
    ee5c31f3
Name
Last commit
Last update
benchmarks Loading commit data...
build_overrides Loading commit data...
docs Loading commit data...
gni Loading commit data...
gypfiles Loading commit data...
include Loading commit data...
infra Loading commit data...
samples Loading commit data...
src Loading commit data...
test Loading commit data...
testing Loading commit data...
third_party Loading commit data...
tools Loading commit data...
.clang-format Loading commit data...
.editorconfig Loading commit data...
.gitignore Loading commit data...
.gn Loading commit data...
.ycm_extra_conf.py Loading commit data...
AUTHORS Loading commit data...
BUILD.gn Loading commit data...
CODE_OF_CONDUCT.md Loading commit data...
ChangeLog Loading commit data...
DEPS Loading commit data...
LICENSE Loading commit data...
LICENSE.fdlibm Loading commit data...
LICENSE.strongtalk Loading commit data...
LICENSE.v8 Loading commit data...
LICENSE.valgrind Loading commit data...
Makefile Loading commit data...
Makefile.android Loading commit data...
OWNERS Loading commit data...
PRESUBMIT.py Loading commit data...
README.md Loading commit data...
WATCHLISTS Loading commit data...
codereview.settings Loading commit data...
snapshot_toolchain.gni Loading commit data...