• Clemens Hammacher's avatar
    Revert "[snapshot] add checksum to startup snapshot" · 0a04f263
    Clemens Hammacher authored
    This reverts commit bcb8d49b.
    
    Reason for revert: MSan compile error: https://ci.chromium.org/p/v8/builders/luci.v8.ci/V8%20Linux%20-%20arm64%20-%20sim%20-%20MSAN/23025
    
    Original change's description:
    > [snapshot] add checksum to startup snapshot
    > 
    > We already had checksumming for code cache data. We now extend
    > checksumming to the startup snapshot to catch data corruption early.
    > 
    > The performance impact for deserialization is a regression of 1-2%,
    > which should be acceptable.
    > 
    > Sample output for the included test with --profile-deserialization:
    > 
    > [Verifying snapshot checksum took 0.023 ms]
    > [Deserializing isolate (134348 bytes) took 1.891 ms]
    > [Verifying snapshot checksum took 0.024 ms]
    > [Deserializing isolate (134348 bytes) took 1.654 ms]
    > [Deserializing context #0 (47208 bytes) took 0.331 ms]
    > Deserialization will reserve:
    >     208168 bytes per isolate
    >     123368 bytes per context #0
    > Snapshot blob consists of:
    >     134492 bytes in 6 chunks for startup
    >     115272 bytes for builtins
    >      47152 bytes in 31 chunks for context #0
    > [Verifying snapshot checksum took 0.048 ms]
    > [Verifying snapshot checksum took 0.043 ms]
    > 
    > R=​peria@chromium.org, petermarshall@chromium.org
    > 
    > Bug: chromium:881417
    > Cq-Include-Trybots: luci.chromium.try:linux_chromium_rel_ng
    > Change-Id: Ibc57520d459c86be8972f731aa35045b5e3751d7
    > Reviewed-on: https://chromium-review.googlesource.com/1241874
    > Reviewed-by: Peter Marshall <petermarshall@chromium.org>
    > Commit-Queue: Yang Guo <yangguo@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#56217}
    
    TBR=peria@chromium.org,yangguo@chromium.org,petermarshall@chromium.org
    
    Change-Id: Iccb82092858ab68a5d6ae9552fa716108eda354b
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: chromium:881417
    Cq-Include-Trybots: luci.chromium.try:linux_chromium_rel_ng
    Reviewed-on: https://chromium-review.googlesource.com/1243190Reviewed-by: 's avatarClemens Hammacher <clemensh@chromium.org>
    Commit-Queue: Clemens Hammacher <clemensh@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#56221}
    0a04f263
Name
Last commit
Last update
..
benchmarks Loading commit data...
cctest Loading commit data...
common Loading commit data...
debugger Loading commit data...
fuzzer Loading commit data...
inspector Loading commit data...
intl Loading commit data...
js-perf-test Loading commit data...
memory Loading commit data...
message Loading commit data...
mjsunit Loading commit data...
mkgrokdump Loading commit data...
mozilla Loading commit data...
preparser Loading commit data...
test262 Loading commit data...
torque Loading commit data...
unittests Loading commit data...
wasm-spec-tests Loading commit data...
webkit Loading commit data...
BUILD.gn Loading commit data...