Repository navigation
[Doc][M9] PD and KV transfer walkthrough - #142
Open
YouHengfei wants to merge 1 commit into
Open
YouHengfei wants to merge 1 commit into
YouHengfei wants to merge 1 commit into
Conversation
YouHengfei
force-pushed
the
docs/m9-pd-refactoring
branch
from
October 23, 2026 03:53
832c993 to
db4922c
Compare
Add a maintained architecture document for PD disaggregation: component ownership, the reserve/prefill/commit/received/ack/activation/release handoff, prefix and layout handling, reservation credits, cancellation and failure ordering, lease and preemption interaction, profiling control routing, and supported scope. Index it in docs/README.md. Follows the maintained kv_cache_walkthrough.md style: current-state mechanics without baseline pinning or review framing. Refs InferMatrixLab#32, InferMatrixLab#111. Signed-off-by: youhengfei <474029121@qq.com>
YouHengfei
force-pushed
the
docs/m9-pd-refactoring
branch
from
October 26, 2026 10:14
db4922c to
0203a2e
Compare
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.
Required human contributor answers
1. What outcome are you trying to achieve?
Provide the M9 PD walkthrough as maintained architecture documentation, replacing the closed, unmerged #64 and reflecting the current configuration and prefill execution contracts.
2. What is the highest-leverage variable for that outcome?
Documentation accuracy and maintainability: explain current ownership and handoff mechanics rather than retain a historical review report.
3. How will this work move that variable rather than create busywork?
The contributor approved maintaining the walkthrough in the style of
docs/kv_cache_walkthrough.md, then reviewed and approved this update after #140 merged. The document explains entry-owned configuration, scheduler-driven prefill/export boundaries and resource retirement, with links to their implementations. Review findings remain separate from the living documentation.4. Who is the customer or user, and what do they actually need?
Maintainers and RFC #111 participants reviewing PD contracts need a current description of which component owns configuration, request execution, transfer progress and safe resource release.
PR necessity and bugfix evidence
Yes: the contributor approved replacing the closed #64 with maintained in-tree documentation through this PR. This is a documentation change, not a runtime bugfix or new feature implementation.
Purpose
Refs #32 and #111. Add
docs/pd_walkthrough.mdand its documentation-index entry. Rebased onto main8fbf4cd, including #140's unified model configuration and #144's wavefront prefill integration.The walkthrough covers:
ModelConfig, spawned configuration copies and runtime-only ready messages; P/D fingerprints still verify runtime compatibility.receivedtriggersackindependently of compute admission.The document describes current behavior without pinned-baseline review framing or historical defect lists. No runtime code is changed.
Test Plan
vllm-rlt Version / Commit:
0203a2e323ea5f53e959c4c5d1bf5b3d89f67258, one documentation commit based on main8fbf4cd9d4c96a84dd64b40f4b56f61e6658d66a.git diff --check.8fbf4cd: CPU tests for PD, transfer connectors, stage engines and split/wavefront prefill.Test Result
git diff --check: passed.