Skip to content

feat(spec-specs,tests): reprice EIP-8038 access-list costs to cold minus warm - #3271

Merged
spencer-tb merged 1 commit into
ethereum:forks/amsterdamfrom
spencer-tb:eips/amsterdam/eip-8038-access-list-cost
Jul 31, 2026
Merged

feat(spec-specs,tests): reprice EIP-8038 access-list costs to cold minus warm#3271
spencer-tb merged 1 commit into
ethereum:forks/amsterdamfrom
spencer-tb:eips/amsterdam/eip-8038-access-list-cost

Conversation

@spencer-tb

@spencer-tb spencer-tb commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Description

EIP-8038 sets TX_ACCESS_LIST_ADDRESS/STORAGE_KEY equal to the new 3000 cold costs, making a prepaid entry cost more than the cold access it replaces. This PR now derives both as the cold cost minus WARM_ACCESS (2900) so prepaying is gas neutral, with EIP-2930's 100 discount intentionally not restored.

Related Issues or PRs

N/A.

Checklist

  • Ran fast static checks to avoid CI fails, see Code Standards & Verifying Changes: just static
  • PR title has the form <type>(<area>): <title>, where <type> and <area> come from an appropriate C-<type>, respectively A-<area>, label. The title should match the target squash commit message.

Cute Animal Picture

Put a link to a cute animal picture inside the parenthesis-->

@codecov

codecov Bot commented Jul 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.49%. Comparing base (a4e9212) to head (9a51ca2).

Additional details and impacted files
@@               Coverage Diff                @@
##           forks/amsterdam    #3271   +/-   ##
================================================
  Coverage            93.49%   93.49%           
================================================
  Files                  625      625           
  Lines                37032    37032           
  Branches              3385     3385           
================================================
  Hits                 34623    34623           
  Misses                1653     1653           
  Partials               756      756           
Flag Coverage Δ
unittests 93.49% <100.00%> (ø)

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

☔ View full report in Codecov by Harness.
📢 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.

@spencer-tb
spencer-tb force-pushed the eips/amsterdam/eip-8038-access-list-cost branch from 3a6caa0 to 9a51ca2 Compare July 31, 2026 08:12
@spencer-tb spencer-tb added A-spec-specs Area: Specification—The Ethereum specification itself (eg. `src/ethereum/*`) C-feat Category: an improvement or new feature A-tests Area: Consensus tests. labels Jul 31, 2026
@spencer-tb spencer-tb changed the title fix(spec-specs,tests): reprice EIP-8038 access-list costs to cold minus warm feat(spec-specs,tests): reprice EIP-8038 access-list costs to cold minus warm Jul 31, 2026
@spencer-tb
spencer-tb marked this pull request as ready for review July 31, 2026 08:18
@spencer-tb

Copy link
Copy Markdown
Contributor Author

Going to merge for the release. We can remove later is consensus disagrees.

@spencer-tb
spencer-tb merged commit a2571e4 into ethereum:forks/amsterdam Jul 31, 2026
26 checks passed
@misilva73

Copy link
Copy Markdown

Update to EIP is here: ethereum/EIPs#12056

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. C-feat Category: an improvement or new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants