Skip to content
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.

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

Add EIP: Linear EVM memory limits #8435

Merged
merged 4 commits into from
Apr 18, 2024
Merged

Add EIP: Linear EVM memory limits #8435

merged 4 commits into from
Apr 18, 2024

Conversation

vbuterin
Copy link
Contributor

Add a hard memory limit equal to the gas limit of the current context. Make the maximum gas cost of a sub-call depend on the memory used in the current context. The two rules together ensure that a transaction with N gas can use at most N bytes of memory.

@vbuterin vbuterin requested a review from eth-bot as a code owner April 15, 2024 12:32
@github-actions github-actions bot added c-new Creates a brand new proposal s-draft This EIP is a Draft t-core labels Apr 15, 2024
@eth-bot
Copy link
Collaborator

eth-bot commented Apr 15, 2024

✅ All reviewers have approved.

@eth-bot eth-bot added e-consensus Waiting on editor consensus e-review Waiting on editor to review labels Apr 15, 2024
@eth-bot eth-bot changed the title New EIP: Linear EVM memory limits Add EIP: Linear EVM memory limits Apr 15, 2024
EIPS/eip-7669.md Outdated
@@ -0,0 +1,75 @@
---
eip: 7669
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
eip: 7669
eip: 7686

EIPS/eip-7686.md Outdated Show resolved Hide resolved
EIPS/eip-7686.md Outdated Show resolved Hide resolved
vbuterin and others added 2 commits April 18, 2024 11:53
Co-authored-by: Andrew B Coathup <28278242+abcoathup@users.noreply.github.com>
Co-authored-by: g11tech <develop@g11tech.io>
Copy link
Contributor

@gcolvin gcolvin left a comment

Choose a reason for hiding this comment

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

I'm slightly worried about the backwards compatibility issue you mention, but only slightly.

@eth-bot eth-bot enabled auto-merge (squash) April 18, 2024 17:58
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 45eb0c3 into master Apr 18, 2024
10 checks passed
@eth-bot eth-bot deleted the vbuterin-patch-1 branch April 18, 2024 17:59
@QBlockQ
Copy link

QBlockQ commented Apr 23, 2024

Fair enough!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c-new Creates a brand new proposal e-consensus Waiting on editor consensus e-review Waiting on editor to review s-draft This EIP is a Draft t-core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants