mirror of
https://github.com/deepseek-ai/DeepEP.git
synced 2026-11-05 00:47:32 +00:00
[GH-ISSUE #415] low_latency_test traffic in dual-port networks is unbalanced #280
Labels
No labels
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
deepseek-ai/DeepEP#280
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 @PhdShi on GitHub (Sep 18, 2025).
Original GitHub issue: https://github.com/deepseek-ai/DeepEP/issues/415
After the test_low_latency test, it was found that the low latency IBGDA qp created through mlx5dv_devx_obj_create would only go through one of the ports in the dual-port network, even though we created num_experts/nranks qps.
For instance, for 2 H20 servers with nranks=16 and 288 experts, by modifying the topk tensor to make it that Rank 0 data will only be sent to Rank 8, only one eth port will be used.
Is it as expected? Thanks.
@alpha-baby commented on GitHub (Sep 18, 2025):
Hello, what command did you use to check the real-time bandwidth of the device? It seems that this is not in line with expectations.
@PhdShi
@PhdShi commented on GitHub (Sep 18, 2025):
rdma_monitor
@PhdShi commented on GitHub (Sep 19, 2025):
fixed by modifying the udp src port of nvshmem IBGDA
@alpha-baby commented on GitHub (Sep 19, 2025):
How did you modify it? I understand that this udp-src port should not be a fixed value by default.
@PhdShi commented on GitHub (Sep 19, 2025):
I think your understanding is correct. However, nvshmem change this value to 49152 after IBGDA RC is created
@alpha-baby commented on GitHub (Sep 19, 2025):
I didn't find the magic number related to 49152 in the nvshmem source code. how do you find this magic number of 49152. tcpdump ?
how do you modify udp-src port ?Configure an environment variable with NVSHMEM_ as the prefix?
@PhdShi commented on GitHub (Sep 19, 2025):
modify nvshmem source code.
#define IBGDA_ROCE_V2_UDP_SPORT_BASE 0xC000