mirror of
https://github.com/status-im/nimbus-eth2.git
synced 2025-01-10 22:36:01 +00:00
5713a3ce4c
* performance fixes
* don't mark tree cache as dirty on read-only List accesses
* store only blob in memory for keys and signatures, parse blob lazily
* compare public keys by blob instead of parsing / converting to raw
* compare Eth2Digest using non-constant-time comparison
* avoid some unnecessary validator copying
This branch will in particular speed up deposit processing which has
been slowing down block replay.
Pre (mainnet, 1600 blocks):
```
All time are ms
Average, StdDev, Min, Max, Samples, Test
Validation is turned off meaning that no BLS operations are performed
3450.269, 0.000, 3450.269, 3450.269, 1, Initialize DB
0.417, 0.822, 0.036, 21.098, 1400, Load block from database
16.521, 0.000, 16.521, 16.521, 1, Load state from database
27.906, 50.846, 8.104, 1507.633, 1350, Apply block
52.617, 37.029, 20.640, 135.938, 50, Apply epoch block
```
Post:
```
3502.715, 0.000, 3502.715, 3502.715, 1, Initialize DB
0.080, 0.560, 0.035, 21.015, 1400, Load block from database
17.595, 0.000, 17.595, 17.595, 1, Load state from database
15.706, 11.028, 8.300, 107.537, 1350, Apply block
33.217, 12.622, 17.331, 60.580, 50, Apply epoch block
```
* more perf fixes
* load EpochRef cache into StateCache more aggressively
* point out security concern with public key cache
* reuse proposer index from state when processing block
* avoid genericAssign in a few more places
* don't parse key when signature is unparseable
* fix `==` overload for Eth2Digest
* preallocate validator list when getting active validators
* speed up proposer index calculation a little bit
* reuse cache when replaying blocks in ncli_db
* avoid a few more copying loops
```
Average, StdDev, Min, Max, Samples, Test
Validation is turned off meaning that no BLS operations are performed
3279.158, 0.000, 3279.158, 3279.158, 1, Initialize DB
0.072, 0.357, 0.035, 13.400, 1400, Load block from database
17.295, 0.000, 17.295, 17.295, 1, Load state from database
5.918, 9.896, 0.198, 98.028, 1350, Apply block
15.888, 10.951, 7.902, 39.535, 50, Apply epoch block
0.000, 0.000, 0.000, 0.000, 0, Database block store
```
* clear full balance cache before processing rewards and penalties
```
All time are ms
Average, StdDev, Min, Max, Samples, Test
Validation is turned off meaning that no BLS operations are performed
3947.901, 0.000, 3947.901, 3947.901, 1, Initialize DB
0.124, 0.506, 0.026, 202.370, 363345, Load block from database
97.614, 0.000, 97.614, 97.614, 1, Load state from database
0.186, 0.188, 0.012, 99.561, 357262, Advance slot, non-epoch
14.161, 5.966, 1.099, 395.511, 11524, Advance slot, epoch
1.372, 4.170, 0.017, 276.401, 363345, Apply block, no slot processing
0.000, 0.000, 0.000, 0.000, 0, Database block store
```
55 lines
1.8 KiB
Nim
55 lines
1.8 KiB
Nim
# beacon_chain
|
|
# Copyright (c) 2018 Status Research & Development GmbH
|
|
# Licensed and distributed under either of
|
|
# * MIT license (license terms in the root directory or at https://opensource.org/licenses/MIT).
|
|
# * Apache v2 license (license terms in the root directory or at https://www.apache.org/licenses/LICENSE-2.0).
|
|
# at your option. This file may not be copied, modified, or distributed except according to those terms.
|
|
|
|
{.used.}
|
|
|
|
import
|
|
unittest, ./testutil,
|
|
../beacon_chain/spec/[datatypes, crypto],
|
|
../beacon_chain/ssz
|
|
|
|
# Sanity checks to make sure all the workarounds introduced
|
|
# to deal with https://github.com/status-im/nimbus-eth2/issues/374
|
|
# and https://github.com/ethereum/eth2.0-specs/issues/1396
|
|
# don't blow up.
|
|
|
|
suiteReport "Zero signature sanity checks":
|
|
# Using signature directly triggers a bug
|
|
# in object_serialization/stew: https://github.com/status-im/nimbus-eth2/issues/396
|
|
|
|
# test "SSZ serialization round-trip doesn't un-zero the signature":
|
|
|
|
# let zeroSig = BlsValue[Signature](kind: OpaqueBlob)
|
|
# check:
|
|
# block:
|
|
# var allZeros = true
|
|
# for val in zeroSig.blob:
|
|
# allZeros = allZeros and val == 0
|
|
# allZeros
|
|
|
|
# let sszZeroSig = SSZ.encode(zeroSig)
|
|
# let deserZeroSig = SSZ.decode(sszZeroSig, ValidatorSig)
|
|
|
|
# check(zeroSIg == deserZeroSig)
|
|
|
|
timedTest "SSZ serialization roundtrip of SignedBeaconBlockHeader":
|
|
|
|
let defaultBlockHeader = SignedBeaconBlockHeader()
|
|
|
|
check:
|
|
block:
|
|
var allZeros = true
|
|
for val in defaultBlockHeader.signature.blob:
|
|
allZeros = allZeros and val == 0
|
|
allZeros
|
|
|
|
let sszDefaultBlockHeader = SSZ.encode(defaultBlockHeader)
|
|
let deserBlockHeader =
|
|
SSZ.decode(sszDefaultBlockHeader, SignedBeaconBlockHeader)
|
|
|
|
check(defaultBlockHeader == deserBlockHeader)
|