Skip to content

Update EIP-8037: Update parameters and rationale section#11616

Merged
eth-bot merged 7 commits into
ethereum:masterfrom
misilva73:update-eip-8037-numbers
May 13, 2026
Merged

Update EIP-8037: Update parameters and rationale section#11616
eth-bot merged 7 commits into
ethereum:masterfrom
misilva73:update-eip-8037-numbers

Conversation

@misilva73

@misilva73 misilva73 commented May 7, 2026

Copy link
Copy Markdown
Contributor
  • Recalibrated to a 150M reference block gas limit and 120 GiB/year target, raising CPSB from 1174 → 1530
  • Updated STATE_BYTES_PER_STORAGE_SET (32 → 64) and STATE_BYTES_PER_NEW_ACCOUNT (112 → 120) to include the leaf's key in the on-disk footprint; added a section deriving these byte sizes
  • Added rationale for the 150M and 120 GiB targets, including a worst-case growth table across 100M–300M block limits
  • Refreshed motivation figures to a January 2026 baseline (~390 GiB state, ~326 MiB/day, ~116 GiB/year post 30M→60M)
  • Replaced hardcoded regular-gas values with references to STORAGE_WRITE, ACCOUNT_WRITE, CREATE_ACCESS, COLD_ACCOUNT_ACCESS (EIP-8038) and PRECOMPILE_ECRECOVER (EIP-7904); added EIP-7904 to requires
  • Defined REGULAR_PER_AUTH_BASE_COST inline and clarified intrinsic regular/state gas accounting
  • Restructured the price-impact table to show state-gas increases plus ETH cost at 0.08 Gwei
  • Misc. typo and wording fixes (SSTORE table, SELFDESTRUCT refunds, revert handling)
  • Removed quantization_analysis.ipynb; refreshed new_state_added.png

@github-actions github-actions Bot added c-update Modifies an existing proposal s-draft This EIP is a Draft t-core labels May 7, 2026
@eth-bot

eth-bot commented May 7, 2026

Copy link
Copy Markdown
Collaborator

✅ All reviewers have approved.

@eth-bot eth-bot changed the title EIP-8037: Update parameters and rationale section Update EIP-8037: Update parameters and rationale section May 7, 2026
@github-actions github-actions Bot added the w-ci Waiting on CI to pass label May 7, 2026
@github-actions

github-actions Bot commented May 7, 2026

Copy link
Copy Markdown

The commit 2fda2b5 (as a parent of dcd40a7) contains errors.
Please inspect the Run Summary for details.

@misilva73 misilva73 marked this pull request as ready for review May 13, 2026 13:46
@misilva73 misilva73 requested a review from eth-bot as a code owner May 13, 2026 13:46
CarlBeek added a commit to CarlBeek/reth that referenced this pull request May 13, 2026
The current draft of EIP-8037 (PR ethereum/EIPs#11616, "Update numbers
and rationale") raises the state-byte parameters:

  CPSB:                       1174 → 1530
  STATE_BYTES_PER_STORAGE_SET:  32 → 64
  STATE_BYTES_PER_NEW_ACCOUNT: 112 → 120
  STATE_BYTES_PER_AUTH_BASE:    23 (unchanged)

revm ships the old numbers hardcoded into `with_spec_and_mainnet_gas_params(AMSTERDAM)`,
so we now overlay the new values via `GasParams::override_gas` after
constructing the cfg env. The same overrides also drive the intrinsic-gas
path: `calculate_initial_tx_gas` builds its own GasParams internally with
no override hook, so we instantiate `GasParams::new_spec(AMSTERDAM)`
ourselves, apply the overrides, and call `initial_tx_gas` directly.

Renamed the regular-gas placeholders to match EIP-8038's naming
(CREATE_ACCESS / STORAGE_WRITE / ACCOUNT_WRITE /
REGULAR_PER_AUTH_BASE_COST) and noted in docstrings that those values are
still TBD per the PR. Added ACCOUNT_WRITE (didn't exist before).

config_fingerprint now embeds every PR-11616-tracked parameter and tags
the spec as `pr-11616`. Any existing block_coverage rows from the old
CPSB=1174 era will fingerprint-miss and re-analyse on the next pass.

5 new unit tests lock the constants, the intrinsic-gas math for both
create-tx and authorization paths, the runtime gas-table overrides, and
the fingerprint string. All 131 lib tests green.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@misilva73

Copy link
Copy Markdown
Contributor Author

@eth-bot rerun

@eth-bot eth-bot enabled auto-merge (squash) May 13, 2026 15:02

@eth-bot eth-bot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

All Reviewers Have Approved; Performing Automatic Merge...

@eth-bot eth-bot merged commit 34258fc into ethereum:master May 13, 2026
16 of 21 checks passed
@misilva73 misilva73 deleted the update-eip-8037-numbers branch May 13, 2026 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c-update Modifies an existing proposal s-draft This EIP is a Draft t-core

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants