Skip to content

Block Level Access Lists #9425

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 182 commits into
base: master
Choose a base branch
from
Draft

Block Level Access Lists #9425

wants to merge 182 commits into from

Conversation

@Marchhill
Copy link
Contributor

Changes

Implement https://eips.ethereum.org/EIPS/eip-7928

See also #9182

Types of changes

What types of changes does your code introduce?

  • Bugfix (a non-breaking change that fixes an issue)
  • New feature (a non-breaking change that adds functionality)
  • Breaking change (a change that causes existing functionality not to work as expected)
  • Optimization
  • Refactoring
  • Documentation update
  • Build-related changes
  • Other: Description

Testing

Requires testing

  • Yes
  • No

If yes, did you write tests?

  • Yes
  • No

Documentation

Requires documentation update

  • Yes
  • No

Requires explanation in Release Notes

  • Yes
  • No

Marchhill added 30 commits August 11, 2025 17:43
@Marchhill
add to release spec
9070ca2
@Marchhill
add hash to header
ea3f4d2
@Marchhill
add to block body
5fae29d
@Marchhill
BAL validation
273905e
@Marchhill
add to constuctor
0246d85
@Marchhill
set BAL at end of processing
db756e6
@Marchhill
constants, ssz containers
8c6f4e2
@Marchhill
add block access tracer
dbed6d2
@Marchhill
construct BAL in tracer
7d7cb29
@Marchhill
call byte encode, move BAL to core
ec347c1
@Marchhill
block access tracer improvements & tests
675485c
@Marchhill
add to execution payload
65bcb30
@Marchhill
update structures
4cd315b
@Marchhill
only record last change of each tx
6f9aabf
@Marchhill
fix test
d884b36
@Marchhill
change storage tracing
0eb0736
@Marchhill
RLP encode bal
603d56a
@Marchhill
prewarm addresses
1726b9b
@Marchhill
use bal as state provider
8ac6add
@Marchhill
reset changes using BAL
190f2e4
@Marchhill
only trace when EIP enabled
f09890c
@Marchhill
handle withdrawals
f93a43c
@Marchhill
handle system contracts
2bf2cef
@Marchhill
fix compile
ce00892
@Marchhill
add block processing test
9d1d9ff
@Marchhill
move bal decoding to suggested block validation
2f6829d
@Marchhill
decoding debug logs
23dc197
@Marchhill
empty rlp decode for bal
496b686
@Marchhill
move RLP decoding
15bbab1
@Marchhill
don't encode/decode for testing
7c31e87
@Marchhill
try fix account changes in wrong order error
30b6f29
@Marchhill
coinbase zero transfer edge case
96bdd95
@Marchhill
code change max increase
e461a6c
@Marchhill
Merge remote-tracking branch 'upstream/master' into bal-devnet-0
7352691
@Marchhill
enforce rlp buffer size limits
cc7e07c
@Marchhill
fix test runners
36937cd
@Marchhill
fix restore
78d054e
@Marchhill
fix reverting storage change
074a9e7
@Marchhill
add account read for extcodecopy
81f3d0f
@Marchhill
don't add for extcodecopy when oog
bc43434
@Marchhill
fix engine test runner
872ff02
@Marchhill
refactor test runner
c3b9447
@Marchhill
don't record storage change if resets to same value
f9ff97a
@Marchhill
remove storage changes if empty
88ae970
@Marchhill
fix storage changes
4023bc2
@Marchhill
only add read if write not present
77c7130
@Marchhill
fix error message
a6081ae
@Marchhill
add changes when resetting to og value
41e689d
@Marchhill
change comparison, add logging
bd76f9d
@Marchhill
more logging
eb23d61
@Marchhill
even more logging
e314c75
@Marchhill
use sequenceequal
8f5604e
@Marchhill
check for previous code deployed being the same
689f10c
@Marchhill
record more account accesses
5536c3d
@Marchhill
extcodesize
8d63f4d
@Marchhill
fix tracing
e7b662d
@Marchhill Marchhill force-pushed the bal-devnet-0 branch 2 times, most recently from bcda1e4 to e7b662d Compare October 28, 2025 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@LukaszRozmej LukaszRozmej Awaiting requested review from LukaszRozmej LukaszRozmej will be requested when the pull request is marked ready for review LukaszRozmej is a code owner

@benaadams benaadams Awaiting requested review from benaadams benaadams will be requested when the pull request is marked ready for review benaadams is a code owner

@asdacap asdacap Awaiting requested review from asdacap asdacap will be requested when the pull request is marked ready for review asdacap is a code owner

@flcl42 flcl42 Awaiting requested review from flcl42 flcl42 will be requested when the pull request is marked ready for review flcl42 is a code owner

@damian-orzechowski damian-orzechowski Awaiting requested review from damian-orzechowski damian-orzechowski will be requested when the pull request is marked ready for review damian-orzechowski is a code owner

@tanishqjasoria tanishqjasoria Awaiting requested review from tanishqjasoria tanishqjasoria will be requested when the pull request is marked ready for review tanishqjasoria is a code owner

@smartprogrammer93 smartprogrammer93 Awaiting requested review from smartprogrammer93 smartprogrammer93 will be requested when the pull request is marked ready for review smartprogrammer93 is a code owner

@emlautarom1 emlautarom1 Awaiting requested review from emlautarom1 emlautarom1 will be requested when the pull request is marked ready for review emlautarom1 is a code owner

@deffrian deffrian Awaiting requested review from deffrian deffrian will be requested when the pull request is marked ready for review deffrian is a code owner

@MarekM25 MarekM25 Awaiting requested review from MarekM25 MarekM25 will be requested when the pull request is marked ready for review MarekM25 is a code owner

@Demuirgos Demuirgos Awaiting requested review from Demuirgos Demuirgos will be requested when the pull request is marked ready for review Demuirgos is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Marchhill @fselmo