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

Transaction recipient should be warm #903

Merged
merged 1 commit into from
Feb 26, 2024

Conversation

pldespaigne
Copy link
Contributor

TL;DR

Before After
image image

Problem

The Berlin hardfork introduced EIP-2929 which is now part of the Yellow Paper specification. This EIP states that the transaction recipient address should be part of the initial accessed account set:
image

The Yellow Paper does enforce this behavior only for contract creation and subsequent xCALL opcodes but not for the initial message call.

Proposed Solution

When the transaction is a message call ($T_t \neq \varnothing$) add the recipient address ($T_t$) to the accessed account set, along with the already present precompiled, sender address, and access list.


I would be happy to discuss this further or to make some change to this PR if needed .
Anyway thanks for your time!

@yperbasis yperbasis merged commit 10a7971 into ethereum:master Feb 26, 2024
1 check passed
@pldespaigne
Copy link
Contributor Author

Hey @yperbasis 👋
Thanks for the review!

I would be interested in updating the YP to the Shangai fork and next, to the Cancun fork, is anyone already working on this?
Do you think I could ask for a small EF grant?

@yperbasis
Copy link
Member

Hey @yperbasis 👋 Thanks for the review!

I would be interested in updating the YP to the Shangai fork and next, to the Cancun fork, is anyone already working on this? Do you think I could ask for a small EF grant?

Thank you for the patch @pldespaigne! I'm not aware of anyone working on the Shanghai/Cancun upgrades – feel free to pick it. Yes, you can probably apply for a EF grant on https://esp.ethereum.foundation/applicants. cc @timbeiko

@pldespaigne
Copy link
Contributor Author

Thanks!
I already applied, I'm waiting for them to answer now 🙂

fulldecent pushed a commit to fulldecent/yellowpaper that referenced this pull request Dec 31, 2024
@pldespaigne pldespaigne deleted the warm-tx-to branch January 16, 2025 15:13
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