[ETHSF_HACKATHON_SUBMISSION] (EIP-1153) Transient Storage in EVM

[epociask]

Description

• Implemented spec according to EIP-1153
• Added /core/vm/transient_storage.go file to represent Transient Storage object with necessary inner data types:
  • Journal for tracking contextual state changes
  • Checkpointing for tracking state between call frame transitions
  • Checkpointing and journal accounting logic to ensure proper reversion upon failure of a call frame
• Propagated Transient Storage struct throughout execution model
• Added two new opcode instructions:
  • TLOAD (0xb3):
    • Loads value from Transient Storage given address & key
  • TSTORE (oxb4):
    • Stores value in Transient Storage given key, address, & value

Tests

• Added /core/vm/transient_storage_test.go with minor unit tests for Transient Storage data structure
• Added runtime test to ensure proper execution of a re-entrancy block using two contracts with TLOAD and TSTORE instructions

Please describe any tests you've added. If you've added no tests, or left important behavior untested, please explain why not.

Additional context

Add any other context about the problem you're solving.

Metadata

• Fixes #[Link to Issue]

[tynes]

Hey @epociask, thanks for implementing this. Upstream geth has merged 1153 in ethereum/go-ethereum#26003. We rebase on upstream every so often and will be able to pull in that implementation.

EIP 1153 has been implemented and merged into a bunch of clients:

• EIP-1153: Transient storage opcodes NethermindEth/nethermind#4126
• VM, Common: Implement EIP 1153, Transient storage opcodes ethereumjs/ethereumjs-monorepo#1768
• all: implement EIP-1153 transient storage ethereum/go-ethereum#26003

There are some clients that are still waiting for merge, so if you are still interested in helping out, these PRs could use review/additional testing:

• all: eip1153 erigontech/erigon#6133
• EIP-1153: Transient storage opcodes hyperledger/besu#4118
• feat: eip 1153 bluealloy/revm#276