• Dan Elphick's avatar
    Reland "Reland "[heap] Move initial objects into RO_SPACE"" · 9ab6621a
    Dan Elphick authored
    This is a reland of 6c68efac
    
    Updated Heap::CommittedMemory and related functions to iterate over all
    spaces rather than including them manually which can lead to a space
    being overlooked. Also adds a test to ensure this the case.
    
    Original change's description:
    > Revert "Reland "[heap] Move initial objects into RO_SPACE""
    >
    > This reverts commit 6c68efac.
    >
    > Reason for revert: https://bugs.chromium.org/p/v8/issues/detail?id=7668
    >
    > Original change's description:
    > > Reland "[heap] Move initial objects into RO_SPACE"
    > >
    > > This is a reland of f8ae62fe
    > >
    > > Original change's description:
    > > > [heap] Move initial objects into RO_SPACE
    > > >
    > > > This moves:
    > > > * the main oddballs (null, undefined, hole, true, false) as well as
    > > > their supporting maps (also adds hole as an internalized string to make
    > > > this work).
    > > > * most of the internalized strings
    > > > * the struct maps
    > > > * empty array
    > > > * empty enum cache
    > > > * the contents of the initial string table
    > > > * the weak_cell_cache for any map in RO_SPACE (and eagerly creates the
    > > > value avoid writing to it during run-time)
    > > >
    > > > The StartupSerializer stats change as follows:
    > > >
    > > >      RO_SPACE  NEW_SPACE  OLD_SPACE  CODE_SPACE  MAP_SPACE  LO_SPACE
    > > > old         0          0     270264       32608      12144         0
    > > > new     21776          0     253168       32608       8184         0
    > > > Overall memory usage has increased by 720 bytes due to the eager
    > > > initialization of the Map weak cell caches.
    > > >
    > > > Also extends --serialization-statistics to print out separate instance
    > > > type stats for objects in RO_SPACE as shown here:
    > > >
    > > >   Read Only Instance types (count and bytes):
    > > >        404      16736  ONE_BYTE_INTERNALIZED_STRING_TYPE
    > > >          2         32  HEAP_NUMBER_TYPE
    > > >          5        240  ODDBALL_TYPE
    > > >         45       3960  MAP_TYPE
    > > >          1         16  BYTE_ARRAY_TYPE
    > > >          1         24  TUPLE2_TYPE
    > > >          1         16  FIXED_ARRAY_TYPE
    > > >          1         32  DESCRIPTOR_ARRAY_TYPE
    > > >         45        720  WEAK_CELL_TYPE
    > > >
    > > > Bug: v8:7464
    > > > Change-Id: I12981c39c82a7057f68bbbe03f89fb57b0b4c6a6
    > > > Reviewed-on: https://chromium-review.googlesource.com/973722
    > > > Commit-Queue: Dan Elphick <delphick@chromium.org>
    > > > Reviewed-by: Hannes Payer <hpayer@chromium.org>
    > > > Reviewed-by: Ross McIlroy <rmcilroy@chromium.org>
    > > > Reviewed-by: Yang Guo <yangguo@chromium.org>
    > > > Cr-Commit-Position: refs/heads/master@{#52435}
    > >
    > > Bug: v8:7464
    > > Change-Id: I50427edfeb53ca80ec4cf46566368fb2213ccf7b
    > > Reviewed-on: https://chromium-review.googlesource.com/999654
    > > Commit-Queue: Dan Elphick <delphick@chromium.org>
    > > Reviewed-by: Yang Guo <yangguo@chromium.org>
    > > Reviewed-by: Hannes Payer <hpayer@chromium.org>
    > > Cr-Commit-Position: refs/heads/master@{#52638}
    >
    > TBR=rmcilroy@chromium.org,yangguo@chromium.org,hpayer@chromium.org,mlippautz@chromium.org,delphick@chromium.org
    >
    > # Not skipping CQ checks because original CL landed > 1 day ago.
    >
    > Bug: v8:7464,v8:7668
    > Change-Id: I10aa03623b51e997f95a3715ea9f0bf5d29d2cdb
    > Reviewed-on: https://chromium-review.googlesource.com/1016600
    > Commit-Queue: Peter Marshall <petermarshall@chromium.org>
    > Reviewed-by: Peter Marshall <petermarshall@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#52667}
    
    Cq-Include-Trybots: luci.chromium.try:linux_chromium_rel_ng
    Change-Id: If4b7490c8c4d31612de8ec132de334955a319b11
    Bug: v8:7464, v8:7668
    Reviewed-on: https://chromium-review.googlesource.com/1019020Reviewed-by: 's avatarJakob Gruber <jgruber@chromium.org>
    Reviewed-by: 's avatarUlan Degenbaev <ulan@chromium.org>
    Commit-Queue: Dan Elphick <delphick@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#52689}
    9ab6621a
Name
Last commit
Last update
..
blink_tests Loading commit data...
cfi Loading commit data...
clusterfuzz Loading commit data...
dev Loading commit data...
gcmole Loading commit data...
generate_shim_headers Loading commit data...
heap-stats Loading commit data...
ignition Loading commit data...
jsfunfuzz Loading commit data...
link_clicker.extension Loading commit data...
mb Loading commit data...
memory Loading commit data...
msan Loading commit data...
ninja Loading commit data...
node Loading commit data...
perf Loading commit data...
profview Loading commit data...
profviz Loading commit data...
release Loading commit data...
sanitizers Loading commit data...
sodium Loading commit data...
testrunner Loading commit data...
toolchain Loading commit data...
turbolizer Loading commit data...
ubsan Loading commit data...
unittests Loading commit data...
v8.xcodeproj Loading commit data...
valgrind/asan Loading commit data...
vim Loading commit data...
visual_studio Loading commit data...
wasm Loading commit data...
BUILD.gn Loading commit data...
DEPS Loading commit data...
Makefile.tags Loading commit data...
OWNERS Loading commit data...
SourceMap.js Loading commit data...
adb-d8.py Loading commit data...
android-build.sh Loading commit data...
android-ll-prof.sh Loading commit data...
android-run.py Loading commit data...
android-sync.sh Loading commit data...
arguments.js Loading commit data...
bash-completion.sh Loading commit data...
bigint-tester.py Loading commit data...
callstats.html Loading commit data...
callstats.py Loading commit data...
check-inline-includes.sh Loading commit data...
check-static-initializers.sh Loading commit data...
check-unused-bailouts.sh Loading commit data...
codemap.js Loading commit data...
compare-table-gen.js Loading commit data...
concatenate-files.py Loading commit data...
consarray.js Loading commit data...
cpu.sh Loading commit data...
cross_build_gcc.sh Loading commit data...
csvparser.js Loading commit data...
detect-builtins.js Loading commit data...
disasm.py Loading commit data...
draw_instruction_graph.sh Loading commit data...
dump-cpp.py Loading commit data...
dumpcpp-driver.js Loading commit data...
dumpcpp.js Loading commit data...
eval_gc_nvp.py Loading commit data...
eval_gc_time.sh Loading commit data...
find-commit-for-patch.py Loading commit data...
find_depot_tools.py Loading commit data...
format-torque.py Loading commit data...
freebsd-tick-processor Loading commit data...
fuzz-harness.sh Loading commit data...
gc-nvp-to-csv.py Loading commit data...
gc-nvp-trace-processor.py Loading commit data...
gc_nvp_common.py Loading commit data...
gdb-v8-support.py Loading commit data...
gdbinit Loading commit data...
gen-inlining-tests.py Loading commit data...
gen-postmortem-metadata.py Loading commit data...
generate-builtins-tests.py Loading commit data...
generate-ten-powers.scm Loading commit data...
get_landmines.py Loading commit data...
grokdump.py Loading commit data...
ic-explorer.html Loading commit data...
ic-processor Loading commit data...
ic-processor-driver.js Loading commit data...
ic-processor.js Loading commit data...
js2c.py Loading commit data...
jsmin.py Loading commit data...
linux-tick-processor Loading commit data...
ll_prof.py Loading commit data...
lldb_commands.py Loading commit data...
lldbinit Loading commit data...
logreader.js Loading commit data...
mac-nm Loading commit data...
mac-tick-processor Loading commit data...
make-torque-parser.py Loading commit data...
map-processor Loading commit data...
map-processor-driver.js Loading commit data...
map-processor.html Loading commit data...
map-processor.js Loading commit data...
mips_toolchain.tar.gz.sha1 Loading commit data...
objdump-v8 Loading commit data...
parse-processor Loading commit data...
parse-processor-driver.js Loading commit data...
parse-processor.html Loading commit data...
parse-processor.js Loading commit data...
parser-shell.cc Loading commit data...
perf-compare.py Loading commit data...
perf-to-html.py Loading commit data...
plot-timer-events Loading commit data...
predictable_wrapper.py Loading commit data...
presubmit.py Loading commit data...
profile.js Loading commit data...
profile_view.js Loading commit data...
run-llprof.sh Loading commit data...
run-num-fuzzer.py Loading commit data...
run-perf.sh Loading commit data...
run-tests.py Loading commit data...
run.py Loading commit data...
run_perf.py Loading commit data...
shell-utils.h Loading commit data...
splaytree.js Loading commit data...
stats-viewer.py Loading commit data...
tick-processor.html Loading commit data...
tickprocessor-driver.js Loading commit data...
tickprocessor.js Loading commit data...
trace-maps-processor.py Loading commit data...
try_perf.py Loading commit data...
turbolizer-perf.py Loading commit data...
v8heapconst.py Loading commit data...
whitespace.txt Loading commit data...
windows-tick-processor.bat Loading commit data...