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

core: attempt at alternative tx-level prefetcher #30389

Closed
wants to merge 1 commit into from

Conversation

holiman
Copy link
Contributor

@holiman holiman commented Sep 4, 2024

This is a proof-of-concept test of an idea based off a discussion with @rjl493456442 . We can try to prefetch transcations in the current block.

Current, prior to this PR, we have

  1. Prefetcher which executes the next block opportunistically. This is not used in regular operations, since we usually don't have the next block -- only while catching up.
  2. Prefetching the trie. This happens post-execution: after executing tx 1, we try to prefetch the trie so we later on can do the commit faster.

This PR changes the 1. prefetcher, so that instead of pre-executing the next block, we skip ahead across a third of the transactions in the block, and (opportunistically) execute it in order to pre-load stuff into caches.

@holiman
Copy link
Contributor Author

holiman commented Sep 17, 2024

Screenshot 2024-09-17 at 09-31-23 Dual Geth - Grafana

I've had this running for a couple of days now, versus master. I see no meaningful difference with this PR, so closing.

@holiman holiman closed this Sep 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