mirror of
https://github.com/kvcache-ai/Mooncake.git
synced 2026-11-05 01:17:55 +00:00
[GH-ISSUE #3500] [TE]: Track RDMA-parity large-buffer support for EFA/CXI/UB #897
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#897
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?
Originally created by @staryxchen on GitHub (Aug 18, 2026).
Original GitHub issue: https://github.com/kvcache-ai/Mooncake/issues/3500
Background
RDMA Transfer Engine already handles buffers larger than the device MR limit end-to-end:
BufferDescs (#2644).BufferDescremainder (SliceLengthCalculatorinRdmaTransport::submitTransferTask), so a single TE read/write that straddles a chunk seam still works.Because of that, Mooncake Store no longer needs to split RDMA segments at
max_mr_size. #3486 is droppingrdmafromGetTransportRegistrationLimit()and leaving the Store-side split in place for EFA, CXI, and UB.Goal
Bring EFA, CXI, and UB to the same TE behavior as RDMA, so Store only needs a registration-size split for transports that still cannot register or transfer across chunk seams.
After all three sub-issues land,
GetTransportRegistrationLimit()should only apply to transports that still lack this support (today that is expected to be none, or UB until its registration shrink is fixed).Why this is split into sub-issues
I do not have EFA or CXI hardware to implement and verify the submit-time path, and UB is a different gap (it still shrinks oversized registrations instead of chunking). Tracking them separately keeps owners and test requirements clear.
Sub-issues
BufferDescslicing (registration chunking already exists)BufferDescslicing (registration chunking already exists)BufferDescslicingReferences
RDMALargeMrTest.WriteStraddlesChunkBoundary/WriteWithSourceStraddlingChunkBoundary@github-actions[bot] commented on GitHub (Aug 18, 2026):
Thanks for opening this issue, @staryxchen!
151037142A maintainer will triage this when possible. To help us respond faster, please include:
Useful links: Documentation · Contributing guide