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

Increase the call's MIN_RETAINED_GAS #95

Open
chfast opened this issue Apr 24, 2024 · 3 comments
Open

Increase the call's MIN_RETAINED_GAS #95

chfast opened this issue Apr 24, 2024 · 3 comments

Comments

@chfast
Copy link
Member

chfast commented Apr 24, 2024

The currently proposed value of MIN_RETAINED_GAS is 5000. This is exactly the cost of updating a "fresh" storage slot (so it cannot be done) but one can update a storage slot that is worm or has been updated/added before the call.

I think we should increase MIN_RETAINED_GAS to the value always allowing updating a storage slot. Also consider adding a new storage value (20000).

@chfast chfast changed the title Tune the call's MIN_RETAINED_GAS Increase the call's MIN_RETAINED_GAS Apr 24, 2024
@axic
Copy link
Member

axic commented Apr 24, 2024

How about also defining the retained gas not by a fixed number, but rather as i.e. "maximum cost of SSTORE * 1.5x" ?

@chfast
Copy link
Member Author

chfast commented Apr 24, 2024

Figure_1

(the legacy CALL has min retained gas 0)

@chfast
Copy link
Member Author

chfast commented Apr 25, 2024

The script for the analysis: #97.
Please review the math there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants