rebase - #8
Draft
sunny2022da wants to merge 72 commits into
Draft
Conversation
This PR implement the Parallel EVM engine Co-authored-by: setunapo Co-authored-by: sunny2022da Co-authored-by: galaio Co-authored-by: andyzhang2023
Fix an issue of incorrectly set the origin storage at parallel stateDB's GetState(). Remove this code because it is already solved by lightCopy PR: #2
rwset: support collect rwset from statedb; mvstates: support export DAG; dag: support travel all execution paths; dag: refactor versioned TxDAG; dag: support profile parallel execution path; protocol: support to transfer TxDAG in NewBLock msg; PR: #4
* fix several UT with racing issues
* fix incorrect nonce balance codehash issue
case: TestEIP1559 / TestDeleteThenCreate
* Fix ExecutionSpec tests
mainly root caused by balance not updated to dirty correctly.
also fix similar issue with nonce and codehash.
* fix TestBlockChain testcase issue
TestBlockchain/ValidBlocks/bcStateTests/refundReset.json
Co-authored-by: Sunny <sunny2022.za@gmail.com>
* dag: add merge execute path method; pevm: support dispatch with TxDAG; * dag: add merge execute path method; pevm: support dispatch with TxDAG; * dag: clean code; * statedb: fix some broken uts; * pevm: support disable slot steal; --------- Co-authored-by: galaio <galaio@users.noreply.github.com>
* fix several UT with racing issues
* fix incorrect nonce balance codehash issue
case: TestEIP1559 / TestDeleteThenCreate
* Fix ExecutionSpec tests
mainly root caused by balance not updated to dirty correctly.
also fix similar issue with nonce and codehash.
* fix TestBlockChain testcase issue
TestBlockchain/ValidBlocks/bcStateTests/refundReset.json
* fix concurrent racing issue of state.accounts.
fix incorrect use of s.accountStorageParallelLock, it is designed
to be used for dirty/pending/original storages, not the accounts and
storages.
Use statedb.AccountMux and statedb.StorageMux for accounts/storages
lock.
* fix issue of DAOTransactions
handle the issue of updateObject of mainDB object touched by DAO transaction.
---------
Co-authored-by: Sunny <sunny2022.za@gmail.com>
There can be a issue that the object updated by mainDB.GetNonce etc is obseleted. The fix use statedb.getStateObjectNoUpdate to avoid touching the stateObjects of mainDB. Case: TestBlockchain/ValidBlocks/bcEIP1559/intrinsic.json
…ng; (#10) * txdag: support txdag transfer in extra; * txdag: support txdag transfer in extra; --------- Co-authored-by: galaio <galaio@users.noreply.github.com>
txdag: record txdag metrics; txdag: opt txdag flag name; Co-authored-by: galaio <galaio@users.noreply.github.com>
…ain#12) Fix 3 issues: - re-execution happens only to new version of baseDB to remove redundancy execution. And remove the retry with same baseIndex that is conflicted. - incorrect merge dirty objects in addrStateChangeInSlot, which cause incorrect data.root copied with obseleted stateDB, this fix handle the created, stateChanged and deleted object separately. - stateObject.GetCommitedState check mainDB of the object delete. Co-authored-by: Sunny <sunny2022.za@gmail.com>
* pevm: support delay gas fee calculation; txdag: check gas fee receiver; tests: support PEVM+TxDAG UTs; * txdag: skip some cost time operation; tests: fix some broken UTs; --------- Co-authored-by: galaio <galaio@users.noreply.github.com>
The originStorage will miss some loading in txn execution,do merge rather than simple copy This fix also use stateObject specific lock for storage update, rather than the one in stateDB. Co-authored-by: Sunny <sunny2022.za@gmail.com>
Co-authored-by: galaio <galaio@users.noreply.github.com>
txdag: opt rw record flag; txdag: opt some logic; Co-authored-by: galaio <galaio@users.noreply.github.com>
Co-authored-by: Sunny <sunny2022.za@gmail.com>
* txdag: opt some logic; txdag: opt rw set collect logic; * pevm: opt logs; * txdag: opt txdag encoding, reduce rlp size; --------- Co-authored-by: galaio <galaio@users.noreply.github.com>
* remove finalise * fix: update maindb txIndex after merge slotDB otherwise there can be issue that txIndex is load before the change in mergeSlotDB. * Fix: avoid update mainDB nonce in executeInSlot It should use slotDB, otherwise it cause the stateObjects change in mainDB, which cause racing issue. * Fix: remove stateDB update during conflict check stateDB.getState() will update the stateDB's stateObjects, which should not be updated for the purpose of state read for conflict check. --------- Co-authored-by: Sunny <sunny2022.za@gmail.com>
* txdag: add excluded flag; mvstates: generate txdag with excluded flag; * pevm: support txdag with excluded tx; * blockchain: opt txdag file mode; * pevm: fix dispatch bugs; * pevm: opt txdag dispatch; --------- Co-authored-by: galaio <galaio@users.noreply.github.com>
* pevm: opt slot trigger mechanism; * txdag: opt execute stat; * pevm: opt slot trigger mechanism; * txdag: add txdag more validation logic; --------- Co-authored-by: galaio <galaio@users.noreply.github.com>
Make the change into the merged mainDB instead of slotDB to avoid the concurrency issue Co-authored-by: Sunny <sunny2022.za@gmail.com>
* txdag: support new txdep resolve method; pevm: avoid read txdag file when generating; * pevm: support read txdag file in const size; * txdag: reduce mem alloc and async resolve tx dependency; --------- Co-authored-by: galaio <galaio@users.noreply.github.com>
This change enable parallel execution when txs count > parallelTxNum/2 + 2, with lower bound as 4, as the parallel execution is slower than serial execution with single thread.
This change avoid the rewrite of reads cache in slotDB.
This change fix the issue that prevDestruct is not recorded correctly and the stateObjectDestruct is not recorded in slotDB's createObject. The incorrect record causes the GetState get obsoleted state as the stateObjectDestruct is not correct.
sunny2022da
marked this pull request as draft
September 25, 2024 13:07
sunny2022da
force-pushed
the
p-rebase
branch
from
September 25, 2024 15:06
d8c3613 to
16b9cd2
Compare
sunny2022da
force-pushed
the
p-rebase
branch
4 times, most recently
from
September 26, 2024 06:56
be4f3b1 to
b8b2373
Compare
sunny2022da
force-pushed
the
p-rebase
branch
3 times, most recently
from
September 27, 2024 07:39
3daa0e3 to
f8ead8f
Compare
sunny2022da
force-pushed
the
p-rebase
branch
from
September 29, 2024 05:44
d237d91 to
ba207bc
Compare
sunny2022da
force-pushed
the
p-rebase
branch
from
September 29, 2024 08:20
789ab40 to
54dd5ad
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
add a description of your changes here...
Rationale
tell us why we need these changes...
Example
add an example CLI or API response...
Changes
Notable changes: