mirror of
https://github.com/kvcache-ai/Mooncake.git
synced 2026-11-05 01:17:55 +00:00
[PR #1076] [MERGED] feat: add PCIe Relaxed Ordering (RO) support and RDMA traffic class (… #1995
Labels
No labels
ascend
auto-closed
bug
bug
CI/Build
Common
documentation
enhancement
feature request
good first issue
help wanted
high priority
MetaX
nightly-failure
pull-request
PyTorch Backend
question
RFC
Roadmap
stale
Store
Tests
Transfer Engine
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
kvcache-ai/Mooncake#1995
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/kvcache-ai/Mooncake/pull/1076
Author: @1998zxn
Created: 11/18/2025
Status: ✅ Merged
Merged: 12/15/2025
Merged by: @stmatengss
Base:
main← Head:feature/relaxed-ordering-and-rdma-tc📝 Commits (7)
eee9562add PCIe Relaxed Ordering (RO) support.bf2b5d5fix: add env variable to control Relaxed Ordering (RO)81ca67brefactor: simplify logic in mooncake-transfer-engine/src/transport/rdma_transport/rdma_transport.cpp716cbe3fix(RO): set default value to 0ab3a83ffix(ci): resolve compilation error in PR build118e808Supplement guidance documents1d7ec64refactor: remove dlopen, use dlsym alone for symbol checking📊 Changes
5 files changed (+67 additions, -4 deletions)
View changed files
📝
docs/source/design/transfer-engine/index.md(+1 -0)📝
docs/source/zh_archive/transfer-engine.md(+1 -0)📝
mooncake-transfer-engine/include/config.h(+2 -0)📝
mooncake-transfer-engine/src/config.cpp(+11 -0)📝
mooncake-transfer-engine/src/transport/rdma_transport/rdma_transport.cpp(+52 -4)📄 Description
…TC) control to improve ordering flexibility and queue-level QoS
Description
This PR optimizes Mooncake’s performance in the 2P1D scenario by introducing two main improvements:
These changes effectively reduce KV Cache transfer time, thereby lowering overall TTFT (Time-To-First-Token) latency.
Background
In our deployment scenario using SGLang DeepSeek v3 with 2P1D configuration:
We observed that KV Cache transfers could account for up to 23% of the total TTFT. The reasons are:
This also addresses the issue discussed in #39.
By enabling these two features, we reduced TTFT from 650ms to ~585ms, and KV Cache transfer time dropped to 15% of TTFT, showing significant performance improvements.
Implementation Details
Fully backward compatible; scenarios not using these new features remain unaffected with no regression risk.
Type of Change
Checklist
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.