core,eth: call frame tracing #23087
Merged
core,eth: call frame tracing #23087
Conversation
This file contains 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
fedekunze
reviewed
Jun 23, 2021
rjl493456442
reviewed
Aug 2, 2021
holiman
reviewed
Aug 25, 2021
core/vm/evm.go
Outdated
| } | ||
| return ret, address, contract.Gas, err | ||
| } | ||
|
|
||
| // Create creates a new contract using code as deployment code. | ||
| func (evm *EVM) Create(caller ContractRef, code []byte, gas uint64, value *big.Int) (ret []byte, contractAddr common.Address, leftOverGas uint64, err error) { | ||
| contractAddr = crypto.CreateAddress(caller.Address(), evm.StateDB.GetNonce(caller.Address())) | ||
| return evm.create(caller, &codeAndHash{code: code}, gas, value, contractAddr) | ||
| return evm.create(caller, &codeAndHash{code: code}, gas, value, contractAddr, CreateType) |
There was a problem hiding this comment.
Doesn't this also trigger for a regular create-transaction?
| this.callstack[len-1].calls.push(call) | ||
| } | ||
| }, | ||
| // finalize recreates a call object using the final desired field oder for json |
There was a problem hiding this comment.
Suggested change
| // finalize recreates a call object using the final desired field oder for json | |
| // finalize recreates a call object using the final desired field order for json |
holiman
reviewed
Sep 9, 2021
holiman
reviewed
Sep 10, 2021
holiman
force-pushed
the
trace-call-frame
branch
from
35f0b78 to
416a747
Compare
|
All tests are passing now! I came across two points:
|
|
One remaining item (if we want to), rename "call_tracer.js" -> "call_tracer_legacy.js" and "callframe_tracer.js" -> "call_tracer.js". When a user asks to use |
MariusVanDerWijden
approved these changes
Sep 16, 2021
| // about internal messages of a transaction. | ||
| { | ||
| callstack: [{}], | ||
| step: function(log, db) {}, |
There was a problem hiding this comment.
Do you need to declare this function here? Wouldn't this defeat the purpose of not having to call back into js for every step?
There was a problem hiding this comment.
Ooof! I need to rerun the benches now!
|
Turns out the speedup is much more, it's only that I had forgotten to remove |
|
|
maoueh
pushed a commit
to streamingfast/go-ethereum
that referenced
this pull request
Jan 31, 2022
* eth/tracers: implement debug.intermediateRoots (ethereum#23594) This PR implements a new debug method, which I've talked briefly about to some other client developers. It allows the caller to obtain the intermediate state roots for a block (which might be either a canon block or a 'bad' block). Signed-off-by: wenbiao <[email protected]> * core, rpc: disable memory output by default in traces (ethereum#23558) * core: cmd: invert disableMemory * core: fix missed inversion * cmd/evm: preserve Flags but change default value * Apply suggestions from code review Co-authored-by: Martin Holst Swende <[email protected]> Co-authored-by: Martin Holst Swende <[email protected]> Signed-off-by: wenbiao <[email protected]> * eth/tracers: abort evm execution when trace is aborted (ethereum#23580) Signed-off-by: wenbiao <[email protected]> * eth/tracers: avoid unsyncronized mutations on trie database (ethereum#23632) This PR fixes an issue in traceChain, where the statedb Commit operation was performed asynchronously with dereference-operations agains the underlying trie.Database instance. Due to how the reference counting works within the trie database (where parent count is recursively updated when new parents are added), doing dereferencing in the middle of Commit can cause the refcount to become wrong, leading to an inconsistent state. This was fixed by doing Commit/Deref from the same routine. Signed-off-by: wenbiao <[email protected]> * core,eth: call frame tracing (ethereum#23087) This change introduces 2 new optional methods; `enter()` and `exit()` for js tracers, and makes `step()` optiona. The two new methods are invoked when entering and exiting a call frame (but not invoked for the outermost scope, which has it's own methods). Currently these are the data fields passed to each of them: enter: type (opcode), from, to, input, gas, value exit: output, gasUsed, error The PR also comes with a re-write of the callTracer. As a backup we keep the previous tracing script under the name `callTracerLegacy`. Behaviour of both tracers are equivalent for the most part, although there are some small differences (improvements), where the new tracer is more correct / has more information. Signed-off-by: wenbiao <[email protected]> * eth/tracers: re-write of 4byte tracer using enter/exit (ethereum#23622) * eth/tracers: add re-write of 4byte tracer using enter/exit * eth/tracers: fix 4byte indent Signed-off-by: wenbiao <[email protected]> * eth/tracers: tx.BaseFee not implemented Signed-off-by: wenbiao <[email protected]> * eth/tracers: do the JSON serialization via .js to capture C faults Signed-off-by: wenbiao <[email protected]> * eth/tracers: fix callTracer fault handling (ethereum#23667) * eth/tracers: fix calltracer fault handling * eth/tracers: fix calltracer indentation Signed-off-by: wenbiao <[email protected]> * eth/tracers: invoke enter/exit on 0-value calls to inex accounts (ethereum#23828) Signed-off-by: wenbiao <[email protected]> * eth: make traceChain avoid OOM on long-running tracing (ethereum#23736) This PR changes long-running chain tracing, so that it at some points releases the memory trie db, and switch over to a fresh disk-backed trie. Signed-off-by: wenbiao <[email protected]> * eth/tracers: expose contextual infos (block hash, tx hash, tx index) Signed-off-by: wenbiao <[email protected]> * eth/tracers: redefine Context Signed-off-by: wenbiao <[email protected]> * eth/tracers: support for golang tracers + add golang callTracer (ethereum#23708) * eth/tracers: add basic native loader * eth/tracers: add GetResult to tracer interface * eth/tracers: add native call tracer * eth/tracers: fix call tracer json result * eth/tracers: minor fix * eth/tracers: fix * eth/tracers: fix benchTracer * eth/tracers: test native call tracer * eth/tracers: fix * eth/tracers: rm extra make Co-authored-by: Martin Holst Swende <[email protected]> * eth/tracers: rm extra make * eth/tracers: make callFrame private * eth/tracers: clean-up and comments * eth/tracers: add license * eth/tracers: rework the model a bit * eth/tracers: move tracecall tests to subpackage * cmd/geth: load native tracers * eth/tracers: minor fix * eth/tracers: impl stop * eth/tracers: add native noop tracer * renamings Co-authored-by: Martin Holst Swende <[email protected]> * eth/tracers: more renamings * eth/tracers: make jstracer non-exported, avoid cast * eth/tracers, core/vm: rename vm.Tracer to vm.EVMLogger for clarity * eth/tracers: minor comment fix * eth/tracers/testing: lint nitpicks * core,eth: cancel evm on nativecalltracer stop * Revert "core,eth: cancel evm on nativecalltracer stop" This reverts commit 01bb908. * eth/tracers: linter nits * eth/tracers: fix output on err Co-authored-by: Martin Holst Swende <[email protected]> Signed-off-by: wenbiao <[email protected]> * eth/tracers: make native calltracer default (ethereum#23867) Signed-off-by: wenbiao <[email protected]> * eth/tracers: package restructuring (ethereum#23857) * eth/tracers: restructure tracer package * core/vm/runtime: load js tracers * eth/tracers: mv bigint js code to own file * eth/tracers: add method docs for native tracers * eth/tracers: minor doc fix * core,eth: cancel evm on nativecalltracer stop * core/vm: fix failing test Co-authored-by: Sina Mahmoodi <[email protected]> Signed-off-by: wenbiao <[email protected]> * eth/tracers: ethapi.TransactionArgs was not merged Signed-off-by: wenbiao <[email protected]> * eth/tracers: fix the api_test with ErrInsufficientFunds to ErrInsufficientFundsForTransfer Signed-off-by: wenbiao <[email protected]> * eth/tracers: check posa before statedb.Prepare in IntermiateRoots api Signed-off-by: wenbiao <[email protected]> * eth/tracers: make js calltracer default, compatible with old version Signed-off-by: wenbiao <[email protected]> * eth/tracers: fix the default callTrace name of callTracerJs Signed-off-by: wenbiao <[email protected]> * Revert "eth/tracers: fix the default callTrace name of callTracerJs" This reverts commit 62a3bc215d9f07e422a4c659289bb3ba4f9ed2fa. Signed-off-by: wenbiao <[email protected]> * Revert "eth/tracers: make js calltracer default, compatible with old version" This reverts commit 85ef42c0ea651f0b228d4209b1b2598b24e12f1f. Signed-off-by: wenbiao <[email protected]> * eth/tracers: fix the variable race condition Signed-off-by: wenbiao <[email protected]> Co-authored-by: Martin Holst Swende <[email protected]> Co-authored-by: Marius van der Wijden <[email protected]> Co-authored-by: Sina Mahmoodi <[email protected]> Co-authored-by: Péter Szilágyi <[email protected]> Co-authored-by: Sina Mahmoodi <[email protected]>
uprendis
pushed a commit
to uprendis/go-ethereum
that referenced
this pull request
Feb 15, 2022
This change introduces 2 new optional methods; `enter()` and `exit()` for js tracers, and makes `step()` optiona. The two new methods are invoked when entering and exiting a call frame (but not invoked for the outermost scope, which has it's own methods). Currently these are the data fields passed to each of them:
enter: type (opcode), from, to, input, gas, value
exit: output, gasUsed, error
The PR also comes with a re-write of the callTracer. As a backup we keep the previous tracing script under the name `callTracerLegacy`. Behaviour of both tracers are equivalent for the most part, although there are some small differences (improvements), where the new tracer is more correct / has more information.
uprendis
added a commit
to Fantom-foundation/go-ethereum
that referenced
this pull request
Mar 4, 2022
core,eth: call frame tracing (ethereum#23087)
yongjun925
pushed a commit
to DODOEX/go-ethereum
that referenced
this pull request
Dec 3, 2022
This change introduces 2 new optional methods; `enter()` and `exit()` for js tracers, and makes `step()` optiona. The two new methods are invoked when entering and exiting a call frame (but not invoked for the outermost scope, which has it's own methods). Currently these are the data fields passed to each of them:
enter: type (opcode), from, to, input, gas, value
exit: output, gasUsed, error
The PR also comes with a re-write of the callTracer. As a backup we keep the previous tracing script under the name `callTracerLegacy`. Behaviour of both tracers are equivalent for the most part, although there are some small differences (improvements), where the new tracer is more correct / has more information.
4 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Introduces 2 new optional methods
enter()andexit()for js tracers which are invoked on stepping in and out a call frame. These methods are not invoked for depth 0. Currently these are the data fields passed to each of them:If these methods are not present in the JS script geth will default to only invoking
step(). On the other hand, you have the option of only providingenter()andexit(), and not providingstep()if call frames are the only thing you care about. This should yield a ~5x tracing speedup because it avoids the go-js barrier on every opcode.The PR also comes with a re-write of the
callTracer. As a backup we keep the previous tracing script under the namecallTracerLegacy. Behaviour of both tracers are equivalent for the most part, although there are some small differences (improvements):CALL*/CREATE*fails without having executed any code, i.e. in the initial conditions like whether sender has enough balance, then no call frame will be recorded.gasandgasUsedvalues are set to0instead ofnilwhere they are unavailable. E.g. a simple transfer has agasUsedof0. ASELFDESTRUCThas bothgasandgasUsedset to0.internal failure