已关闭
[代码检视] PR#4461 hcomm 合入后遗留检视意见跟踪(2 条检视意见 / 2 MEDIUM) #754
严正行创建于  8月7日关闭于  8月20日
严正行
严正行成员
8月7日 创建

本 Issue 用于跟踪 PR #4461 合入后的遗留检视意见。
PR 已合入(合入时间 2026-08-07T15:13:53+08:00),无法再按行提交 DiffNote,故将检视意见汇总至此 Issue 跟踪修复。

  • 仓库: cann/hcomm
  • PR: #4461
  • 合入时间: 2026-08-07T15:13:53+08:00
  • PR head commit: a2f1318afbd78ff227ba197c59d4c13616074bf6
  • 检视意见总数: 2(2 MEDIUM)

检视时基于上述 head commit 读取代码;若作者已在新 commit 修复,请对照本 Issue 逐条核对。


检视意见清单

1. [MEDIUM] 约束说明缺少"仅支持AIV引擎",与代码实际校验不一致

📍 文件: docs/zh/api_ref/comm_opdev/control_plane_api/basic_resource_mgmt/HcommChannelCreateWithConfig.md:62
🏷️ 维度: 文档与代码一致性
🔗 代码位置: docs/zh/api_ref/comm_opdev/control_plane_api/basic_resource_mgmt/HcommChannelCreateWithConfig.md:62

[MEDIUM] HcommChannelCreateWithConfig.md 约束说明缺少"仅支持AIV引擎"

问题分析:

  • 第62行,IS_SHARED_QUEUE=true 的约束列表中仅提及"仅支持UB网络语义协议",未提及"仅支持AIV引擎"。
  • 代码 src/base_comm/primitives/api_c_adpt/hcomm_channel_c_adpt.cc 第518行明确校验:if (engine != COMM_ENGINE_AIV) { return HCCL_E_NOT_SUPPORT; },注释说明"共享 jetty 仅支持 AIV 引擎"。
  • 同类接口 HcclChannelAcquireWithConfig.md 第55行已正确包含"仅支持AIV引擎"约束,Hcomm 版本遗漏。
  • 本 PR 触碰该文件(去除中英文间空格),但未修复此文档与代码不一致问题。

建议修复:
在第62行"仅支持UB网络语义协议"前增加一条约束:

  • 仅支持AIV引擎。
  • 仅支持UB网络语义协议(COMM_PROTOCOL_UBC_CTP、COMM_PROTOCOL_UBC_TP),不支持UBMem/RoCE/UBOE/UBG。

2. [MEDIUM] IS_SHARED_QUEUE 枚举值描述缺少"AIV引擎",与代码及Hccl版本不一致

📍 文件: docs/zh/api_ref/comm_opdev/datatype_definition/HcommChannelConfigType.md:21
🏷️ 维度: 文档与代码一致性
🔗 代码位置: docs/zh/api_ref/comm_opdev/datatype_definition/HcommChannelConfigType.md:21

[MEDIUM] HcommChannelConfigType.md IS_SHARED_QUEUE 枚举值描述缺少"AIV引擎"

问题分析:

  • 第21行,IS_SHARED_QUEUE 枚举值描述为"仅支持UB网络语义协议(UBC_CTP/UBC_TP)",缺少"AIV引擎"。
  • 代码 src/base_comm/primitives/api_c_adpt/hcomm_channel_c_adpt.cc 第518行明确校验 IS_SHARED_QUEUE 仅支持 AIV 引擎。
  • 对应的 HcclChannelConfigType.md 第22行已正确描述为"仅支持AIV引擎的UB网络语义协议(UBC_CTP/UBC_TP)",Hcomm 版本与 Hccl 版本不一致。
  • 本 PR 触碰该文件(去除中英文间空格),但未修复此文档与代码不一致问题。

建议修复:
将第21行"仅支持UB网络语义协议(UBC_CTP/UBC_TP)"修改为"仅支持AIV引擎的UB网络语义协议(UBC_CTP/UBC_TP)"。


修复跟踪

请逐条核查上述意见:

  • CRITICAL / HIGH 优先处理;
  • 修复后在本 Issue 下评论说明修复情况与对应 commit(或新 PR 链接);
  • 全部修复后可关闭本 Issue。
likedislike
严正行严正行成员
8月8日 添加了label:tech-debt
Leewis成员
8月10日 评论:

感谢对于社区的关注与反馈,当前问题已同步对应PR合入责任人;

likedislike
Leewis成员
8月10日 评论:

/assign @acoolntly

likedislike
CANN-robot
CANN-robot成员
8月10日 评论:

Notice

This issue can not be assigned to acoolntly. Please try to assign to the repository members.

likedislike
CANN-robotCANN-robot成员
8月20日 关闭了 issue
CANN-robotCANN-robot成员
8月20日 添加了label:resolved
LLeewis成员
9月2日 关联了看板:HCCL