• 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
serializer-common.h 11 KB