已合并
UBC_CTP->UB_CTP #176
chen_xu0513创建于 25 天前
UBC_CTP->UB_CTP #176
已合并
Pull Request已成功合入, 合并人@CANN-robot
（感谢 chen_xu0513 的贡献）25 天前 创建了 pull request，commit c7ccc697
atomgit-bot
25 天前 评论：
25 天前 评论：
变更摘要
本 PR 的核心改动是将通信协议 COMM_PROTOCOL_UBC_CTP 更名为 COMM_PROTOCOL_UB_CTP（华为统一总线 UB_CTP），并在协议枚举中保留 COMM_PROTOCOL_UBC_CTP 作为向后兼容的旧名别名，同时新增 COMM_PROTOCOL_UB_RTP（值 9）及其兼容别名 COMM_PROTOCOL_UBG。改动覆盖协议枚举定义、Hcomm/SIMT Hcomm 模板默认参数、HcommImpl 的 URMA 特化类（AIV/SIMT 平台）、MaskToCommProtocol 默认映射、示例代码与单测，属于全量协议名替换与兼容性保留，不改变既有协议行为。
主要改动
- 协议枚举重命名与扩展（
include/aicore/hcomm/hcomm_common.h）：将枚举值 4 由COMM_PROTOCOL_UBC_CTP更名为COMM_PROTOCOL_UB_CTP，并新增COMM_PROTOCOL_UBC_CTP = COMM_PROTOCOL_UB_CTP兼容旧名别名；另新增枚举值 9COMM_PROTOCOL_UB_RTP及其别名COMM_PROTOCOL_UBG。 - 模板默认参数与实现特化同步更新：
include/aicore/hcomm/hcomm.h与include/aicore/hcomm/hcomm_simt.h中Hcomm类模板及推导指引的默认commProtocol由COMM_PROTOCOL_UBC_CTP改为COMM_PROTOCOL_UB_CTP；hcomm_aiv_urma_def.h、hcomm_simt_urma_def.h中的特化类HcommImpl<COMM_PROTOCOL_UBC_CTP>改为HcommImpl<COMM_PROTOCOL_UB_CTP>，hcomm_aiv_urma.h、hcomm_simt_urma.h中的全部成员函数定义、hcomm_impl.h中的static_assert及hcomm_simt_base.h注释随之一并替换。 - AIN 默认协议映射更新（
src/aicore/ain/impl/ain_impl_def.h）：MaskToCommProtocol在AIN_MASK_DEFAULT分支的返回值由COMM_PROTOCOL_UBC_CTP改为COMM_PROTOCOL_UB_CTP。 - 示例代码协议常量替换：
examples/ain/basic_ring/basic_ring.asc、examples/hcomm_batch_write/hcomm_batch_write.asc与hcomm_batch_write_kernel.cpp、examples/hcomm_write_read_nbi/hcomm_rw_def.h、examples/simt_urma/main.cpp、examples/simt_urma_perftest/main.cpp中的协议常量（如desc.protocol、desc.channelProtocol、HOST_COMM_PROTOCOL、kTargetCommProtocol、AscendC::Hcomm<...>模板实参）及错误日志文案均由UBC_CTP替换为UB_CTP。 - 单元测试同步更新（
tests/ut/aicore/hcomm/test_hcomm_urma_950.cpp）：全部测试用例中的AscendC::Hcomm<AscendC::COMM_PROTOCOL_UBC_CTP>实例化替换为AscendC::Hcomm<AscendC::COMM_PROTOCOL_UB_CTP>，覆盖读写、批量提交/排空、锁操作等 URMA 相关用例。
不准确？
atomgit-bot
25 天前 评论：
25 天前 评论：
25 天前 添加了label：cann-cla/yes
chen_xu0513
25 天前 评论：
25 天前 评论：
compile
25 天前 预合并成功（commit_id: 3700e81fa51802e7761a8ba3e3f434f28742d7d6）
CANN-robot
25 天前 评论：
25 天前 评论：
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 |
|---|---|---|
| examples | ✅ 闫庆尚, YeZZzzz1 (2/2) | ✅ 闫庆尚 (1/1) |
| include | ✅ 李东锋, YeZZzzz1 (2/2) | ✅ 李东锋 (1/1) |
| repo-cann/asc-comm | ✅ 闫庆尚, 李东锋, YeZZzzz1 (3/2) | ✅ 闫庆尚, 李东锋 (2/1) |
| src | ✅ 闫庆尚, YeZZzzz1 (2/2) | ✅ 闫庆尚 (1/1) |
| tests | ✅ 闫庆尚, YeZZzzz1 (2/2) | ✅ 闫庆尚 (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
chen_xu0513, thanks for your pull request. All authors of the commits have signed the CLA. 👍
CANN-robot
25 天前 评论：
25 天前 评论：
| 🚀 CI 流水线已启动 |
|---|
| 📋 执行详情: 点击查看流水线 |
25 天前 添加了label：ci-pipeline-running
此处折叠了6条事件消息 查看更多
25 天前 添加了label：cann-cla/yes
chen_xu0513
25 天前 评论：
25 天前 评论：
compile
25 天前 预合并成功（commit_id: 705bacef18d9249b310c1955a244dd15622f776d）
CANN-robot
25 天前 评论：
25 天前 评论：
| 🚀 CI 流水线已启动 |
|---|
| 📋 执行详情: 点击查看流水线 |
25 天前 删除了label：ci-pipeline-failed
此处折叠了7条事件消息 查看更多
25 天前 删除了label：ci-pipeline-passed
CANN-robot
25 天前 评论：
25 天前 评论：
25 天前 添加了label：cann-cla/yes
examples/ain/basic_ring/README_en.md
已过期
@@ -12,7 +12,7 @@ The Host side allocates memory, initializes the HCCL communicator, creates the H
1212| --- | --- |
1313| Product | `Ascend 950PR / Ascend 950DT` |
1414| NPU architecture | `dav-3510` |
15-| Communication protocol | `COMM_PROTOCOL_UBC_CTP` |
改名字的原因？
25 天前 解决了最后一个问题
chen_xu0513
24 天前 评论：
24 天前 评论：
compile
24 天前 预合并成功（commit_id: 0c133578639db40322af75c13ac3e96da1d7e8da）
CANN-robot
24 天前 评论：
24 天前 评论：
| 🚀 CI 流水线已启动 |
|---|
| 📋 执行详情: 点击查看流水线 |
24 天前 添加了label：ci-pipeline-running
24 天前 删除了label：ci-pipeline-running
24 天前 添加了label：ci-pipeline-passed
yanqingshang
24 天前 评论：
24 天前 评论：
/approve
mlidongfeng
24 天前 评论：
24 天前 评论：
/approve
24 天前 添加了label：lgtmapproved
24 天前 合入了pull request
当前PR是否有AI参与:
[x] 否
[ ] 是
__1. AI Agent 平台:
__2. AI 模型:
__3. Prompt上下文 :
PR功能描述 / 为什么需要这个合入**:
该PR关联的issue
(格式为fixes #<issue号>, 或者resolves #<issue号>): fixes #
希望检视人员了解:
改动类型 / Change Type
测试信息 / Testing
检查清单 / Checklist