已合并
[fix]确定性计算strict模式放行910_93 #4485
weixin_47168537创建于 8月7日
[fix]确定性计算strict模式放行910_93 #4485
已合并
weixin_47168537创建于 8月7日
weixin_47168537
8月7日

[fix]确定性计算strict模式放行910_93,在 910_93上放行,并同步更新文档与 UT

变更点:

  • op_base.cc — HcclSetConfig:strict 校验由"仅 910B 放行"扩为"910B/910_93 放行",其余芯片仍返回 HCCL_E_NOT_SUPPORT;
  • comm_config.cc — CommConfig::SetConfigDeterministic(HcclCommInitRootInfoConfig 路径):同步放行 910_93;
  • HcclCommConfig.md — A3 与 A2 的 hcclDeterministic 说明合并为 "Atlas A2/A3";删除 A3"静态图模式暂不支持 2"的旧描述;
  • UT 配套:ut_HcclGetConfig_API_test.cc 失败路径改用 910 设备;ut_HcclSetConfig_API_test.cc 拆分为"非放行设备→NOT_SUPPORT"与"910_93→SUCCESS";ut_comm_config.cc 启用原 #if 0 用例并改验证 910_93 成功路径(GetConfigDeterministic()==2)。

自验证场景:

  • pre-commit:OAT 合规通过;clang-format 通过。
  • UT 编译通过;用例覆盖:910_93+strict→HCCL_SUCCESS、910+strict→HCCL_E_NOT_SUPPORT、set 失败后取值 DETERMINISTIC_DISABLE、CommConfig::SetConfigByVersion 910_93 成功且 GetConfigDeterministic()==2
likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 weixin_47168537 的贡献)
weixin_47168537weixin_47168537
8月7日 创建了 pull request,commit dfac51e6
atomgit-bot
atomgit-bot
8月7日 评论:

变更摘要

此 PR 放宽了确定性计算 strict 模式(DETERMINISTIC_STRICT)的设备类型限制:原先仅允许 DEV_TYPE_910B(A2)设备使用 strict 模式,现在同时放行 DEV_TYPE_910_93(A3)设备。修改涉及 CommConfig::SetConfigDeterministicHcclSetConfig 两处设备类型校验逻辑,并同步更新了对应的错误提示信息与文档。

主要改动

  • CommConfig::SetConfigDeterministic 放宽设备校验:在 src/legacy/ascend910/framework/communicator/comm_config.cc 中,将 strict 模式的设备类型判定从仅排除非 DEV_TYPE_910B,改为同时排除非 DEV_TYPE_910B 且非 DEV_TYPE_910_93,使 A3 设备也可使用 strict 确定性计算。
  • HcclSetConfig 放宽设备校验:在 src/legacy/ascend910/framework/op_base/src/op_base.cc 中,对 DETERMINISTIC_STRICT 配置值的设备类型检查做了相同逻辑修改,新增对 DEV_TYPE_910_93 的放行。
  • 错误提示信息更新:两处校验失败时的错误日志和报错信息均由 "only support A2" 更新为 "only support A2/A3",与新的设备支持范围保持一致。
  • 文档同步修改:根据 PR 描述,对应文档也已更新以反映 strict 模式新增对 A3 设备的支持。
likedislike
不准确?
atomgit-bot
atomgit-bot
8月7日 评论:

代码审查

✅ 未发现问题

likedislike
不准确?
CANN-robotCANN-robot成员
8月7日 添加了label:cann-cla/no
CANN-robotCANN-robot成员
8月7日 添加了label:stat/needs-squash
此处折叠了121条消息 查看更多
CANN-robotCANN-robot成员
13 天前 添加了label:ci-pipeline-passed
CANN-robot
CANN-robot成员
13 天前 评论:

The MR can not be merged, because of CodeReview discussion not resolved

If you want to solve this problem, you can click here to do it in the FAQs.

likedislike
CANN-robot
CANN-robot成员
13 天前 评论:
流水线 PR-pipeline_cann_hcomm#19320 [ commitID:e498f8ae ] 已完成
likedislike
claire8090claire8090成员
13 天前 解决了最后一个问题
CANN-robotCANN-robot成员
13 天前 合入了pull request