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

Convert eip7667 into an rip #26

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Convert eip7667 into an rip #26

wants to merge 2 commits into from

Conversation

Eikix
Copy link

@Eikix Eikix commented Jun 14, 2024

Hi all!

Posting my first RIP here (literally just converting an existing EIP into an RIP).
Talked to Vitalik this morning about bumping gas price of hash functions for ZK-Rollups eagerly, as proposed in: https://eips.ethereum.org/EIPS/eip-7667

Still in draft, particularly on pricing for keccak / SHA256 etc.; Values posted by Vitalik are still very much placeholders.

Actionables by Kakarot ZK-EVM team:

  • Check with Starkware STWO team a reasonable gas price for each hash functions listed in the RIP.
  • Check with other rollups if their internal pricing is close enough to the one we come up with so that everyone can adopt a common value. Otherwise, we can plan a per-rollup pricing of keccak.

Closing notes: this RIP is a specific type of RIP that only benefits ZK-Rollups. As such, I don't expect ORUs to adopt or consider this EIP, as they don't suffer from non-ZK friendly hash functions.

@Eikix Eikix changed the title convert vbuterin eip into an rip Convert eip7667 into an rip Jun 14, 2024
@Eikix Eikix mentioned this pull request Jun 14, 2024
@abcoathup
Copy link

@Eikix as the RIP is a clone of the EIP, with potential for divergence in the spec now or in the future between EIP and RIP, then it should have a different number. I am issuing 7724.

If @vbuterin permits, then you should be added as an author to the RIP so you can steer the RIP through the process

RIPS/rip-7667.md Outdated
@@ -0,0 +1,70 @@
---
rip: 7667

Choose a reason for hiding this comment

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

Suggested change
rip: 7667
rip: 7724

Assigning next sequential EIP/ERC/RIP number.

Please also update the filename.

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

Successfully merging this pull request may close these issues.

2 participants