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

eip6800: add concurrent execution information #3827

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

Conversation

gballet
Copy link
Member

@gballet gballet commented Jul 2, 2024

Modifies the stateless witness to mark each stem with a transaction access bitmap. This bitmap is used by the client to run compatible transactions concurrently.

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.

awesome, once could construct partial order chains using this for parallel execution

@jsign
Copy link

jsign commented Jul 5, 2024

I'm not yet convinced about the utility of this change for the following reasons:

  • The EVM execution today isn't a bottleneck in verifiers.
  • In the Verkle fork, we aren't planning to 10x the gas limit or similar, which can change the above bullet assumption.
  • The end game is SNARKifying L1, so this will eventually become irrelevant.

If we 10x the gas limit or similar in a future fork after Verkle, then maybe it will make sense at that point if we have proof that ELs would have a speedup benefit that justifies by increasing the witness size or impl complexity for something that will be abandoned long-term.

@gballet
Copy link
Member Author

gballet commented Jul 5, 2024

To be clear, I'm not thinking of this for Osaka. It should de defined as a different upgrade. I'm thinking of a better way of doing this anyways, I'll redo the proposal then.

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.

3 participants