| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
调整common目录结构 Co-authored-by: hello_simida<wangyi206@huawei.com> # message auto-generated for no-merge-commit merge: !2465 merge master_v3 into master 调整common目录结构 Created-by: hello_simida Commit-by: hello_simida Merged-by: cann-robot Description: ## 描述 调整common目录调整 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-transformer!2465 | 3 个月前 | |
调整common目录结构 Co-authored-by: hello_simida<wangyi206@huawei.com> # message auto-generated for no-merge-commit merge: !2465 merge master_v3 into master 调整common目录结构 Created-by: hello_simida Commit-by: hello_simida Merged-by: cann-robot Description: ## 描述 调整common目录调整 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-transformer!2465 | 3 个月前 | |
调整common目录结构 Co-authored-by: hello_simida<wangyi206@huawei.com> # message auto-generated for no-merge-commit merge: !2465 merge master_v3 into master 调整common目录结构 Created-by: hello_simida Commit-by: hello_simida Merged-by: cann-robot Description: ## 描述 调整common目录调整 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-transformer!2465 | 3 个月前 | |
调整common目录结构 Co-authored-by: hello_simida<wangyi206@huawei.com> # message auto-generated for no-merge-commit merge: !2465 merge master_v3 into master 调整common目录结构 Created-by: hello_simida Commit-by: hello_simida Merged-by: cann-robot Description: ## 描述 调整common目录调整 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-transformer!2465 | 3 个月前 | |
调整common目录结构 Co-authored-by: hello_simida<wangyi206@huawei.com> # message auto-generated for no-merge-commit merge: !2465 merge master_v3 into master 调整common目录结构 Created-by: hello_simida Commit-by: hello_simida Merged-by: cann-robot Description: ## 描述 调整common目录调整 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-transformer!2465 | 3 个月前 | |
调整common目录结构 Co-authored-by: hello_simida<wangyi206@huawei.com> # message auto-generated for no-merge-commit merge: !2465 merge master_v3 into master 调整common目录结构 Created-by: hello_simida Commit-by: hello_simida Merged-by: cann-robot Description: ## 描述 调整common目录调整 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-transformer!2465 | 3 个月前 | |
调整common目录结构 Co-authored-by: hello_simida<wangyi206@huawei.com> # message auto-generated for no-merge-commit merge: !2465 merge master_v3 into master 调整common目录结构 Created-by: hello_simida Commit-by: hello_simida Merged-by: cann-robot Description: ## 描述 调整common目录调整 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-transformer!2465 | 3 个月前 | |
[AICPU] 算子根据环境变量选择commMode通信引擎 Co-authored-by: libohao6<libohao3@huawei.com> # message auto-generated for no-merge-commit merge: !4961 merge aicpu_only into master [AICPU] 算子根据环境变量选择commMode通信引擎 Created-by: libohao6 Commit-by: libohao6 Merged-by: cann-robot Description: ## 描述 [AICPU] 算子根据环境变量选择commMode通信引擎 #4961 ## 关联的Issue [Requirement|需求建议]: [AICPU] 算子根据环境变量选择commMode通信引擎 #4961 #2354 ## 测试 level0测试。 ## 文档更新 不涉及。 ## 类型标签 <!-- [x] 表示选中 --> - [.] 🐛 Bug 修复 - [x] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： # MC2代码检视报告 - MR#4961 ## 基本信息 | 项目 | 内容 | |------|------| | MR编号 | #4961 | | 分支 | aicpu_only | | Commit | c3c02630a | | 检视时间 | 2026-05-11 | | 检视人 | AI Code Reviewer | ## 修改概述 支持 ENV_MC2_COMM_MODE_AICPU环境变量切换CCU/AICPU通信模式： 1. 新增Mc2Comm::GetCommModeFromEnv()统一环境变量读取入口(header-only) 2. 全层级环境变量判断: aclnn + gen_task + tiling + kernel 3. gen_task结合平台判断(A5→CCU, 非A5→AICPU) + 环境变量覆盖 4. Kernel层HcclTypeSelector模板选择HCCL服务器类型 5. tilingKey删除D_T_MM和TILINGKEY_MM维度，模板实例数减少67% ## 检视范围 - mc2/common/utils/mc2_comm_utils.h - mc2/allto_allv_quant_grouped_mat_mul/op_kernel/mc2_templates/communication/hccl_a2av_op.h - mc2/allto_allv_grouped_mat_mul/op_kernel/allto_allv_grouped_mat_mul_coarse_grained.h - mc2/allto_allv_quant_grouped_mat_mul/op_host/op_tiling/arch35/allto_allv_quant_grouped_mat_mul_tiling_common.cpp - mc2/allto_allv_grouped_mat_mul/op_host/op_tiling/allto_allv_grouped_mat_mul_tiling.cpp --- ## MC2领域规则检视 ### 规则对照分析 | 规则ID | 规则名称 | 检视结果 | 说明 | |--------|----------|----------|------| | MC2-01 | 核间同步必要性 | ✅ 合规 | 本次修改未新增循环结构调用通信API | | MC2-02 | 流同步正确性 | ✅ 合规 | 未涉及SetFlag/WaitFlag修改 | | MC2-03 | SyncAll同步生效 | ✅ 合规 | 未涉及SyncAll修改 | | MC2-04 | 全局操作一致性 | ✅ 合规 | 环境变量全局生效，所有rank参数一致 | | MC2-16 | 编译规范 | ✅ 合规 | 正确使用ASC_DEVKIT_MAJOR宏隔离 | --- ## 潜在风险点分析 ### 1. 环境变量线程安全性（低风险） **代码位置**: mc2/common/utils/mc2_comm_utils.h **代码片段**: cpp inline uint8_t GetCommModeFromEnv() { auto commModeEnv = getenv(COMM_MODE_ENV); // getenv非线程安全 if (commModeEnv != nullptr) { return COMM_MODE_AICPU; } return COMM_MODE_CCU; } **假设检验**: - 原假设H0：代码线程安全 - 证据：getenv()是非线程安全的POSIX函数（C标准） - 自信值：30%（低风险） **结论**: 低风险，算子执行前环境变量已稳定，无需修改。 --- ### 2. HcclTypeSelector模板默认值一致性（无风险） **代码位置**: - mc2/allto_allv_quant_grouped_mat_mul/op_kernel/mc2_templates/communication/hccl_a2av_op.h - mc2/allto_allv_grouped_mat_mul/op_kernel/allto_allv_grouped_mat_mul_coarse_grained.h **代码片段**: cpp // 量化算子 template<int commMode = TILINGKEY_TPL_CCU> struct HcclTypeSelector { using type = Hccl<HcclServerType::HCCL_SERVER_TYPE_CCU>; // 默认CCU }; // 非量化算子 template<int commMode = CCU_COMM_MODE> struct HcclTypeSelector { using type = Hccl<HcclServerType::HCCL_SERVER_TYPE_CCU>; // 默认CCU }; **假设检验**: - 原假设H0：两个算子默认值一致 - 证据：两者默认值都为CCU（0），一致 - 自信值：0%（无风险） **结论**: ✅ 合规，默认行为一致。 --- ## 环境变量逻辑验证 ### 默认行为（无环境变量） | 层级 | 平台 | 默认行为 | |------|------|----------| | mc2_comm_utils.h | - | 返回CCU(0) | | gen_task | A5 | CCU server/stream | | gen_task | 非A5 | AICPU server/stream | | aclnn | - | NnopbaseSetHcclServerType(CCU) | | tiling | - | SetCommEngine(CCU_ENGINE) | | kernel | - | HCCL_SERVER_TYPE_CCU | ### 强制AICPU行为（设置ENV_MC2_COMM_MODE_AICPU） | 层级 | 行为 | |------|------| | mc2_comm_utils.h | 返回AICPU(1) | | gen_task | 强制aicpu kfc server | | aclnn | NnopbaseSetHcclServerType(AICPU) | | tiling | SetCommEngine(AICPU_ENGINE) | | kernel | HCCL_SERVER_TYPE_AICPU | --- ## 总体结论 **✅ 代码检视通过** 本次修改符合MC2领域规则： - 无新增同步风险点 - HCCL模板选择正确 - 编译规范合规 - 全局参数一致性保证 - 环境变量逻辑清晰，默认行为与原代码一致 --- ## 附录 ### MR链接 https://gitcode.com/cann/ops-transformer/merge_requests/4961 ### Commit链接 https://gitcode.com/cann/ops-transformer/commit/c3c02630a See merge request: cann/ops-transformer!4961 | 1 个月前 | |
target source gen task files to opgraph lib Co-authored-by: liusixia<liusixia@h-partners.com> # message auto-generated for no-merge-commit merge: !2625 merge master into master target source gen task files to opgraph lib Created-by: liusixia_gitcode Commit-by: liusixia Merged-by: cann-robot Description: ## 描述 gen task 文件由ophost 改编入opgraph； mc2_log.h 抽取基础LOG宏为mc2_common_log.h ## 关联的Issue https://gitcode.com/cann/ops-transformer/issues/1215 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [x] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!2625 | 3 个月前 | |
修复A2A3环境下EsTranspose和aclrtGetArgsFromExceptionInfo符号未定义问题 Co-authored-by: y00951989<yangshengjun3@huawei.com> # message auto-generated for no-merge-commit merge: !5077 merge A2A3_pr into master 修复A2A3环境下EsTranspose和aclrtGetArgsFromExceptionInfo符号未定义问题 Created-by: yangshengjun703 Commit-by: y00951989 Merged-by: cann-robot Description: ## 描述 解决A2A3环境下EsTranspose和aclrtGetArgsFromExceptionInfo符号未定义问题 ## 关联Issue 关联的Issue [https://gitcode.com/cann/ops-transformer/issues/2334](url) ## 测试 完成线上A5 A2 A3 rdv 及 本地问题单复现场景A2A3的用例测试，精度通过 # Ascend C 算子代码检视报告 ## 1. 检视概要 **检视文件**：C:\y00951989\5077.diff（171行） **检视模式**：PR 检视（C++安全检视） **检视范围**： - **文件1**：mc2/common/utils/mc2_exception_dump.h - 异常处理动态库加载改造 - **文件2**：mc2/matmul_allto_all/op_graph/fusion_pass/matmul_all_to_all_transpose_a5_fusion_pass.cpp - fusion pass 动态库加载 **核心变更**： - 使用 dlopen/dlsym 动态加载 API 函数，替代直接调用 - 目的：解决符号依赖问题，提升版本兼容性 **问题统计**： - **CRITICAL级别**：2个（空指针解引用、资源泄露） - **MEDIUM级别**：2个（外部输入校验、LOG安全） - **LOW级别**：0个 **检视结论**：该PR引入动态库加载机制解决版本兼容性问题，但存在**2个CRITICAL级别安全风险**，必须在合并前修复。 --- ## 2. 问题详情（按严重级别排序） ### CRITICAL-1：空指针解引用风险（cpp-secure 3.5） **严重级别**：CRITICAL（高） **假设检验过程**： | 步骤 | 假设与证据 | 自信值 | |------|-----------|-------| | **H0** | func 指针在使用前已判空 | 0% | | **证据1** | 行152： static EsTransposeFunc func = GetEsTransposeFunc(); 从 GetEsTransposeFunc() 获取指针 | +40% | | **证据2** | GetEsTransposeFunc() 可能返回 nullptr（dlopen 或 dlsym 失败时，第139、145行） | +30% | | **证据3** | TransposeDL 函数中第154行直接调用 func(...)，未进行判空检查 | +30% | | **H1** | **func 指针可能为 nullptr，直接调用会导致程序崩溃** | **100%** | **关联条款**：cpp-secure 3.5（指针使用前判空）、TOPK-1（必须校验函数返回值） **代码路径**：mc2/matmul_allto_all/op_graph/fusion_pass/matmul_all_to_all_transpose_a5_fusion_pass.cpp（diff行152-154） **问题类型**：内存安全 - 空指针解引用 **问题代码片段**： cpp // 行150-157（TransposeDL 函数） ge::es::EsTensorHolder TransposeDL(const ge::es::EsTensorLike& x, const ge::es::EsTensorLike& perm) { static EsTransposeFunc func = GetEsTransposeFunc(); // ❌ 可能返回 nullptr auto* builder = ge::es::ResolveBuilder(x, perm); auto result = func(x.ToTensorHolder(builder).GetCTensorHolder(), // ❌ 直接调用，未判空 perm.ToTensorHolder(builder).GetCTensorHolder()); return result; } **修改建议**： cpp // ✅ 正确：添加判空检查 ge::es::EsTensorHolder TransposeDL(const ge::es::EsTensorLike& x, const ge::es::EsTensorLike& perm) { static EsTransposeFunc func = GetEsTransposeFunc(); if (func == nullptr) { OPS_LOG_E("MatmulAllToAllTransposeA5FusionPass", "EsTranspose function not loaded"); return ge::es::EsTensorHolder(); // 返回空对象或抛出异常 } auto* builder = ge::es::ResolveBuilder(x, perm); auto result = func(x.ToTensorHolder(builder).GetCTensorHolder(), perm.ToTensorHolder(builder).GetCTensorHolder()); return result; } --- ### CRITICAL-2：资源泄露风险（cpp-secure 5.2） **严重级别**：CRITICAL（高） **假设检验过程**： | 步骤 | 假设与证据 | 自信值 | |------|-----------|-------| | **H0** | dlopen handle 资源正确释放 | 0% | | **证据1** | 行136：dlopen("libes_math.so", RTLD_LAZY | RTLD_GLOBAL) 使用 RTLD_LAZY | RTLD_GLOBAL 标志 | +40% | | **证据2** | RTLD_LAZY | RTLD_GLOBAL 标志会增加动态库的引用计数 | +30% | | **证据3** | 整个函数作用域内无 dlclose 调用（第136-148行） | +30% | | **证据4** | handle 为局部变量，函数返回后丢失，无法后续释放 | +20% | | **H1** | **dlopen 增加了引用计数，但未配对 dlclose，导致资源泄露** | **100%** | **关联条款**：cpp-secure 5.2（资源申请和释放必须匹配） **代码路径**：mc2/matmul_allto_all/op_graph/fusion_pass/matmul_all_to_all_transpose_a5_fusion_pass.cpp（diff行136-148） **问题类型**：资源泄露 - dlopen未配对dlclose **问题分析**： - RTLD_LAZY | RTLD_GLOBAL 标志会加载库并增加引用计数 - 函数返回后 handle 局部变量丢失，无法后续调用 dlclose - 如果库被多次加载（虽然 static func 缓存了结果，但每次检查都会执行 dlopen），会导致库无法被正常卸载 **修改建议**： **方案1：使用 RTLD_NOLOAD，仅查询不增加引用计数**（推荐） cpp // ✅ 正确：使用 RTLD_NOLOAD，不增加引用计数 EsTransposeFunc GetEsTransposeFunc() { void* handle = dlopen("libes_math.so", RTLD_NOLOAD | RTLD_LAZY); // 仅查询 if (!handle) { OPS_LOG_E("MatmulAllToAllTransposeA5FusionPass", "dlopen failed: %s", dlerror()); return nullptr; } dlerror(); auto func = reinterpret_cast<EsTransposeFunc>(dlsym(handle, "EsTranspose")); if (dlerror() != nullptr) { OPS_LOG_E("MatmulAllToAllTransposeA5FusionPass", "dlsym EsTranspose failed"); return nullptr; } return func; // RTLD_NOLOAD 不增加引用计数，无需 dlclose } --- ### MEDIUM-1：外部输入参数未判空（cpp-secure 4.1） **严重级别**：MEDIUM（中） **假设检验过程**： | 步骤 | 假设与证据 | 自信值 | |------|-----------|-------| | **H0** | args 参数正确校验 | 0% | | **证据1** | args 参数作为 aclrtExceptionInfo 指针，属于外部输入（来自异常回调） | +40% | | **证据2** | diff 第81行直接使用 args 参数调用函数，未显示判空检查 | +30% | | **证据3** | 虽然理论上异常回调不应传入 nullptr，但规范要求对外部输入必须校验 | +20% | | **H1** | **args 参数作为外部输入，建议添加判空检查** | **70%** | **关联条款**：cpp-secure 4.1（外部输入数据合法性校验） **代码路径**：mc2/common/utils/mc2_exception_dump.h（diff行81） **问题类型**：外部输入校验 - 参数未判空 **问题代码片段**： cpp // 行81（Mc2ExceptionImpl 函数） auto ret = aclrtGetArgsFromExceptionInfoFunc(args, &devArgsPtr, &devArgsLen); // args 未判空 **修改建议**： cpp // ✅ 正确：添加参数校验 inline void Mc2ExceptionImpl(aclrtExceptionInfo *args, void *userdata, const char *op) { // 添加外部输入校验 if (args == nullptr) { OP_LOGE(OP_NAME, "args is nullptr, invalid exception info"); return; } // Get addr of hccl context from ExceptionInfo void* devArgsPtr = nullptr; uint32_t devArgsLen = 0; ... } --- ### MEDIUM-2：LOG API 潜在空指针风险（cpp-secure 11.1） **严重级别**：MEDIUM（中） **假设检验过程**： | 步骤 | 假设与证据 | 自信值 | |------|-----------|-------| | **H0** | dlerror() 返回值作为 LOG 参数安全 | 0% | | **证据1** | dlerror() 函数在无错误时返回 nullptr | +40% | | **证据2** | 代码中直接将 dlerror() 作为 %s 参数传入 OP_LOGE（第34、37、55、60行） | +30% | | **证据3** | 虽然在错误分支调用理论上应返回错误字符串，但不同平台行为可能有差异 | +20% | | **H1** | **dlerror() 可能返回 nullptr，作为 %s 参数可能导致段错误** | **50%** | **结论**：自信值 **50%** < 60%，无法推翻原假设H0，但存在**需关注**风险点。 **关联条款**：cpp-secure 11.1（LOG API 禁止传入空指针作为字符串参数） **代码路径**：mc2/common/utils/mc2_exception_dump.h（diff行34、37、55、60） **问题类型**：LOG安全 - 潜在空指针风险 **问题代码片段**： cpp // 行34、37、55、60 OP_LOGE(OP_NAME, "dlsym aclrtGetArgsFromExceptionInfo failed: %s", dlerror()); // ❌ dlerror() 可能为 nullptr OP_LOGE(OP_NAME, "dlopen failed: %s", dlerror()); // ❌ dlerror() 可能为 nullptr **修改建议**： cpp // ✅ 正确：判空或使用安全格式 // 方案1：判空 const char* errMsg = dlerror(); OP_LOGE(OP_NAME, "dlopen failed: %s", errMsg ? errMsg : "unknown error"); // 方案2：使用安全格式 OP_LOGE(OP_NAME, "dlopen failed, error code: %p", dlerror()); // 打印指针地址 --- ## 3. 通过条款 以下条款经检视后确认代码符合规范： ### TOPK-13：禁止 thread_local 变量 - ✅ **通过**：代码中使用 static 变量，未使用 thread_local 关键字 ### TOPK-1：必须校验函数返回值 - ✅ **通过**：dlopen、dlsym、aclrtGetArgsFromExceptionInfo、acldumpGetPath 等关键函数均有返回值校验 ### cpp-secure 3.1：禁止使用未初始化的变量 - ✅ **通过**：所有 static 变量均显式初始化（func = nullptr、initialized = false） ### cpp-secure 11.3：LOG API 参数类型必须与格式化说明符匹配 - ✅ **通过**：%s 对应 dlerror() 返回值（char*），%d 对应 aclError 类型（整型），类型匹配 ### mc2_exception_dump.h 中的 RTLD_NOLOAD 使用 - ✅ **合理**：GetAclrtGetArgsFromExceptionInfoFunc() 和 GetAcldumpGetPathFunc() 使用 RTLD_NOLOAD | RTLD_LAZY，仅查询库是否已加载，不增加引用计数，无需 dlclose --- ## 4. 整体评价 ### 优点总结： 1. **架构设计合理**：使用动态加载机制解决版本兼容性问题，避免硬编码符号依赖 2. **函数指针管理规范**：使用 static 变量缓存函数指针，避免重复加载开销 3. **错误处理完善**：dlopen/dlsym 失败时记录日志，提供错误信息 4. **返回值校验完整**：关键函数调用均有返回值校验和错误处理 5. **RTLD_NOLOAD 使用正确**：mc2_exception_dump.h 中使用 RTLD_NOLOAD 避免增加引用计数 ### 需改进项总结： **必须修改（CRITICAL）**： 1. ✅ TransposeDL 函数添加 func 指针判空检查（防止空指针解引用崩溃） 2. ✅ GetEsTransposeFunc() 修改为 RTLD_NOLOAD 或添加 dlclose（防止资源泄露） **建议修改（MEDIUM）**： 1. ✅ Mc2ExceptionImpl 函数添加 args 参数判空检查（外部输入校验） 2. ✅ LOG 宏中的 dlerror() 返回值判空或使用安全格式（防止段错误） ### 综合评分： | 维度 | 评分 | 说明 | |------|------|------| | **安全编码** | 60/100 | 2个CRITICAL安全问题需立即修复 | | **API使用** | 75/100 | dlopen/dlsym 使用基本规范，但存在资源泄露风险 | | **性能优化** | 90/100 | static 缓存设计合理，避免重复加载 | | **代码规范** | 85/100 | 注释清晰，格式规范 | | **版本兼容性** | 95/100 | 动态加载机制解决符号依赖问题 | **总分**：**81/100** --- ## 5. 结论与建议 ### 检视结论： 该PR引入动态库加载机制（dlopen/dlsym）解决版本兼容性问题，架构设计合理。但存在**2个CRITICAL级别的安全风险**（空指针解引用、资源泄露），必须在合并前修复。 ### 修改要求： **必须修复（阻塞合并）**： 1. ✅ **TransposeDL 函数添加判空检查**（cpp-secure 3.5） - 添加 if (func == nullptr) 检查，防止空指针解引用崩溃 - 返回空对象或抛出异常作为错误处理 2. ✅ **GetEsTransposeFunc() 资源泄露修复**（cpp-secure 5.2） - **方案1（推荐）**：改用 RTLD_NOLOAD | RTLD_LAZY，仅查询不增加引用计数 - **方案2**：缓存 handle 并添加清理函数，确保引用计数匹配 **建议修复（推荐）**： 1. ✅ Mc2ExceptionImpl 函数添加 args 参数校验（cpp-secure 4.1） 2. ✅ LOG 宏中 dlerror() 返回值判空（cpp-secure 11.1） ### 建议操作： 1. **立即修复 CRITICAL 问题**：优先修复空指针解引用和资源泄露问题 2. **补充单元测试**：测试 dlopen/dlsym 失败场景，验证错误处理逻辑 3. **添加文档说明**：在 README 或代码注释中说明动态加载机制的使用场景和注意事项 4. **代码 Review**：邀请团队成员 Review 修复后的代码 5. **性能测试**：验证动态加载对性能的影响（虽然 static 缓存可避免重复加载） --- **报告生成时间**：2026-05-08 **检视人**：AI Code Reviewer (ascendc-ops-reviewer) **检视模式**：PR检视（C++安全检视） **规范版本**：cpp-secure.md、ascendc-topk.md **检视状态**：已完成，发现2个 See merge request: cann/ops-transformer!5077 | 1 个月前 | |
MC2切换加载HCCL接口的so Co-authored-by: lidongsheng<lidongsheng43@huawei.com> # message auto-generated for no-merge-commit merge: !2939 merge else into master MC2切换加载HCCL接口的so Created-by: qq_46353993 Commit-by: lidongsheng Merged-by: cann-robot Description: ## 描述 切换mc2算子加载HCCL接口的so名称 ## 关联的Issue [MC2切换加载HCCL接口的so](https://gitcode.com/cann/ops-transformer/issues/1338) ## 测试 编译触发、二级冒烟 ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [x] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!2939 | 3 个月前 | |
调整common目录结构 Co-authored-by: hello_simida<wangyi206@huawei.com> # message auto-generated for no-merge-commit merge: !2465 merge master_v3 into master 调整common目录结构 Created-by: hello_simida Commit-by: hello_simida Merged-by: cann-robot Description: ## 描述 调整common目录调整 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-transformer!2465 | 3 个月前 | |
[修复] MRS算子日志双空格问题 - 移除OP_LOGD消息字符串中的多余前导空格 Co-authored-by: libohao6<libohao3@huawei.com> # message auto-generated for no-merge-commit merge: !5045 merge fix_mmrs_bias_issue into master [修复] MRS算子日志双空格问题 - 移除OP_LOGD消息字符串中的多余前导空格 Created-by: libohao6 Commit-by: libohao6 Merged-by: cann-robot Description: ## 描述 [修复] MRS算子日志双空格问题 - 移除OP_LOGD消息字符串中的多余前导空格 ## 关联的Issue [Bug-Report|缺陷反馈]: MRS算子日志双空格问题 #2291 ## 测试 不涉及逻辑改动。 ## 文档更新 不涉及。 ## 类型标签 <!-- [x] 表示选中 --> - [x] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： # 代码检视报告 **项目名称**：MRS算子日志格式修复检视报告 **检视模块**： - mc2/common/utils/mc2_log.cpp - mc2/matmul_reduce_scatter/op_host/op_tiling/matmul_reduce_scatter_tiling.cpp - mc2/matmul_reduce_scatter_v2/op_host/op_tiling/arch22/matmul_reduce_scatter_v2_aiv_mode_tiling.cpp - mc2/matmul_reduce_scatter_v2/op_host/op_tiling/arch35/matmul_reduce_scatter_v2_tiling.cpp - mc2/matmul_reduce_scatter_v2/op_host/op_tiling/arch35/quant_bmm_reduce_scatter_tiling.cpp - mc2/matmul_reduce_scatter_v2/op_host/op_tiling/matmul_reduce_scatter_tiling_base.cpp **检视人**：Sisyphus **检视日期**：2026-05-09 ## 🔍 检视概览 | 统计项 | 数值 | | ---- | ---- | | 检视文件数 | 6 个 | | 变更行数 | 314行删除 + 315行新增 | | 发现问题总数 | 0 个 | | 严重级（CRITICAL）问题 | 0 个 | | 中等级（MEDIUM）问题 | 0 个 | | 轻微级（LOW）问题 | 1 个（预存问题，非本次修改引入） | | 误报数量 | 0 个 | **核心结论**：本次修改为纯格式修复，移除OP_LOGD格式字符串中的多余前导空格，消除日志输出中的双空格问题。修改范围明确，不涉及任何逻辑变更，不引入新的安全风险。**通过检视，无阻塞问题。** ## ✅ 合规确认 ### 1. 修改类型分析 | 分类 | 说明 | | ---- | ---- | | 修改类型 | 纯格式修复（日志格式字符串空格移除） | | 修改模式 | OP_LOGD(opName, " xxx") → OP_LOGD(opName, "xxx") | | 影响范围 | 仅影响日志输出格式，不影响任何计算逻辑 | | 风险等级 | 低（仅日志可见性变更） | ### 2. 各文件变更统计 | 文件 | 变更行数 | 说明 | | ---- | -------- | ---- | | mc2/common/utils/mc2_log.cpp | 169行 | 公共Tiling日志打印函数 | | matmul_reduce_scatter_tiling.cpp（V1） | 67行 | V1 MRS tiling日志 | | matmul_reduce_scatter_v2_aiv_mode_tiling.cpp | 23行 | V2 AIV模式tiling日志 | | matmul_reduce_scatter_v2_tiling.cpp（arch35） | 9行 | V2 arch35 tiling日志 | | quant_bmm_reduce_scatter_tiling.cpp | 33行 | 量化BMM MRS tiling日志 | | matmul_reduce_scatter_tiling_base.cpp（V2） | 2行 | V2 MRS tiling基类日志 | ### 3. 格式字符串安全性检查 对所有OP_LOGD调用进行逐一核对： | 检查项 | 结果 | 说明 | | ------ | ---- | ---- | | 格式说明符与参数类型匹配 | ✅ 通过 | %d→int、%u→unsigned、%lu→unsigned long、%s→字符串，均正确匹配 | | 格式字符串完整性 | ✅ 通过 | 移除的仅为前导空格字符，未破坏任何格式说明符 | | 参数数量与格式说明符一致 | ✅ 通过 | 未修改任何参数传递，仅修改格式字符串前缀 | | 新增格式化漏洞 | ✅ 无风险 | 无用户输入拼接到格式字符串，均为固定的tiling字段名 | ### 4. C++ 通用编码规范合规性（条款对照） | 规范编号 | 规范名称 | 适用性 | 合规性 | | -------- | -------- | ------ | ------ | | 1.1 | 外部数据合法性检查 | 不涉及 | N/A（无外部数据输入变更） | | 4.2 | 禁止使用魔鬼数字/字符串 | ✅ 合规 | 修改未引入魔鬼数字/字符串 | | 11.1 | 字符串存储确保有'\\0'结束符 | ✅ 合规 | 格式字符串为编译期常量，自动以'\\0'结尾 | | 1.3 | 删除无效冗余代码 | ✅ 合规 | 移除多余空格属于代码清理 | ### 5. 日志输出影响分析 **修改前日志示例**： [opName] tiling.usedCoreNum 32 [opName] tiling.M 1024 （注意opName]后的双空格） **修改后日志示例**： [opName] tiling.usedCoreNum 32 [opName] tiling.M 1024 （正常单空格分隔） **OP_LOGD宏展开分析**：OP_LOGD实现中，opName与格式字符串之间已有分隔符（通常为空格或]），格式字符串内的前导空格导致日志输出出现双空格[opName] message，移除后为标准单空格[opName] message。 ## ⚠️ 预存问题（非本次修改引入） ### 预存问题ID：PREEXIST-001 | 严重级别：LOW（代码风格） **代码路径**：matmul_reduce_scatter_tiling.cpp:152（修改后行号） **问题描述**：该行使用TAB缩进而其他行使用空格缩进，存在缩进不一致。 **代码片段**： cpp OP_LOGD("MatmulReduceScatter", "rcsTiling.storageGather %d", rcsTiling.storageGather); **说明**：此问题为原代码预存问题，本次修改仅移除了前导空格，未改变缩进字符。不影响功能，仅影响代码风格一致性。建议在后续专项代码风格修复中统一处理。 ## 📊 检视结论 | 结论 | 说明 | | ---- | ---- | | 安全性 | **通过** - 无安全风险引入 | | 功能正确性 | **通过** - 仅修改日志格式字符串，不影响计算逻辑 | | 格式字符串安全 | **通过** - 所有格式说明符与参数类型正确匹配 | | 编码规范合规 | **通过** - 符合C++通用编码规范要求 | | 预存问题 | 1个LOW级代码风格问题（缩进不一致），非本次修改引入 | **检视结果：✅ 通过，无阻塞问题。** ## 报告生成时间 2026-05-09 09:44:00 ## 报告状态 已完成检视，通过 See merge request: cann/ops-transformer!5045 | 1 个月前 | |
target source gen task files to opgraph lib Co-authored-by: liusixia<liusixia@h-partners.com> # message auto-generated for no-merge-commit merge: !2625 merge master into master target source gen task files to opgraph lib Created-by: liusixia_gitcode Commit-by: liusixia Merged-by: cann-robot Description: ## 描述 gen task 文件由ophost 改编入opgraph； mc2_log.h 抽取基础LOG宏为mc2_common_log.h ## 关联的Issue https://gitcode.com/cann/ops-transformer/issues/1215 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [x] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!2625 | 3 个月前 | |
调整common目录结构 Co-authored-by: hello_simida<wangyi206@huawei.com> # message auto-generated for no-merge-commit merge: !2465 merge master_v3 into master 调整common目录结构 Created-by: hello_simida Commit-by: hello_simida Merged-by: cann-robot Description: ## 描述 调整common目录调整 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-transformer!2465 | 3 个月前 | |
调整common目录结构 Co-authored-by: hello_simida<wangyi206@huawei.com> # message auto-generated for no-merge-commit merge: !2465 merge master_v3 into master 调整common目录结构 Created-by: hello_simida Commit-by: hello_simida Merged-by: cann-robot Description: ## 描述 调整common目录调整 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-transformer!2465 | 3 个月前 | |
extract mc2 platform info common files Co-authored-by: liusixia<liusixia@h-partners.com> # message auto-generated for no-merge-commit merge: !3149 merge master into master extract mc2 platform info common files Created-by: liusixia_gitcode Commit-by: liusixia Merged-by: cann-robot Description: ## 描述 从mc2的gen_task_utils里抽取 判断当前芯片架构/平台类型的公共接口到mc2/utils/路径下，并相应新增mc2_platform_info.h/.cpp，保持其编入opgraph的lib库不变。 以此满足后续fusion pass 迁移对同类接口需求，不再需要新写接口，避免重复代码。 ## 关联的Issue https://gitcode.com/cann/ops-transformer/issues/1370 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [x] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!3149 | 3 个月前 | |
extract mc2 platform info common files Co-authored-by: liusixia<liusixia@h-partners.com> # message auto-generated for no-merge-commit merge: !3149 merge master into master extract mc2 platform info common files Created-by: liusixia_gitcode Commit-by: liusixia Merged-by: cann-robot Description: ## 描述 从mc2的gen_task_utils里抽取 判断当前芯片架构/平台类型的公共接口到mc2/utils/路径下，并相应新增mc2_platform_info.h/.cpp，保持其编入opgraph的lib库不变。 以此满足后续fusion pass 迁移对同类接口需求，不再需要新写接口，避免重复代码。 ## 关联的Issue https://gitcode.com/cann/ops-transformer/issues/1370 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [x] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!3149 | 3 个月前 | |
调整common目录结构 Co-authored-by: hello_simida<wangyi206@huawei.com> # message auto-generated for no-merge-commit merge: !2465 merge master_v3 into master 调整common目录结构 Created-by: hello_simida Commit-by: hello_simida Merged-by: cann-robot Description: ## 描述 调整common目录调整 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-transformer!2465 | 3 个月前 | |
调整common目录结构 Co-authored-by: hello_simida<wangyi206@huawei.com> # message auto-generated for no-merge-commit merge: !2465 merge master_v3 into master 调整common目录结构 Created-by: hello_simida Commit-by: hello_simida Merged-by: cann-robot Description: ## 描述 调整common目录调整 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-transformer!2465 | 3 个月前 | |
调整common目录结构 Co-authored-by: hello_simida<wangyi206@huawei.com> # message auto-generated for no-merge-commit merge: !2465 merge master_v3 into master 调整common目录结构 Created-by: hello_simida Commit-by: hello_simida Merged-by: cann-robot Description: ## 描述 调整common目录调整 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-transformer!2465 | 3 个月前 | |
调整common目录结构 Co-authored-by: hello_simida<wangyi206@huawei.com> # message auto-generated for no-merge-commit merge: !2465 merge master_v3 into master 调整common目录结构 Created-by: hello_simida Commit-by: hello_simida Merged-by: cann-robot Description: ## 描述 调整common目录调整 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-transformer!2465 | 3 个月前 |
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
| 3 个月前 | ||
| 3 个月前 | ||
| 3 个月前 | ||
| 3 个月前 | ||
| 3 个月前 | ||
| 3 个月前 | ||
| 3 个月前 | ||
| 1 个月前 | ||
| 3 个月前 | ||
| 1 个月前 | ||
| 3 个月前 | ||
| 3 个月前 | ||
| 1 个月前 | ||
| 3 个月前 | ||
| 3 个月前 | ||
| 3 个月前 | ||
| 3 个月前 | ||
| 3 个月前 | ||
| 3 个月前 | ||
| 3 个月前 | ||
| 3 个月前 | ||
| 3 个月前 |