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

adjust tpu coalesce channel size to 250K #4859

Merged

Conversation

lijunwangs
Copy link

Problem

Based on further testing of TPU coalescer channel size on the receive performance. Adjust the size to reduce memory usage.

Summary of Changes

Found the channel size with 250K has similar performance with the size of 1 million (2.8 M/5s). When the size drops to 100K, it drops to about 2.18 M/5s

Fixes #

Copy link

@alexpyattaev alexpyattaev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but please document how you have measured the performance (and on which hardware) so we can reproduce/backtrack later.

@lijunwangs
Copy link
Author

lijunwangs commented Feb 10, 2025

Using bench-vote tool

Run server

./target/release/solana-bench-vote --use-connection-cache --num-recv-sockets 32 --num-producers 1 --server-only --server-address 0.0.0.0:11228 --verbose --identity ~/validator-keypair.json --use-quic true

Run client:

./target/release/solana-bench-vote --num-recv-sockets 1 --num-producers 32 --client-only --server-address 145.40.90.189:11228 --use-quic true --identity ~/validator-keypair.json

Then monitor the new_streams in the metrics bench_vote_metrics.

Hardware:

*-core
description: Motherboard
physical id: 0
*-memory
description: System memory
physical id: 0
size: 512GiB
*-cpu
product: AMD EPYC 7502P 32-Core Processor
vendor: Advanced Micro Devices [AMD]
physical id: 1
bus info: cpu@0
version: 23.49.0
width: 64 bits

I was using the branch https://github.com/lijunwangs/solana/tree/bench_vote_quic_mcep.2 which has some tweaks on the client side to maximize performance for this exercise.

@lijunwangs lijunwangs merged commit 9db430e into anza-xyz:master Feb 10, 2025
66 checks passed
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