perf(dpa): add session-aware routing and post-prefill decode protection - #2016
Open
yhl-amd wants to merge 4 commits into
Open
perf(dpa): add session-aware routing and post-prefill decode protection#2016yhl-amd wants to merge 4 commits into
yhl-amd wants to merge 4 commits into
Conversation
Contributor
🏷️ CI GuideRuns automatically on every eligible PR before approval:
Heavy model tests:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
X-Dynamo-Session-ID/X-Correlation-IDthrough the OpenAI path and load-place new DPA sessions using estimated uncached prefill tokens plus request-equivalent decode pressureEnablement
All new behavior is opt-in: session affinity defaults to off and the post-prefill decode interval defaults to
0. Existing sessions never migrate.Benchmark evidence
On the 3600-second DPA8 agentic comparison, the combined routing + PDI configuration measured:
+0.05%(neutral)-4.15%-3.10%-1.16%-0.23%That run also included the dense low-batch-size CUDA Graph change, so the result is evidence for the combined configuration and is not presented as isolated PDI attribution.
A separate 600-second replay-cost migration experiment reduced input throughput by
0.71%, cache read by0.94 pp, and actual prefix hits by roughly2.04 pp. This PR therefore keeps established session ownership strict and excludes migration.The prior
+4.47%DPA result depended on a separate device-synchronization change and is intentionally not claimed here.Validation
69 passed, 1 skipped: DP routing, API header propagation, prefill delayer, and partial-prefill-tail tests33 passed: sequence and prefill scheduler testscompileallandgit diff --checkpassedScope notes
This PR intentionally excludes device synchronization, session migration, LMCache/offload, MegaMoE fixed-slot transport, and independent-replica experiments.
InferenceX currently reports
Unsupported agentic server metrics backendfor ATOM metrics during final aggregation. Raw AIPerf JSON/CSV export completes successfully; aggregator support is outside this ATOM change.