Skip to content

feat(specs,tests): EIP-7928 - Update BAL index to uint64#2713

Merged
spencer-tb merged 1 commit intoethereum:forks/amsterdamfrom
nerolation:toni/update-index-uint64
Apr 20, 2026
Merged

feat(specs,tests): EIP-7928 - Update BAL index to uint64#2713
spencer-tb merged 1 commit intoethereum:forks/amsterdamfrom
nerolation:toni/update-index-uint64

Conversation

@nerolation
Copy link
Copy Markdown
Contributor

As discussed in discord, we should update the BAL index to use uint64 instead of uint16 to be future proof, having a large enough range to support a 300 million gas limit + EIP-2780 to be shipped.

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 18, 2026

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.26%. Comparing base (c3462e0) to head (8c08a16).

Files with missing lines Patch % Lines
src/ethereum/forks/amsterdam/block_access_lists.py 0.00% 2 Missing ⚠️
Additional details and impacted files
@@               Coverage Diff                @@
##           forks/amsterdam    #2713   +/-   ##
================================================
  Coverage            86.26%   86.26%           
================================================
  Files                  599      599           
  Lines                37038    37038           
  Branches              3795     3795           
================================================
  Hits                 31949    31949           
  Misses                4525     4525           
  Partials               564      564           
Flag Coverage Δ
unittests 86.26% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Contributor

@spencer-tb spencer-tb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Would like to get in the next release!

@fselmo, flag if there are any issues from your side :)

@spencer-tb spencer-tb added A-spec-specs Area: Specification—The Ethereum specification itself (eg. `src/ethereum/*`) A-tests Area: Consensus tests. labels Apr 20, 2026
@spencer-tb spencer-tb merged commit 554bd41 into ethereum:forks/amsterdam Apr 20, 2026
19 of 20 checks passed
Copy link
Copy Markdown
Contributor

@fselmo fselmo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm 👍🏼

AskAlexSharov pushed a commit to JayeTurn/erigon that referenced this pull request Apr 23, 2026
…#20679)

Updates the Block Access List index type from `uint16` to `uint32`
across the entire stack to future-proof the BAL for higher gas limits
(`300M+`) and `EIP-2780` compatibility. The current uint16 index caps
out at `65,535` entries per block, which is insufficient once gas limits
scale beyond current levels. Widening to `uint32` eliminates this
ceiling entirely, ensuring the BAL remains viable without further type
migrations as the protocol evolves.

Spec Change Refrence:
ethereum/execution-specs#2713
EIP Change Refrence: ethereum/EIPs#11535
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-spec-specs Area: Specification—The Ethereum specification itself (eg. `src/ethereum/*`) A-tests Area: Consensus tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants