| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
[build] replace public options with intf_pub of cann/cmake repo Co-authored-by: ouyangcaike<ouyangcaike@h-partners.com> # message auto-generated for no-merge-commit merge: !2529 merge fix-10 into master [build] replace public options with intf_pub of cann/cmake repo Created-by: ouyangcaike Commit-by: ouyangcaike Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动,包括改动的原因和所采取的方法。--> 1. 启用 -Wextra 并修复了大部分编译警告,启用 -Werror 将编译警告转为编译错误,提升代码质量。 2. 构建系统变更通过add_cann_target_options集中管理编译选项,添加intf_pub链接,并移除冗余的针对每个目标的标志。 3. 警告修复解决了 -Wcast-function-type、-Wunused-function、-Wunused-variable、-Woverloaded-virtual、-Wimplicit-fallthrough、-Wtype-limit、-Wunused-command-line-argument、-Wsign-compare、-Wextra 和 -Wignored-qualifiers 等警告,覆盖80个文件。 还剩:-Wno-unused-parameter、-Wno-missing-field-initializers ## 变更类型 请选择本次引入的变更类型: <!-- [x] 表示选中 --> - [ ] 🐛 Bug修复 - [ ] ✨ 新特性 - [ ] 🚀 性能优化 - [ ] 📝 文档更新 - [x] 📋 其他,请描述:工程构建 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。--> <!-- 如果这个PR不涉及Issue,可填写"NA"。--> NA ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于构造对应xx测试用例、二级冒烟、算子泛化等。--> 已完成的测试用例和场景: - 编译成功 - 全量拦截任务通过 - 前冒烟测试通过 补充的UT用例: - NA ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> NA ## 合入检查 <!-- 在正式合入前,请做好必要的代码测试,用例补充,软件代码风格检查等。提高合入效率。--> <!-- [x] 表示选中 --> - [x] 🧐 已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 - [x] 🔍 邀请 committer评论 /lgtm前的必要检查 - [x] 🏷️ 标题中使用了合适的类型标签(如:[feat], [fix]) - [x] 📄 代码修改内容已简要描述,相关文档已更新 - [x] 📝 代码注释已更新,代码遵循项目整体代码风格 - [x] 🧪 代码UT测试已更新,覆盖率已达标 - [x] 🔬 验证方法已更新到"测试"部分 - [x] 🛠️ 代码已通过静态分析工具检查,无错误 - [x] 👥 代码检视/code review/同行评议和必要的代码串讲,确保代码质量 - [x] ✅ 代码检视意见已处理或答复,无未处理的检视意见 - [x] 🚀 预约 前冒烟 用例前的必要检查 - [x] ✔️ 代码已有committer的/lgtm 和 模块committer的/lgtm评论 - [x] 🔧 代码已通过compile,编译无错误,无告警 - [x] 🖥️ 代码已通过基本功能本地测试或者在线测试,确保基本功能正常 - [x] 🎯 预约 approver评论/approve,正式合入前的必要检查 - [x] 📊 前冒烟 用例已全量通过 - [x] 📦 新增功能已同步补充基本功能测试用例到前冒烟里 See merge request: cann/hcomm!2529 | 2 个月前 | |
error msg Co-authored-by: liuwanke152<liuwanke1@hisilicon.com> # message auto-generated for no-merge-commit merge: !3401 merge master into master error msg Created-by: liuwanke152 Commit-by: liuwanke152 Merged-by: cann-robot Description: ## 描述 ERROR MSG打印整改: 1、修改EI0002和EI0006网页链接,改为非绝对路径: 2、统一内存单位形式:Byte -> bytes 3、规范EI0006打印格式,对端过多时防止刷屏 ## 变更类型 请选择本次引入的变更类型: <!-- [x] 表示选中 --> - [ ] 🐛 Bug修复 - [ ] ✨ 新特性 - [ ] 🚀 性能优化 - [ ] 📝 文档更新 - [ ] 📋 其他,请描述: ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。--> <!-- 如果这个PR不涉及Issue,可填写"NA"。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于构造对应xx测试用例、二级冒烟、算子泛化等。--> 已完成的测试用例和场景: 1. 2. 补充的UT用例: ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> ## 合入检查 <!-- 在正式合入前,请做好必要的代码测试,用例补充,软件代码风格检查等。提高合入效率。--> <!-- [x] 表示选中 --> - [ ] 🧐 已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 - [ ] 🔍 邀请 committer评论 /lgtm前的必要检查 - [ ] 🏷️ 标题中使用了合适的类型标签(如:[feat], [fix]) - [ ] 📄 代码修改内容已简要描述,相关文档已更新 - [ ] 📝 代码注释已更新,代码遵循项目整体代码风格 - [ ] 🧪 代码UT测试已更新,覆盖率已达标 - [ ] 🔬 验证方法已更新到"测试"部分 - [ ] 🛠️ 代码已通过静态分析工具检查,无错误 - [ ] 👥 代码检视/code review/同行评议和必要的代码串讲,确保代码质量 - [ ] ✅ 代码检视意见已处理或答复,无未处理的检视意见 - [ ] 🚀 预约 前冒烟 用例前的必要检查 - [ ] ✔️ 代码已有committer的/lgtm 和 模块committer的/lgtm评论 - [ ] 🔧 代码已通过compile,编译无错误,无告警 - [ ] 🖥️ 代码已通过基本功能本地测试或者在线测试,确保基本功能正常 - [ ] 🎯 预约 approver评论/approve,正式合入前的必要检查 - [ ] 📊 前冒烟 用例已全量通过 - [ ] 📦 新增功能已同步补充基本功能测试用例到前冒烟里 See merge request: cann/hcomm!3401 | 2 个月前 | |
[fix]将ReadWriteLock修改成c++17标准库提供的读写锁 Co-authored-by: dingweiqin_57<D_Viking@163.com> # message auto-generated for no-merge-commit merge: !3695 merge read_write_lock_switch into master [fix]将ReadWriteLock修改成c++17标准库提供的读写锁 Created-by: dingweiqin_57 Commit-by: dingweiqin_57 Merged-by: cann-robot Description: ## 描述 [fix]hcomm仓原先基于C++11进行编码,由于没有标准库提供的读写锁,代码仓内自己实现了ReadWriteLockBase,ReadWriteLock等类和相关成员函数来实现读写锁功能。 现在hcomm仓可以基于c++17进行编码,已经提供了标准读写锁功能。把代码仓内自己实现的读写锁相关代码,改成标准读写锁,并删除ReadWriteLockBase和ReadWriteLock的相关头文件和函数。静态检查和编译验证修改的正确性。 ## 变更类型 请选择本次引入的变更类型: <!-- [x] 表示选中 --> - [ ] 🐛 Bug修复 - [ ] ✨ 新特性 - [ ] 🚀 性能优化 - [ ] 📝 文档更新 - [x] 📋 其他,请描述: ## 关联的Issue NA ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于构造对应xx测试用例、二级冒烟、算子泛化等。--> 已完成的测试用例和场景: 1. 已完成静态扫描 2. rg 全局搜索:ReadWriteLockBase/ReadWriteLock/read_write_lock.h 引用 0残留,.readLock()/.readUnlock()/.writeLock()/.writeUnlock() 方法调用 0残留 语法检查:对 hcclCommDfx.cc 等文件做 g++ -fsyntax-only,0个锁相关编译错误(31个错误全部为CANN框架类型缺失,与本次变更无关) 3. 行为保持: 所有手动 unlock 调用已删除,由 RAII 析构自动释放 提前返回路径(CHK_RET/CHK_PTR_NULL)不再需要手动 unlock 2处"同一对象先读后写"场景拆为独立作用域的 shared_lock + unique_lock 轮询函数(AicpuGetCommbyGroup)保持 per-iteration 锁释放/重获语义 补充的UT用例: 新增了 15个测试用例,涉及 4个文件(3个修改 + 1个新建),共 +220行 | 文件 | 新增用例数 | 覆盖的目标行 | 覆盖率提升 | | --- | --- | --- | --- | | ut_hcclCommDfx.cc | 5个 | L91(shared_lock读路径·AddTaskInfoCallback CCU)、L171(shared_lock读路径·GetChannelRemoteRankId) | 33%→~100% | | ut_ns_recovery_func_lite.cc | 2个 | L25(shared_lock·Call) | 0%→~100% | | ut_aicpu_one_side_service.cc | 4个 | L54(unique_lock·Process finalize)、L69(shared_lock·GetService lookup)、L81(unique_lock·GetService insert) | 21%→~90% | | ut_coll_comm_aicpu_destroy_func.cc (新建) | 4个 | L39(shared_lock·Process) | 0%→~80% | ## 文档更新 无需文档更新 ## 合入检查 <!-- 在正式合入前,请做好必要的代码测试,用例补充,软件代码风格检查等。提高合入效率。--> <!-- [x] 表示选中 --> - [x] 🧐 已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 - [x] 🔍 邀请 committer评论 /lgtm前的必要检查 - [x] 🏷️ 标题中使用了合适的类型标签(如:[feat], [fix]) - [x] 📄 代码修改内容已简要描述,相关文档已更新 - [x] 📝 代码注释已更新,代码遵循项目整体代码风格 - [x] 🧪 代码UT测试已更新,覆盖率已达标 - [x] 🔬 验证方法已更新到"测试"部分 - [x] 🛠️ 代码已通过静态分析工具检查,无错误 - [x] 👥 代码检视/code review/同行评议和必要的代码串讲,确保代码质量 - [x] ✅ 代码检视意见已处理或答复,无未处理的检视意见 - [x] 🚀 预约 前冒烟 用例前的必要检查 - [x] ✔️ 代码已有committer的/lgtm 和 模块committer的/lgtm评论 - [x] 🔧 代码已通过compile,编译无错误,无告警 - [x] 🖥️ 代码已通过基本功能本地测试或者在线测试,确保基本功能正常 - [x] 🎯 预约 approver评论/approve,正式合入前的必要检查 - [x] 📊 前冒烟 用例已全量通过 - [x] 📦 新增功能已同步补充基本功能测试用例到前冒烟里 See merge request: cann/hcomm!3695 | 2 个月前 | |
[build] replace public options with intf_pub of cann/cmake repo Co-authored-by: ouyangcaike<ouyangcaike@h-partners.com> # message auto-generated for no-merge-commit merge: !2529 merge fix-10 into master [build] replace public options with intf_pub of cann/cmake repo Created-by: ouyangcaike Commit-by: ouyangcaike Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动,包括改动的原因和所采取的方法。--> 1. 启用 -Wextra 并修复了大部分编译警告,启用 -Werror 将编译警告转为编译错误,提升代码质量。 2. 构建系统变更通过add_cann_target_options集中管理编译选项,添加intf_pub链接,并移除冗余的针对每个目标的标志。 3. 警告修复解决了 -Wcast-function-type、-Wunused-function、-Wunused-variable、-Woverloaded-virtual、-Wimplicit-fallthrough、-Wtype-limit、-Wunused-command-line-argument、-Wsign-compare、-Wextra 和 -Wignored-qualifiers 等警告,覆盖80个文件。 还剩:-Wno-unused-parameter、-Wno-missing-field-initializers ## 变更类型 请选择本次引入的变更类型: <!-- [x] 表示选中 --> - [ ] 🐛 Bug修复 - [ ] ✨ 新特性 - [ ] 🚀 性能优化 - [ ] 📝 文档更新 - [x] 📋 其他,请描述:工程构建 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。--> <!-- 如果这个PR不涉及Issue,可填写"NA"。--> NA ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于构造对应xx测试用例、二级冒烟、算子泛化等。--> 已完成的测试用例和场景: - 编译成功 - 全量拦截任务通过 - 前冒烟测试通过 补充的UT用例: - NA ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> NA ## 合入检查 <!-- 在正式合入前,请做好必要的代码测试,用例补充,软件代码风格检查等。提高合入效率。--> <!-- [x] 表示选中 --> - [x] 🧐 已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 - [x] 🔍 邀请 committer评论 /lgtm前的必要检查 - [x] 🏷️ 标题中使用了合适的类型标签(如:[feat], [fix]) - [x] 📄 代码修改内容已简要描述,相关文档已更新 - [x] 📝 代码注释已更新,代码遵循项目整体代码风格 - [x] 🧪 代码UT测试已更新,覆盖率已达标 - [x] 🔬 验证方法已更新到"测试"部分 - [x] 🛠️ 代码已通过静态分析工具检查,无错误 - [x] 👥 代码检视/code review/同行评议和必要的代码串讲,确保代码质量 - [x] ✅ 代码检视意见已处理或答复,无未处理的检视意见 - [x] 🚀 预约 前冒烟 用例前的必要检查 - [x] ✔️ 代码已有committer的/lgtm 和 模块committer的/lgtm评论 - [x] 🔧 代码已通过compile,编译无错误,无告警 - [x] 🖥️ 代码已通过基本功能本地测试或者在线测试,确保基本功能正常 - [x] 🎯 预约 approver评论/approve,正式合入前的必要检查 - [x] 📊 前冒烟 用例已全量通过 - [x] 📦 新增功能已同步补充基本功能测试用例到前冒烟里 See merge request: cann/hcomm!2529 | 2 个月前 | |
fix: resolve channelId via HcommChannelGet in GenErrorInfo functions Co-authored-by: hulk1997<hulinkang1@huawei.com> Co-authored-by: qq_46994783<sunenguang1@h-partners.com> # message auto-generated for no-merge-commit merge: !4086 merge master into master fix: resolve channelId via HcommChannelGet in GenErrorInfo functions Created-by: hulk1997 Commit-by: hulk1997;qq_46994783 Merged-by: cann-robot Description: ## 描述 ### 简要描述 GenErrorInfo 系列函数改为通过 HcommChannelGet 动态解析 channelId/signalId/varId,替代直接从 rep 对象读取,提升错误信息准确性。 ### 问题描述 CcuTaskException 的 GenErrorInfoRemPostSem、GenErrorInfoRemWaitSem、GenErrorInfoRemPostVar、GenErrorInfoRead、GenErrorInfoWrite、GenErrorInfoBufRead、GenErrorInfoBufWrite 等函数生成错误信息时,channelId、signalId、paramId 等字段直接从 rep 对象的 GetChannelId()、GetId()、GetRmtCkeId()、GetRmtXnId() 等方法读取。这些 ID需要在翻译时解析得到,但翻译过程已迁移到InsGenerator,导致 DFX 错误信息中的 channel/signal/variable 标识不准确,影响任务异常定位。 ### 解决方案 1. 在 CcuTaskException 中新增三个静态私有方法作为统一的信息获取接口: - GetChannelIdByHandle:调用 HcommChannelGet 解析 channelHandle 为 channel 指针,dynamic_cast 为 CcuUrmaChannel 后调用 GetChannelId() 获取真实 channelId。 - GetSignalIdByHandle:同样解析 channel 指针,根据 isRmtSig 标志分别调用 GetRmtCkeByIndex(远端信号)或 GetLocCkeByIndex(本地信号)获取 signalId。 - GetVariableIdByHandle:解析 channel 指针后调用 GetRmtXnByIndex 获取 varId。 2. 上述方法均包含 HcommChannelGet 失败、channel 指针为空、dynamic_cast 失败等异常分支的错误处理与日志输出。 3. 重构 GenErrorInfoRemPostSem/RemWaitSem/RemPostVar/Read/Write/BufRead/BufWrite 七个函数,将原来直接从 rep 读取 ID 的逻辑替换为调用上述新方法,通过 channelHandle 实时解析得到 channelId、signalId、paramId,确保错误信息中的标识与 channel 实例状态一致。 ### 影响范围 - 新增:CcuTaskException 新增 GetChannelIdByHandle/GetSignalIdByHandle/GetVariableIdByHandle 三个静态私有方法及对应头文件声明 - 修改:GenErrorInfoRemPostSem、GenErrorInfoRemWaitSem、GenErrorInfoRemPostVar、GenErrorInfoRead、GenErrorInfoWrite、GenErrorInfoBufRead、GenErrorInfoBufWrite 七个函数的 channelId/signalId/paramId 获取逻辑 - 功能:DFX 错误信息中的 channel/signal/variable 标识改为实时解析,提升异常定位准确性 - 兼容性:对外接口无变化,仅影响内部错误信息生成逻辑 - 错误处理:新增 HcommChannelGet 失败、空指针等异常分支的处理与日志 ### 新增用例 - 单元测试 ut_ccuTaskException_GetChannelByHandle_test.cc,共 12 个用例: - GetChannelIdByHandle:正常场景返回成功且 channelId 正确;HcommChannelGet 失败返回错误;channel 指针为空返回指针错误 - GetSignalIdByHandle:远端信号正常场景;本地信号正常场景;GetRmtCkeByIndex 失败返回 UNAVAIL;GetLocCkeByIndex 失败返回 UNAVAIL;channel 指针为空返回指针错误 - GetVariableIdByHandle:正常场景返回成功且 varId 正确;GetRmtXnByIndex 失败返回 UNAVAIL;channel 指针为空返回指针错误 - GenErrorInfoRead:正常场景验证 channelId 来自 handle 解析 - GenErrorInfoWrite:正常场景验证 channelId 来自 handle 解析 ## 变更类型 请选择本次引入的变更类型: <!-- [x] 表示选中 --> - [ ] 🐛 Bug修复 - [ ] ✨ 新特性 - [ ] 🚀 性能优化 - [ ] 📝 文档更新 - [ ] 📋 其他,请描述: ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。--> <!-- 如果这个PR不涉及Issue,可填写"NA"。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于构造对应xx测试用例、二级冒烟、算子泛化等。--> 已完成的测试用例和场景: 1. 2. 补充的UT用例: ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> ## 合入检查 <!-- 在正式合入前,请做好必要的代码测试,用例补充,软件代码风格检查等。提高合入效率。--> <!-- [x] 表示选中 --> - [ ] 🧐 已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 - [ ] 🔍 邀请 committer评论/lgtm前的必要检查 - [ ] 🏷️ 标题中使用了合适的类型标签(如:[feat], [fix]) - [ ] 📄 代码修改内容已简要描述,相关文档已更新 - [ ] 📝 代码注释已更新,代码遵循项目整体代码风格 - [ ] 🧪 代码UT测试已更新,覆盖率已达标 - [ ] 🔬 验证方法已更新到"测试"部分 - [ ] 🛠️ 代码已通过静态分析工具检查,无错误 - [ ] 👥 代码检视/code review/同行评议和必要的代码串讲,确保代码质量 - [ ] ✅ 代码检视意见已处理或答复,无未处理的检视意见 - [ ] 🚀 预约 前冒烟 用例前的必要检查 - [ ] ✔️ 代码已有committer的/lgtm 和 模块committer的/lgtm评论 - [ ] 🔧 代码已通过compile,编译无错误,无告警 - [ ] 🖥️ 代码已通过基本功能本地测试或者在线测试,确保基本功能正常 - [ ] 🎯 预约 approver评论/approve,正式合入前的必要检查 - [ ] 📊 前冒烟 用例已全量通过 - [ ] 📦 新增功能已同步补充基本功能测试用例到前冒烟里 See merge request: cann/hcomm!4086 | 1 个月前 | |
group sendrecv with no sync Co-authored-by: chengjing43<chengjing43@huawei.com> # message auto-generated for no-merge-commit merge: !3495 merge test-no-sync-2 into master group sendrecv with no sync Created-by: chengjing43 Commit-by: chengjing43 Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动,包括改动的原因和所采取的方法。--> group sendrecv优化,在保证不卡死的情况下进行性能优化 1. hccl_group.cc中 恢复走GroupSendRecv 2. 去除了原先的本地拷贝和跨卡拷贝之间的同步操作,这是卡死的原因 3. 为保证机内/超节点内的性能,采取pingpong操作:send stream在等待对端Read时,进行下一片数据的local copy;待对端Read完,可直接进行下一片数据的Read,不用等待本端的local copy ## 变更类型 请选择本次引入的变更类型: <!-- [x] 表示选中 --> - [x] 🐛 Bug修复 - [ ] ✨ 新特性 - [x] 🚀 性能优化 - [ ] 📝 文档更新 - [ ] 📋 其他,请描述: ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。--> <!-- 如果这个PR不涉及Issue,可填写"NA"。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于构造对应xx测试用例、二级冒烟、算子泛化等。--> 已完成的测试用例和场景: 1. 2. 补充的UT用例: ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> ## 合入检查 <!-- 在正式合入前,请做好必要的代码测试,用例补充,软件代码风格检查等。提高合入效率。--> <!-- [x] 表示选中 --> - [ ] 🧐 已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 - [ ] 🔍 邀请 committer评论 /lgtm前的必要检查 - [ ] 🏷️ 标题中使用了合适的类型标签(如:[feat], [fix]) - [ ] 📄 代码修改内容已简要描述,相关文档已更新 - [ ] 📝 代码注释已更新,代码遵循项目整体代码风格 - [ ] 🧪 代码UT测试已更新,覆盖率已达标 - [ ] 🔬 验证方法已更新到"测试"部分 - [ ] 🛠️ 代码已通过静态分析工具检查,无错误 - [ ] 👥 代码检视/code review/同行评议和必要的代码串讲,确保代码质量 - [ ] ✅ 代码检视意见已处理或答复,无未处理的检视意见 - [ ] 🚀 预约 前冒烟 用例前的必要检查 - [ ] ✔️ 代码已有committer的/lgtm 和 模块committer的/lgtm评论 - [ ] 🔧 代码已通过compile,编译无错误,无告警 - [ ] 🖥️ 代码已通过基本功能本地测试或者在线测试,确保基本功能正常 - [ ] 🎯 预约 approver评论/approve,正式合入前的必要检查 - [ ] 📊 前冒烟 用例已全量通过 - [ ] 📦 新增功能已同步补充基本功能测试用例到前冒烟里 See merge request: cann/hcomm!3495 | 2 个月前 | |
【2026 HCCL通信库创新大赛-长三角赛区】【4100】修复 profiling 日志中 Unexpected 的拼写错误 Co-authored-by: Xie21<2578926836@qq.com> # message auto-generated for no-merge-commit merge: !3547 merge fix-Unexpected-spelling into master 【2026 HCCL通信库创新大赛-长三角赛区】【4100】修复 profiling 日志中 Unexpected 的拼写错误 Created-by: Xie21 Commit-by: Xie21 Merged-by: cann-robot Description: ## 描述 将日志中的: Unexcepeted 修改为: Unexpected ## 变更类型 请选择本次引入的变更类型: <!-- [x] 表示选中 --> [x ] 🐛 Bug修复 - [ ] ✨ 新特性 - [ ] 🚀 性能优化 - [ ] 📝 文档更新 - [ ] 📋 其他,请描述: ## 关联的Issue 关联#328 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于构造对应xx测试用例、二级冒烟、算子泛化等。--> 已完成的测试用例和场景: 1. 2. 补充的UT用例: ## 文档更新 hcomm/src/legacy/ascend910/common/debug/profiling/command_handle.cc ## 合入检查 <!-- 在正式合入前,请做好必要的代码测试,用例补充,软件代码风格检查等。提高合入效率。--> <!-- [x] 表示选中 --> - [ ] 🧐 已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 - [ ] 🔍 邀请 committer评论 /lgtm前的必要检查 - [ ] 🏷️ 标题中使用了合适的类型标签(如:[feat], [fix]) - [ ] 📄 代码修改内容已简要描述,相关文档已更新 - [ ] 📝 代码注释已更新,代码遵循项目整体代码风格 - [ ] 🧪 代码UT测试已更新,覆盖率已达标 - [ ] 🔬 验证方法已更新到"测试"部分 - [ ] 🛠️ 代码已通过静态分析工具检查,无错误 - [ ] 👥 代码检视/code review/同行评议和必要的代码串讲,确保代码质量 - [ ] ✅ 代码检视意见已处理或答复,无未处理的检视意见 - [ ] 🚀 预约 前冒烟 用例前的必要检查 - [ ] ✔️ 代码已有committer的/lgtm 和 模块committer的/lgtm评论 - [ ] 🔧 代码已通过compile,编译无错误,无告警 - [ ] 🖥️ 代码已通过基本功能本地测试或者在线测试,确保基本功能正常 - [ ] 🎯 预约 approver评论/approve,正式合入前的必要检查 - [ ] 📊 前冒烟 用例已全量通过 - [ ] 📦 新增功能已同步补充基本功能测试用例到前冒烟里 See merge request: cann/hcomm!3547 | 2 个月前 | |
校验toolkit(cann-hcomm)和ops(cann-hccl)版本不一致时,准确指出是哪个包版本不一致 Co-authored-by: sakura_pc<yaopengcheng3@huawei.com> # message auto-generated for no-merge-commit merge: !3971 merge ypc_versioncheck into master 校验toolkit(cann-hcomm)和ops(cann-hccl)版本不一致时,准确指出是哪个包版本不一致 Created-by: sakura_pc Commit-by: sakura_pc Merged-by: cann-robot Description: ## 描述 问题: 当本端和远端toolkit(cann-hcomm)和ops(cann-hccl)版本不一致时,会报错打印对应的版本号,但是打印出来的版本号是hcomm和hccl的版本号拼接起来的,开发者不知道是哪个包版本不一致。 修改方案: 将版本号拆分,准确打印出是toolkit(cann-hcomm)还是ops(cann-hccl)版本号不一致。 ## 变更类型 请选择本次引入的变更类型: <!-- [x] 表示选中 --> - [x] 🐛 Bug修复 - [ ] ✨ 新特性 - [ ] 🚀 性能优化 - [ ] 📝 文档更新 - [ ] 📋 其他,请描述: ## 关联的Issue NA ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于构造对应xx测试用例、二级冒烟、算子泛化等。--> 已完成的测试用例和场景: 1. 2. 补充的UT用例: ## 文档更新 NA ## 合入检查 <!-- 在正式合入前,请做好必要的代码测试,用例补充,软件代码风格检查等。提高合入效率。--> <!-- [x] 表示选中 --> - [x] 🧐 已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 - [ ] 🔍 邀请 committer评论 /lgtm前的必要检查 - [ ] 🏷️ 标题中使用了合适的类型标签(如:[feat], [fix]) - [ ] 📄 代码修改内容已简要描述,相关文档已更新 - [ ] 📝 代码注释已更新,代码遵循项目整体代码风格 - [ ] 🧪 代码UT测试已更新,覆盖率已达标 - [ ] 🔬 验证方法已更新到"测试"部分 - [ ] 🛠️ 代码已通过静态分析工具检查,无错误 - [ ] 👥 代码检视/code review/同行评议和必要的代码串讲,确保代码质量 - [ ] ✅ 代码检视意见已处理或答复,无未处理的检视意见 - [ ] 🚀 预约 前冒烟 用例前的必要检查 - [ ] ✔️ 代码已有committer的/lgtm 和 模块committer的/lgtm评论 - [ ] 🔧 代码已通过compile,编译无错误,无告警 - [ ] 🖥️ 代码已通过基本功能本地测试或者在线测试,确保基本功能正常 - [ ] 🎯 预约 approver评论/approve,正式合入前的必要检查 - [ ] 📊 前冒烟 用例已全量通过 - [ ] 📦 新增功能已同步补充基本功能测试用例到前冒烟里 See merge request: cann/hcomm!3971 | 1 个月前 | |
fix ut Co-authored-by: XuHaoran<xuhaoran45@huawei.com> # message auto-generated for no-merge-commit merge: !3914 merge fix_ut into master fix ut Created-by: howtoexitvim Commit-by: XuHaoran Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动,包括改动的原因和所采取的方法。--> hccp打桩部分使用全局部分使用静态变量修复 ## 变更类型 请选择本次引入的变更类型: <!-- [x] 表示选中 --> - [x] 🐛 Bug修复 - [ ] ✨ 新特性 - [ ] 🚀 性能优化 - [ ] 📝 文档更新 - [ ] 📋 其他,请描述: ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。--> <!-- 如果这个PR不涉及Issue,可填写"NA"。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于构造对应xx测试用例、二级冒烟、算子泛化等。--> 已完成的测试用例和场景: 1. 2. 补充的UT用例: ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> ## 合入检查 <!-- 在正式合入前,请做好必要的代码测试,用例补充,软件代码风格检查等。提高合入效率。--> <!-- [x] 表示选中 --> - [x] 🧐 已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 - [x] 🔍 邀请 committer评论 /lgtm前的必要检查 - [x] 🏷️ 标题中使用了合适的类型标签(如:[feat], [fix]) - [x] 📄 代码修改内容已简要描述,相关文档已更新 - [x] 📝 代码注释已更新,代码遵循项目整体代码风格 - [x] 🧪 代码UT测试已更新,覆盖率已达标 - [x] 🔬 验证方法已更新到"测试"部分 - [x] 🛠️ 代码已通过静态分析工具检查,无错误 - [x] 👥 代码检视/code review/同行评议和必要的代码串讲,确保代码质量 - [x] ✅ 代码检视意见已处理或答复,无未处理的检视意见 - [x] 🚀 预约 前冒烟 用例前的必要检查 - [x] ✔️ 代码已有committer的/lgtm 和 模块committer的/lgtm评论 - [x] 🔧 代码已通过compile,编译无错误,无告警 - [x] 🖥️ 代码已通过基本功能本地测试或者在线测试,确保基本功能正常 - [x] 🎯 预约 approver评论/approve,正式合入前的必要检查 - [x] 📊 前冒烟 用例已全量通过 - [x] 📦 新增功能已同步补充基本功能测试用例到前冒烟里 See merge request: cann/hcomm!3914 | 1 个月前 | |
refact aiv cq Co-authored-by: everglow2<zhangxiaobo40@huawei.com> # message auto-generated for no-merge-commit merge: !3973 merge jfc into master refact aiv cq Created-by: everglow2 Commit-by: everglow2 Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动,包括改动的原因和所采取的方法。--> ### 变更摘要 本次 PR 对 AIV直驱 URMA 场景下的 CQ/JFC 句柄管理机制进行了重构。核心改动是在 DevUbConnection 及其子类(DevUbTpConnection、DevUbCtpConnection、DevUbUboeConnection、DevUbUbgConnection)的构造函数中引入 CommEngine 参数,用于区分 AIV 引擎与其他通信引擎的 JFC 句柄获取路径。针对 AIV 场景,采用无复用的"每次创建新 JFC"策略,直接调用 HrtRaUbCreateJfcUserCtl 创建句柄,并在资源释放时通过 HrtRaUbDestroyJfc 单独销毁。 ### 主要改动 构造函数新增 CommEngine 参数:DevUbConnection 及其四个子类的构造函数签名均增加 CommEngine engine 参数(默认值为 COMM_ENGINE_RESERVED),同时新增 engine_ 成员变量,各子类通过初始化列表逐层传递该参数至基类。 AIV 引擎的 JFC 句柄获取分支:在 DevUbConnection 构造函数中,当 engine_ == COMM_ENGINE_AIV 时,调用新增的 CreateAivUrmaJfc() 方法直接通过 HrtRaUbCreateJfcUserCtl 创建全新 JFC 句柄,绕过原有的 GetJfcHandle / GetJfcHandleAndCqInfo 复用逻辑。 AIV JFC 句柄的独立释放:在 ReleaseResource() 中新增 AIV 引擎判断分支,当 engine_ == COMM_ENGINE_AIV 且 jfcHandle != 0 时,调用 HrtRaUbDestroyJfc 销毁 JFC 句柄,与非 AIV 场景的句柄生命周期管理解耦。 ### 调用侧适配 AivUrmaChannel::BuildConnection() 中创建 DevUbTpConnection 和 DevUbCtpConnection 时,传入 COMM_ENGINE_AIV 以启用 AIV 专用的 JFC 句柄获取路径。 ## 变更类型 请选择本次引入的变更类型: <!-- [x] 表示选中 --> - [x] 🐛 Bug修复 - [ ] ✨ 新特性 - [ ] 🚀 性能优化 - [ ] 📝 文档更新 - [ ] 📋 其他,请描述: ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。--> <!-- 如果这个PR不涉及Issue,可填写"NA"。--> NA ## 测试 已完成的测试用例和场景: 连续创建两个channel的测试用例 二级冒烟 补充的UT用例: Ut_AivEngine_Constructor_CreatesAivUrmaJfc ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> ## 合入检查 <!-- 在正式合入前,请做好必要的代码测试,用例补充,软件代码风格检查等。提高合入效率。--> <!-- [x] 表示选中 --> - [x] 🧐 已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 - [x] 🔍 邀请 committer评论 /lgtm前的必要检查 - [x] 🏷️ 标题中使用了合适的类型标签(如:[feat], [fix]) - [x] 📄 代码修改内容已简要描述,相关文档已更新 - [x] 📝 代码注释已更新,代码遵循项目整体代码风格 - [x] 🧪 代码UT测试已更新,覆盖率已达标 - [x] 🔬 验证方法已更新到"测试"部分 - [x] 🛠️ 代码已通过静态分析工具检查,无错误 - [x] 👥 代码检视/code review/同行评议和必要的代码串讲,确保代码质量 - [x] ✅ 代码检视意见已处理或答复,无未处理的检视意见 - [x] 🚀 预约 前冒烟 用例前的必要检查 - [x] ✔️ 代码已有committer的/lgtm 和 模块committer的/lgtm评论 - [x] 🔧 代码已通过compile,编译无错误,无告警 - [x] 🖥️ 代码已通过基本功能本地测试或者在线测试,确保基本功能正常 - [ ] 🎯 预约 approver评论/approve,正式合入前的必要检查 - [x] 📊 前冒烟 用例已全量通过 - [ ] 📦 新增功能已同步补充基本功能测试用例到前冒烟里 See merge request: cann/hcomm!3973 | 1 个月前 | |
回退Hcomm拓扑探测迁移 Co-authored-by: xumochi<xumochi@huawei.com> # message auto-generated for no-merge-commit merge: !4171 merge revert-pr-3703 into master 回退Hcomm拓扑探测迁移 Created-by: xumochi Commit-by: xumochi Merged-by: cann-robot Description: ## 描述 回退拓扑探测整改导致的整网用例批量失败 ## 变更类型 请选择本次引入的变更类型: <!-- [x] 表示选中 --> - [x] 🐛 Bug修复 - [ ] ✨ 新特性 - [ ] 🚀 性能优化 - [ ] 📝 文档更新 - [ ] 📋 其他,请描述: ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。--> <!-- 如果这个PR不涉及Issue,可填写"NA"。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于构造对应xx测试用例、二级冒烟、算子泛化等。--> 已完成的测试用例和场景: 1. 2. 补充的UT用例: ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> ## 合入检查 <!-- 在正式合入前,请做好必要的代码测试,用例补充,软件代码风格检查等。提高合入效率。--> <!-- [x] 表示选中 --> - [ ] 🧐 已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 - [ ] 🔍 邀请 committer评论 /lgtm前的必要检查 - [ ] 🏷️ 标题中使用了合适的类型标签(如:[feat], [fix]) - [ ] 📄 代码修改内容已简要描述,相关文档已更新 - [ ] 📝 代码注释已更新,代码遵循项目整体代码风格 - [ ] 🧪 代码UT测试已更新,覆盖率已达标 - [ ] 🔬 验证方法已更新到"测试"部分 - [ ] 🛠️ 代码已通过静态分析工具检查,无错误 - [ ] 👥 代码检视/code review/同行评议和必要的代码串讲,确保代码质量 - [ ] ✅ 代码检视意见已处理或答复,无未处理的检视意见 - [ ] 🚀 预约 前冒烟 用例前的必要检查 - [ ] ✔️ 代码已有committer的/lgtm 和 模块committer的/lgtm评论 - [ ] 🔧 代码已通过compile,编译无错误,无告警 - [ ] 🖥️ 代码已通过基本功能本地测试或者在线测试,确保基本功能正常 - [ ] 🎯 预约 approver评论/approve,正式合入前的必要检查 - [ ] 📊 前冒烟 用例已全量通过 - [ ] 📦 新增功能已同步补充基本功能测试用例到前冒烟里 See merge request: cann/hcomm!4171 | 1 个月前 |