deepseek-ai/DeepEP
1
0
Fork 0
mirror of https://github.com/deepseek-ai/DeepEP.git synced 2026-11-05 00:47:32 +00:00
Code Issues 308 Projects Packages Wiki Activity

[GH-ISSUE #69] Does normal internode_dispatch/internode_combine need IBGDA in adaptive routing scenario? #66

New issue
Closed
opened 2026-10-26 09:53:37 +00:00 by bsakaguchi · 2 comments
bsakaguchi commented 2026-10-26 09:53:37 +00:00
Owner
Copy link

Originally created by @Tuvie on GitHub (Mar 12, 2025).
Original GitHub issue: https://github.com/deepseek-ai/DeepEP/issues/69

I notice there was a discussion on IBGDA for test_internode, which mentioned that when AR is supported later, IBGDA is always required.

https://github.com/deepseek-ai/DeepEP/issues/36

I wonder why IBGDA is required in AR case.
If I still use IBRC, what kind of bug will I encounter?

Thanks

Originally created by @Tuvie on GitHub (Mar 12, 2025). Original GitHub issue: https://github.com/deepseek-ai/DeepEP/issues/69 I notice there was a discussion on IBGDA for test_internode, which mentioned that when AR is supported later, IBGDA is always required. https://github.com/deepseek-ai/DeepEP/issues/36 I wonder why IBGDA is required in AR case. If I still use IBRC, what kind of bug will I encounter? Thanks
bsakaguchi commented 2026-10-26 09:53:38 +00:00
Author
Owner
Copy link

@sphish commented on GitHub (Mar 12, 2025):

We previously aimed to support AR using an RDMA completion-based mechanism, which is easier to implement with IBGDA. However, we have now switched to using RDMA atomic mechanisms to support AR for normal kernels while still utilizing IBRC mode, so the earlier discussions are now outdated.

<!-- gh-comment-id:2717195186 --> @sphish commented on GitHub (Mar 12, 2025): We previously aimed to support AR using an RDMA completion-based mechanism, which is easier to implement with IBGDA. However, we have now switched to using RDMA atomic mechanisms to support AR for normal kernels while still utilizing IBRC mode, so the earlier discussions are now outdated.
bsakaguchi commented 2026-10-26 09:53:38 +00:00
Author
Owner
Copy link

@Tuvie commented on GitHub (Mar 13, 2025):

I see. Thanks for your reply.

<!-- gh-comment-id:2719727441 --> @Tuvie commented on GitHub (Mar 13, 2025): I see. Thanks for your reply.
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
hybrid-ep
chore/remove-clang-format-yapf
v2.5
nvDev
try-fix-cu128
mori-ep
antgroup-opt
compat-old-torch
add-linter
eager-rdma
v1.2.1
Labels
Clear labels   
pull-request
Mirrored from GitHub Pull Request

No labels
pull-request
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
deepseek-ai/DeepEP#66
No description provided.