Skip to content

Update EIP-7825: Lower Tx Gas Limit Cap #9986

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
Merged
merged 2 commits into from
Jul 13, 2025
Merged

Update EIP-7825: Lower Tx Gas Limit Cap #9986

merged 2 commits into from
Jul 13, 2025

Conversation

@nerolation
Copy link
Contributor

@nerolation nerolation commented Jul 7, 2025

Based on this new EIP and discussions on ACDT, I created this PR to proposer lowering the tx gas limit cap.

@nerolation nerolation requested a review from eth-bot as a code owner July 7, 2025 15:12
@github-actions github-actions bot added c-update Modifies an existing proposal s-stagnant This EIP is Stagnant t-core labels Jul 7, 2025
@eth-bot
Copy link
Collaborator

eth-bot commented Jul 7, 2025 •
edited
Loading

✅ All reviewers have approved.

@eth-bot eth-bot added the a-review Waiting on author to review label Jul 7, 2025
@eth-bot eth-bot changed the title Modify EIP-7825: Lower Gas Limit Cap Update EIP-7825: Lower Gas Limit Cap Jul 7, 2025
benaadams
benaadams reviewed Jul 7, 2025
View reviewed changes
EIPS/eip-7825.md Outdated
description: Introduce a protocol-level cap on the maximum gas used by a transaction to 16,777,216 (2^24).
author: Giulio Rebuffo (@Giulio2002)
discussions-to: https://ethereum-magicians.org/t/eip-7825-transaction-gas-limit-cap/21848
status: Stagnant
Copy link
Contributor

@benaadams benaadams Jul 7, 2025

Choose a reason for hiding this comment

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

Why is it Stagnant if its literally going in a hardfork 😅

Copy link
Member

@jochem-brouwer jochem-brouwer Jul 9, 2025

Choose a reason for hiding this comment

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

I think the EIP bot does this if it stays on Draft for too long 🤔

benaadams
benaadams reviewed Jul 7, 2025
View reviewed changes
@benaadams benaadams mentioned this pull request Jul 7, 2025
Merged
9 tasks
@github-actions github-actions bot added c-status Changes a proposal's status s-draft This EIP is a Draft and removed c-update Modifies an existing proposal s-stagnant This EIP is Stagnant labels Jul 7, 2025
@eth-bot eth-bot changed the title Update EIP-7825: Lower Gas Limit Cap Update EIP-7825: Move to Draft Jul 7, 2025
@Gabriel-Trintinalia Gabriel-Trintinalia mentioned this pull request Jul 8, 2025
Closed
@Gabriel-Trintinalia
Copy link

Gabriel-Trintinalia commented Jul 8, 2025

just double checking that system calls (EIP-2935, EIP-7002, etc) will keep the current limit of 30M?

@benaadams
Copy link
Contributor

benaadams commented Jul 9, 2025

just double checking that system calls (EIP-2935, EIP-7002, etc) will keep the current limit of 30M?

Yeah, system calls are kinda unrelated as they don't count to block gas and if they cause a problem can be fixed in protocol.

Is more for unknown and unpredictable user txs

@gurukamath gurukamath mentioned this pull request Jul 9, 2025
Closed
@spencer-tb spencer-tb mentioned this pull request Jul 9, 2025
Merged
5 tasks
@nerolation nerolation mentioned this pull request Jul 10, 2025
Closed
@benaadams benaadams mentioned this pull request Jul 10, 2025
Closed
@nerolation nerolation changed the title Update EIP-7825: Move to Draft Update EIP-7825: Reduce Max Tx Gas Cap to 16.7m Jul 11, 2025
@jochem-brouwer
Copy link
Member

jochem-brouwer commented Jul 13, 2025 •
edited
Loading

Can it be added in the EIP why it is specifically choosen to be 16,777,216 (2^24) and not 16,777,215 (2^24 - 1)? The former fits in 4 bytes, while the second fits in 3 bytes. I know the other proposed EIP has a small motivation for the 2^24 number which I think should be added here too. (I do not completely grasp the motivation though, but maybe @vbuterin could chime in here?)

@nerolation
Copy link
Contributor Author

nerolation commented Jul 13, 2025

We may still want to gather some feedback from dapp builders. We've already stumbled across apps with emergency modules that were never called on mainnet but could require more gas than 16.7m.

g11tech
g11tech approved these changes Jul 13, 2025
View reviewed changes
Copy link
Contributor

@g11tech g11tech left a comment

Choose a reason for hiding this comment

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

approving, would still requires author's approval

@eth-bot eth-bot changed the title Update EIP-7825: Reduce Max Tx Gas Cap to 16.7m Update EIP-7825: Move to Draft Jul 13, 2025
@benaadams benaadams mentioned this pull request Jul 13, 2025
Closed
Giulio2002
Giulio2002 approved these changes Jul 13, 2025
View reviewed changes
Copy link
Contributor

@Giulio2002 Giulio2002 left a comment

Choose a reason for hiding this comment

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

I shall give you my blessings

@eth-bot eth-bot enabled auto-merge (squash) July 13, 2025 16:41
eth-bot
eth-bot approved these changes Jul 13, 2025
View reviewed changes
Copy link
Collaborator

@eth-bot eth-bot left a comment

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 fc6c724 into ethereum:master Jul 13, 2025
24 checks passed
@nerolation nerolation changed the title Update EIP-7825: Move to Draft Update EIP-7825: Lower Tx Gas Limit Cap Jul 17, 2025
@LouisTsai-Csie LouisTsai-Csie mentioned this pull request Jul 18, 2025
Merged
8 tasks
@taratorio taratorio mentioned this pull request Jul 19, 2025
Closed
advaita-saha added a commit to status-im/nimbus-eth1 that referenced this pull request Jul 21, 2025
@advaita-saha
update EIP-7825 ethereum/EIPs#9986
62eddb1
@advaita-saha advaita-saha mentioned this pull request Jul 21, 2025
Merged
@rkrasiuk rkrasiuk mentioned this pull request Jul 22, 2025
Merged
jangko pushed a commit to status-im/nimbus-eth1 that referenced this pull request Jul 23, 2025
@advaita-saha
update EIP-7825 ethereum/EIPs#9986 (#3490)
bdb0d91
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@g11tech g11tech g11tech approved these changes

@eth-bot eth-bot eth-bot approved these changes

+4 more reviewers

@benaadams benaadams benaadams left review comments

@jochem-brouwer jochem-brouwer jochem-brouwer left review comments

@Giulio2002 Giulio2002 Giulio2002 approved these changes

@joetosly414 joetosly414 joetosly414 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

a-review Waiting on author to review c-status Changes a proposal's status s-draft This EIP is a Draft t-core

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@nerolation @eth-bot @Gabriel-Trintinalia @benaadams @jochem-brouwer @Giulio2002 @g11tech @joetosly414