已合并
skip sub-comm creation when global process group was reinit'd and fix p2p longname err #38988
skip sub-comm creation when global process group was reinit'd and fix p2p longname err #38988
已合并
limuan创建于 6月22日
limuan
limuan成员
6月22日

【合入来源】

【修改方案】

HCCLUdi在rootinfo场景下的P2P通信中,被覆盖为groupname,导致hccl算子拼接后hcclUdi和hcclCommname后,字符超长,拼接后名字:groupbfa08d29a349dccf55aaf7ea17c96695164d8ba6_p2p_0_1_groupbfa08d29a349dccf55aaf7ea17c96695164d8ba6_p2p_0_1
修复方案:Udi仅在rankfiletable场景下,被覆盖为groupname

【资料变更】

不涉及

【接口变更】

不涉及

【功能验证】

ut用例通过:
image.png
pp并行时,正常拉起训练
image.png
Udi正常更新:
image.png

【CheckList】

PR提交人对以下CheckList自检项进行全量自检,自检通过或不涉及,均修改 [ ] 为 [x]

likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 limuan 的贡献)
limuanlimuan成员
6月22日 创建了 pull request,commit 396a2ea4
ascend-robotascend-robot成员
6月22日 添加了label:ascend-cla/yes
ascend-robot
ascend-robot成员
6月22日 评论:

Thanks for your pull-request.
The full list of commands accepted by me can be found at here
You can get sig-info at here


PR Approval Progress

Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
test renyujin, pengjingyou (2/2) renyujin (1/1)
torch_npu/csrc/distributed renyujin, pengjingyou (2/2) renyujin (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

limuan, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
ascend-robotascend-robot成员
6月22日 添加了label:needs-issue
此处折叠了111条消息 查看更多
pengjingyou成员
6月23日 评论:

/lgtm

likedislike
renyujin成员
6月23日 评论:

/lgtm
/approve

likedislike
ascend-robotascend-robot成员
6月23日 添加了label:approvedlgtm
ascend-robotascend-robot成员
6月23日 合入了pull request
ascend-robot
ascend-robot成员
6月23日 评论:
流水线 pytorch_gitcode_PR_multiVersion#11148 [ commitID:909db93f ] 已完成
likedislike