Skip to content

Conversation

vgao1996
Copy link
Contributor

@vgao1996 vgao1996 commented Oct 9, 2025

This adds a script to automate building and running Rust binaries with PGO.

Example usage:

   ./pgo.sh profile a.profdata
   ./pgo.sh run a.profdata -p aptos-vm-profiling --bin run-aptos-p2p
                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
                           You can replace this with arbitrary cargo run args

Build without running:

   ./pgo.sh build a.profdata -p aptos-vm-profiling --bin run-aptos-p2p

@vgao1996 vgao1996 requested review from sionescu and zekun000 October 9, 2025 23:11
@vgao1996 vgao1996 requested a review from a team as a code owner October 9, 2025 23:11
@vgao1996 vgao1996 requested a review from mkurnikov October 9, 2025 23:11
@vgao1996 vgao1996 force-pushed the pgo branch 2 times, most recently from e96167c to 753c389 Compare October 10, 2025 00:20
@vgao1996 vgao1996 enabled auto-merge (squash) October 15, 2025 20:04

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

Copy link
Contributor

✅ Forge suite compat success on cef91202017aab6af466f948737fcd7d97dc7d13 ==> 169b1d8a401c0a79fe5bb5ffa677dc67ea0268c7

Compatibility test results for cef91202017aab6af466f948737fcd7d97dc7d13 ==> 169b1d8a401c0a79fe5bb5ffa677dc67ea0268c7 (PR)
1. Check liveness of validators at old version: cef91202017aab6af466f948737fcd7d97dc7d13
compatibility::simple-validator-upgrade::liveness-check : committed: 14316.82 txn/s, latency: 2383.20 ms, (p50: 2500 ms, p70: 2600, p90: 2700 ms, p99: 3100 ms), latency samples: 477180
2. Upgrading first Validator to new version: 169b1d8a401c0a79fe5bb5ffa677dc67ea0268c7
compatibility::simple-validator-upgrade::single-validator-upgrade : committed: 5175.62 txn/s, latency: 6601.80 ms, (p50: 7300 ms, p70: 7400, p90: 7500 ms, p99: 7600 ms), latency samples: 181100
3. Upgrading rest of first batch to new version: 169b1d8a401c0a79fe5bb5ffa677dc67ea0268c7
compatibility::simple-validator-upgrade::half-validator-upgrade : committed: 5301.62 txn/s, latency: 6502.54 ms, (p50: 7200 ms, p70: 7300, p90: 7400 ms, p99: 7400 ms), latency samples: 182920
4. upgrading second batch to new version: 169b1d8a401c0a79fe5bb5ffa677dc67ea0268c7
compatibility::simple-validator-upgrade::rest-validator-upgrade : committed: 8386.55 txn/s, latency: 4064.61 ms, (p50: 4400 ms, p70: 4500, p90: 4600 ms, p99: 4800 ms), latency samples: 278180
5. check swarm health
Compatibility test for cef91202017aab6af466f948737fcd7d97dc7d13 ==> 169b1d8a401c0a79fe5bb5ffa677dc67ea0268c7 passed
Test Ok

Copy link
Contributor

✅ Forge suite realistic_env_max_load success on 169b1d8a401c0a79fe5bb5ffa677dc67ea0268c7

two traffics test: inner traffic : committed: 14071.27 txn/s, submitted: 14071.79 txn/s, expired: 0.53 txn/s, latency: 2672.62 ms, (p50: 2400 ms, p70: 2600, p90: 2700 ms, p99: 4800 ms), latency samples: 5350260
two traffics test : committed: 100.00 txn/s, latency: 1465.13 ms, (p50: 1300 ms, p70: 1500, p90: 2000 ms, p99: 9700 ms), latency samples: 1760
Latency breakdown for phase 0: ["MempoolToBlockCreation: max: 2.016, avg: 1.365", "ConsensusProposalToOrdered: max: 0.166, avg: 0.164", "ConsensusOrderedToCommit: max: 0.586, avg: 0.446", "ConsensusProposalToCommit: max: 0.750, avg: 0.610"]
Max non-epoch-change gap was: 1 rounds at version 35321 (avg 0.00) [limit 4], 9.11s no progress at version 5495922 (avg 0.07s) [limit 15].
Max epoch-change gap was: 1 rounds at version 5484101 (avg 0.50) [limit 4], 1.57s no progress at version 5484101 (avg 0.97s) [limit 16].
Test Ok

@vgao1996 vgao1996 merged commit 490e80c into aptos-labs:main Oct 15, 2025
46 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.

3 participants