合并受阻
变更摘要
本 PR 针对 NHR 优化的 dpu_alg_nhr_opt_wrapper.h 头文件，将 step 同步的超时时间常量 STEP_SYNC_TIMEOUT 由 18000 增大至 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 索引说明）保持不变，仅常量值变更，无接口或行为逻辑层面的其他改动。
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
⚠️ This PR does not yet meet the following requirements:lgtm (requires ≥ 2 person(s) per module)、approve (requires ≥ 1 person(s) per module)
Module Approval Details
| module | lgtm status | approve status |
|---|---|---|
| 9.1.0 | ❌ (0/2)(You can also ask: 陈轲, 严正行, 李标智, 杨邵华, 钟琴) | ❌ (0/1)(You can also ask: gcw_kUomxQ2l, Andy-lb, hanxiaolong, jiangtao_rts, 陈娇) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CANN-robot
[检视] 🟡 Medium — 资源管理
STEP_SYNC_TIMEOUT 从 18000 增大到 180000（10 倍）。
超时时间增大 10 倍意味着同步失败时等待时间从 18 秒延长到 180 秒（3 分钟）。这会显著影响故障检测的及时性——如果某个 rank 挂死，其他 rank 需要等 3 分钟才能超时返回。
反证：如果 DPU 侧 step 同步确实需要更长时间（如大规模集群或高延迟网络），增大超时是合理的。
建议：确认增大的原因。建议通过环境变量（如 HCCL_STEP_SYNC_TIMEOUT）配置，而非硬编码，让用户根据集群规模调整。
描述
关联的Issue
测试
文档更新
类型标签