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

Performance: PoH Record #4264

Open
bw-solana opened this issue Jan 3, 2025 · 0 comments
Open

Performance: PoH Record #4264

bw-solana opened this issue Jan 3, 2025 · 0 comments
Assignees

Comments

@bw-solana
Copy link

bw-solana commented Jan 3, 2025

With high number of txs, the channel used by poh.record() does a catastrophic number of syscalls and takes like 70% of the poh service thread.

Banking stage sends batch of transactions + hash to be mixed in with ticks to poh recorder and waits for result to decide if we can commit the state updates.

This is easy to fix, we know how to fix, we just need to write the code. We want to signal banking stage immediately if the tx will be included in the slot or not. The only case it will not is if we have reached the end of the slot. We don’t want banking stage thread to have to sleep. PoH can defer the actual mixing in of the tx to later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In progress
Development

No branches or pull requests

2 participants