Skip to content

Update EIP-7907: Account for large contracts as tx entry point#9955

Merged
eth-bot merged 2 commits into
ethereum:masterfrom
marioevz:eip-7907-tx-to-large-contract-charge
Jul 3, 2025
Merged

Update EIP-7907: Account for large contracts as tx entry point#9955
eth-bot merged 2 commits into
ethereum:masterfrom
marioevz:eip-7907-tx-to-large-contract-charge

Conversation

@marioevz

Copy link
Copy Markdown
Member

Clarify behavior when a large contract is the entry point of a transaction.

@marioevz marioevz requested a review from eth-bot as a code owner June 30, 2025 15:15
@github-actions github-actions Bot added c-update Modifies an existing proposal s-draft This EIP is a Draft t-core labels Jun 30, 2025
@eth-bot

eth-bot commented Jun 30, 2025

Copy link
Copy Markdown
Collaborator

✅ All reviewers have approved.

Comment thread EIPS/eip-7907.md Outdated
`COLD_ACCOUNT_ACCESS_COST` and `WARM_STORAGE_READ_COST` are defined in [EIP-2929](./eip-2929.md#parameters).

3. Update the [EIP-3860](./eip-3860.md) contract initcode size limit of 48KB (`0xc000` bytes) to 512KB (`0x80000` bytes).
4. If a large contract is the entry point of a transaction, the cost calculated in (2) is charged before the execution of any opcode and contract code becomes warm immediately. The execution can run out-of-gas at this point if not enough gas was provided with the transaction to pay for this extra cost.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Would be good to specify what would happen with balance transfered (maybe reverted? Not sure what is best). And that this gas doesn't get included in initial gas fee.

@rakita rakita left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We should include this. Left nit on clarification around balance revert.

Comment thread EIPS/eip-7907.md Outdated
Co-authored-by: rakita <rakita@users.noreply.github.com>

@rakita rakita left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approved in ACDE

@charles-cooper charles-cooper left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Awesome. Thanks!

@eth-bot eth-bot enabled auto-merge (squash) July 3, 2025 15:41

@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 e9ec317 into ethereum:master Jul 3, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a-review Waiting on author to review 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.

4 participants