risechain / pevm

Blazingly fast Parallel EVM
MIT License
227 stars 47 forks source link

feat: fix nonce deadlock #408

Open kien-rise opened 1 week ago

kien-rise commented 1 week ago

Improvement from https://github.com/risechain/pevm/pull/406

Self-tested in the PEVM integration, using ERC20 and Uniswap benchmarks.

Performance Effect

There is a slight decrease in performance.

# Before
Average: x1.95
Max: x4.18
Min: x0.88

# After
Average: x1.94
Max: x4.15
Min: x0.88

The gigagas benchmark got some performance decrease too:

Benchmarking Independent Raw Transfers/Parallel:
    time:   [60.038 ms 60.205 ms 60.372 ms]
    change: [+6.3255% +6.8597% +7.3995%] (p = 0.00 < 0.05)
    Performance has regressed.

Benchmarking Independent ERC20/Parallel:
    time:   [68.304 ms 68.664 ms 69.010 ms]
    change: [+0.2251% +0.9273% +1.5822%] (p = 0.01 < 0.05)
    Change within noise threshold.

Benchmarking Independent Uniswap/Parallel:
    time:   [33.608 ms 33.654 ms 33.703 ms]
    change: [+1.3274% +1.5758% +1.8054%] (p = 0.00 < 0.05)
    Performance has regressed.