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

[DO NOT MERGE] EIP-7742 Interop #8011

Closed
wants to merge 10 commits into from
Closed

Conversation

siladu
Copy link
Contributor

@siladu siladu commented Dec 10, 2024

POC branch for remainder of EIP-7742, most code is already on main or on #7840 which is pending review.

Not ready to merge as it is a quick a dirty solution to make interop work.

Based on #7840 and aims to resolve the remaining two pieces to enable interop with a 6/9 network:

The max isn't wired through to block creation so that still assumes Cancun's 6 blobs max.

Resolved with siladu@f4140c8 "Wire maxBlobsPerBlock into PragueTargetingGasLimitCalculator"

The 4844 block/tx max blob validation (via newPayload) hasn't been skipped for Prague so a newPayload with > 6 blobs may fail

Still TODO

  1. Pass execution-spec-tests once available

Still TODO

Signed-off-by: Simon Dudley <simon.dudley@consensys.net>
…eUpdatedV4

Signed-off-by: Simon Dudley <simon.dudley@consensys.net>
Signed-off-by: Simon Dudley <simon.dudley@consensys.net>
Signed-off-by: Simon Dudley <simon.dudley@consensys.net>
Signed-off-by: Simon Dudley <simon.dudley@consensys.net>
Signed-off-by: Simon Dudley <simon.dudley@consensys.net>
Signed-off-by: Simon Dudley <simon.dudley@consensys.net>
Signed-off-by: Simon Dudley <simon.dudley@consensys.net>
Signed-off-by: Simon Dudley <simon.dudley@consensys.net>
Signed-off-by: Simon Dudley <simon.dudley@consensys.net>
@siladu
Copy link
Contributor Author

siladu commented Dec 17, 2024

Removed from Pectra

@siladu siladu closed this Dec 17, 2024
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.

1 participant