Skip to content

Update EIP-8037: add additional tx validity condition#11503

Closed
jwasinger wants to merge 3 commits into
ethereum:masterfrom
jwasinger:eip-8037-tx-validity
Closed

Update EIP-8037: add additional tx validity condition#11503
jwasinger wants to merge 3 commits into
ethereum:masterfrom
jwasinger:eip-8037-tx-validity

Conversation

@jwasinger

@jwasinger jwasinger commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

This just codifies a rule that was already implemented by all (?) clients and eest.

it feels like the change would be more consistent with the spirit of the EIP if the validity condition was:

Inclusion of a transaction in a block requires that the cumulative gas used in the bottleneck dimension added to tx.gas_limit not exceed the block gas limit.

but with this PR it's:

Inclusion of a transaction in a block requires that the cumulative regular gas used of all previous transactions added to min(TX_MAX_GAS_LIMIT, tx.gas_limit) not exceed the block gas limit.

… sum of all regular gas used in the block thus far not exceed the block gas limit
@jwasinger jwasinger requested a review from eth-bot as a code owner April 9, 2026 16:48
@github-actions github-actions Bot added c-update Modifies an existing proposal s-draft This EIP is a Draft t-core labels Apr 9, 2026
@eth-bot

eth-bot commented Apr 9, 2026

Copy link
Copy Markdown
Collaborator

File EIPS/eip-8037.md

Requires 1 more reviewers from @adietrichs, @anderselowsson, @CPerezz, @fradamt, @jochem-brouwer, @LukaszRozmej, @misilva73

@spencer-tb spencer-tb 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.

Small suggestion to align with the outcome of the gas repricings call on 15/04/26!

Comment thread EIPS/eip-8037.md Outdated
Co-authored-by: spencer <spencer.tb@ethereum.org>
@github-actions

Copy link
Copy Markdown

The commit 4699eab (as a parent of b1a38c7) contains errors.
Please inspect the Run Summary for details.

@misilva73 misilva73 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.

I don't think this check is correct. Also, I think we can make the wording fit better the the next paragraph. I created this PR to replace it: #11536

@jwasinger jwasinger closed this Apr 20, 2026
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 w-ci Waiting on CI to pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants