已合并
fix: 增大 STEP_SYNC_TIMEOUT 至 180000 #2768
cann_wangyq创建于 23 天前
fix: 增大 STEP_SYNC_TIMEOUT 至 180000 #2768
已合并
cann_wangyq创建于 23 天前
cann_wangyq成员
23 天前

描述

原代码中STEP_SYNC_TIMEOUT=18000,实际是bug,要改为180000ms

关联的Issue

测试

文档更新

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 cann_wangyq 的贡献)
Ccann_wangyq成员
23 天前 创建了 pull request,commit 1f22e0fd
atomgit-bot
atomgit-bot
23 天前 评论:

变更摘要

本 PR 针对 NHR 优化的 dpu_alg_nhr_opt_wrapper.h 头文件,将 step 同步的超时时间常量 STEP_SYNC_TIMEOUT18000 增大至 180000(10 倍),以放宽 step 同步等待的时间窗口,降低同步超时触发的概率。本次改动仅涉及常量取值的调整,不影响接口签名、数据结构或函数逻辑。

主要改动

  • 调整 STEP_SYNC_TIMEOUT 取值: 将 src/ops/op_common/template/wrapper/dpu_alg_nhr_opt_wrapper.h 中 step 同步专用 notify(NOTIFY_IDX_STEP_SYNC)的超时时间常量由 18000 增大为 180000,使同步等待更宽松。
  • 同步注释保持对应: 该常量所在处用于 step 同步的注释(不与 ACK(0)/DATA(1)/FIN_ACK(2) 冲突的 notify 索引说明)保持不变,仅常量值变更,无接口或行为逻辑层面的其他改动。
likedislike
不准确?
atomgit-bot
atomgit-bot
23 天前 评论:

代码审查

✅ 未发现问题

likedislike
不准确?
CANN-robotCANN-robot成员
23 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
23 天前 评论:

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.
You can self-configure the PR merge rules for this repository. For more details, please refer to Here.
For more, you also can visit HICANN.


PR Approval Progress

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

Module Approval Details

module lgtm status approve status
9.1.0 Andy-lb, 李标智 (2/2) Andy-lb (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
CANN-robot
CANN-robot成员
23 天前 评论:

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
23 天前 将leabclove,yanglianxiang2,linyf950,yanzhenghang,chenliang267,TianXL,yanyefeng,jiangtao_rts,chenhao_,hanxiaolong,dingweiqin_57,songmingyang,wenxuemin,gcw_kUomxQ2l,yangxiaoyuan,Andy-lb,laodazhao1,yangshaohua,chenke2026,zhongqin2020,bz_lee,Innoecho,chenyia,temper7620,zhangxp1030,lilin_137,hulk1997,ouyangxizi,one_random设为评审人
CANN-robotCANN-robot成员
23 天前 将TianXL,jiangtao_rts,hanxiaolong,gcw_kUomxQ2l,Andy-lb设为审查人
57_Viking
57_Viking成员22 天前进行代码检视1
src/ops/op_common/template/wrapper/dpu_alg_nhr_opt_wrapper.h
@@ -19,3 +19,3 @@
1919// 用于 step 同步的专用 notify 索引(不与 ACK(0)/DATA(1)/FIN_ACK(2) 冲突)
2020constexpr u32 NOTIFY_IDX_STEP_SYNC = 0;
21-constexpr u32 STEP_SYNC_TIMEOUT = 18000;
21+constexpr u32 STEP_SYNC_TIMEOUT = 180000;
57_Viking
57_Viking22 天前评论:

[检视] 🟡 Medium — 资源管理

STEP_SYNC_TIMEOUT 从 18000 增大到 180000(10 倍)。

超时时间增大 10 倍意味着同步失败时等待时间从 18 秒延长到 180 秒(3 分钟)。这会显著影响故障检测的及时性——如果某个 rank 挂死,其他 rank 需要等 3 分钟才能超时返回。

反证:如果 DPU 侧 step 同步确实需要更长时间(如大规模集群或高延迟网络),增大超时是合理的。

建议:确认增大的原因。建议通过环境变量(如 HCCL_STEP_SYNC_TIMEOUT)配置,而非硬编码,让用户根据集群规模调整。

likedislike
Ccann_wangyq成员
15 天前 关闭了 pull request
Ccann_wangyq成员
8 天前 重新打开了 pull request
Ccann_wangyq成员
8 天前 update merge request[project id:8277906, iid:2768, commit_id:f19066b21ea3d7e4b011e5a9fc527f7b2dc45b3a], virtual merging success
CANN-robotCANN-robot成员
8 天前 删除了label:cann-cla/yes
CANN-robotCANN-robot成员
8 天前 添加了label:cann-cla/yes
cann_wangyq成员
8 天前 评论:

/compile

likedislike
Ccann_wangyq成员
8 天前 预合并成功(commit_id: 44c50637d35dc63e400e3ff6e8e8e3e88bc33e82)
CANN-robot
CANN-robot成员
8 天前 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
8 天前 添加了label:ci-pipeline-running
此处折叠了13条事件消息 查看更多
CANN-robotCANN-robot成员
8 天前 删除了label:stat/needs-squash
CANN-robot
CANN-robot成员
8 天前 评论:

Notification

This pull request has been changed(code update) or closed, so removes the following label(s): stat/needs-squash.

likedislike
CANN-robotCANN-robot成员
8 天前 添加了label:cann-cla/yes
cann_wangyq成员
8 天前 评论:

/compile

likedislike
Ccann_wangyq成员
8 天前 预合并成功(commit_id: 0250f2700b11aaa0aabb7869e410a65f9146fe19)
CANN-robot
CANN-robot成员
8 天前 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
8 天前 删除了label:ci-pipeline-failed
此处折叠了5条事件消息 查看更多
Ccann_wangyq成员
8 天前 解决了最后一个问题
Andy-lb成员
8 天前 评论:

/approve

likedislike
CANN-robotCANN-robot成员
8 天前 添加了label:approved
bz_lee成员
8 天前 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
8 天前 添加了label:lgtm
CANN-robotCANN-robot成员
8 天前 合入了pull request