0
代码
代码
Issues
Pull Requests
讨论
Wiki
分析
0
cann-robotcann-robotfix: cleancode
c847b7b3创建于 8 天前历史提交
文件最后提交记录最后更新时间
arch22
【A2/A3】【MC2】MatmulReduceScatterV2支持bias输入 Co-authored-by: sangzhenguo<sangzhenguo@huawei.com> # message auto-generated for no-merge-commit merge: !2648 merge matmulreducescatter_bias into master 【A2/A3】【MC2】MatmulReduceScatterV2支持bias输入 Created-by: sangzhenguo Commit-by: sangzhenguo Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> MatmulReduceScatterV2可以支持非空Tensor输入 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!264810 天前
arch35
mc2算子op_kernel代码文件代际隔离 Co-authored-by: hblnb<huangbolin3@huawei.com> # message auto-generated for no-merge-commit merge: !1830 merge agmv2_csv into master mc2算子op_kernel代码文件代际隔离 Created-by: hblnb Commit-by: hblnb Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-transformer!183010 天前
block_mmad_preload_fixpipe.h
Change Template Linear Algebra Co-authored-by: urbub<linhailong3@huawei.com> # message auto-generated for no-merge-commit merge: !3260 merge catlass_3rd into master Change Template Linear Algebra Created-by: urbub Commit-by: urbub Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> 修改mc2/3rd/template_linear_algebra路径下的文件夹名，include到op_kernel。 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> [#1529](https://gitcode.com/cann/ops-transformer/issues/1529) ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [x] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!32602 个月前
common_def.h
MatmulReduceScatterV2 切换AICPU通信引擎 Co-authored-by: andingfangt<miaominghao@huawei.com> # message auto-generated for no-merge-commit merge: !5002 merge master into master MatmulReduceScatterV2 切换AICPU通信引擎 Created-by: andingfangt Commit-by: andingfangt Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> https://gitcode.com/cann/ops-transformer/issues/2423 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： # Ascend C 代码检视报告 **检视对象**：ops-transformer_AICPU eb6bffd57 — MatmulReduceScatterV2 支持环境变量切换 CCU/AICPU 通信引擎 (V2版本) **检视时间**：2026-05-13 **检视技能**：ascendc-code-review（假设检验驱动） **适用规范**：cpp-secure.md、ascendc-topk.md、ascendc-api.md **变更规模**：12 files, +175 / -96 --- ## 一、检视范围 | 层级 | 文件 | 变更 | |------|------|------| | API | op_api/aclnn_matmul_reduce_scatter_v2.cpp | +37 | | Graph | op_graph/matmul_reduce_scatter_v2_gen_task.cpp | +27 | | Tiling | arch35/matmul_reduce_scatter_v2_tiling.cpp | +7 | | Tiling | arch35/quant_bmm_reduce_scatter_tiling.cpp | +10 | | Tiling | matmul_reduce_scatter_tiling_base.cpp | +4 | | Kernel | arch35/matmul_a2a_vec_reduce_fp16_bf16.h | +27 (含删 PRINTF) | | Kernel | arch35/matmul_reduce_scatter_fp16_bf16.h | +49 (含删 PRINTF) | | Kernel | arch35/quant_bmm_a2a_vec_reduce_fp8_hif8.h | +27 (含删 PRINTF) | | Kernel | arch35/quant_bmm_reduce_scatter_fp8_hif8.h | +25 (含删 PRINTF) | | 公共 | op_kernel/common_def.h | +14 | | 公共 | op_kernel/matmul_reduce_scatter_v2_apt.cpp | +32 | | 公共 | op_kernel/matmul_reduce_scatter_v2_apt_tiling_key.h | +12 | --- ## 二、假设检验过程 ### 代码段 1: mc2_comm_utils.h — GetCommModeFromEnv()（Host 侧） **代码**（:27-34）： ```cpp inline uint8_t GetCommModeFromEnv() { auto commModeEnv = getenv(COMM_MODE_ENV); if (commModeEnv != nullptr) { return COMM_MODE_AICPU; } return COMM_MODE_CCU; } ``` | 假设 | 证据收集 | 自信值增量 | |------|---------|-----------| | H1: 存在风险 | 红线#4 指针保护：getenv() 返回 char*，已判空 ✓ | 0% | | | 红线#5 变量初始化：commModeEnv 由 getenv() 赋值 ✓ | 0% | | | 4.1 外部输入校验：环境变量已判空 ✓ | 0% | | | TOPK-1 校验返回值：已校验 ✓ | 0% | | | 无除法、无数组访问、无动态内存 | — | **自信值：0%** → **✅ PASS** --- ### 代码段 2: aclnn_matmul_reduce_scatter_v2.cpp — SetNnopbaseHcclServerTypeByArch()（Host 侧） **代码**（:87-93）： ```cpp static void SetNnopbaseHcclServerTypeByArch(aclOpExecutor *executor) { if ((executor == nullptr) || (NnopbaseSetHcclServerType == nullptr)) { return; } if (GetCurrentPlatformInfo().GetCurNpuArch() == NpuArch::DAV_2201) { NnopbaseSetHcclServerType(executor, NnopbaseHcclServerType::NNOPBASE_HCCL_SERVER_TYPE_MTE); } else if (GetCurrentPlatformInfo().GetCurNpuArch() == NpuArch::DAV_3510) { uint8_t commMode = Mc2Comm::GetCommModeFromEnv(); NnopbaseHcclServerType serverType = (commMode == Mc2Comm::COMM_MODE_AICPU) ? NnopbaseHcclServerType::NNOPBASE_HCCL_SERVER_TYPE_AICPU : NnopbaseHcclServerType::NNOPBASE_HCCL_SERVER_TYPE_CCU; OP_LOGD("[COMM_MODE] Set HcclServerType to %s for DAV_3510.", (commMode == Mc2Comm::COMM_MODE_AICPU) ? "AICPU" : "CCU"); NnopbaseSetHcclServerType(executor, serverType); } } ``` | 假设 | 证据收集 | 自信值增量 | |------|---------|-----------| | H1: 存在风险 | 红线#4 指针保护：executor 判空 ✓、NnopbaseSetHcclServerType 函数指针判空 ✓ | 0% | | | 红线#5 变量初始化：commMode 函数调用赋值 ✓、serverType 三元表达式赋值 ✓ | 0% | | | 11.1 LOG API 空指针：%s 参数为字符串字面量（非 nullptr） ✓ | 0% | | | 11.2 参数数量：1个%s + 1个参数 ✓ | 0% | | | 11.3 类型匹配：%s 匹配 const char* ✓ | 0% | **自信值：0%（无安全风险）** → **✅ PASS** --- ### 代码段 3: aclnn_matmul_reduce_scatter_v2.cpp — 调用侧判空（Host 侧） **代码**（:357-359）： ```cpp if ((executor != nullptr) && (*executor != nullptr)) { SetNnopbaseHcclServerTypeByArch(*executor); } ``` | 假设 | 证据收集 | 自信值增量 | |------|---------|-----------| | H1: 存在风险 | 红线#4 指针保护：双重指针 executor 和 *executor 都判空 ✓ | 0% | **自信值：0%** → **✅ PASS** --- ### 代码段 4: common_def.h — HcclTypeSelector（Kernel 侧） **代码**（:597-604）： ```cpp constexpr int CCU_COMM_MODE = 0; constexpr int AICPU_COMM_MODE = 1; template <int commMode> struct HcclTypeSelector { using type = AscendC::Hccl<AscendC::HcclServerType::HCCL_SERVER_TYPE_CCU>; }; template <> struct HcclTypeSelector<AICPU_COMM_MODE> { using type = AscendC::Hccl<AscendC::HcclServerType::HCCL_SERVER_TYPE_AICPU>; }; ``` | 假设 | 证据收集 | 自信值增量 | |------|---------|-----------| | H1: 存在风险 | 编译期纯类型计算，无运行时行为 ✓ | 0% | | | API-7（动态内存）：无动态分配 ✓ | 0% | | | API-2（std::）：未使用 std:: ✓ | 0% | **自信值：0%（无安全风险）** → **✅ PASS** --- ### 代码段 5: 4 个 Kernel Header — HCCL V1→V2 升级 + PRINTF 清理（Kernel 侧） **代码**（以 matmul_reduce_scatter_fp16_bf16.h:334-337 为例）： ```cpp const void* hcclInitTilingV2 = &(tilingData_->mc2InitTiling); uint64_t hcclCcTilingOffset = offsetof(Mc2Tiling::MatmulReduceScatterV2TilingData, mc2CcTiling); hccl_.InitV2(contextGM, hcclInitTilingV2); hccl_.SetCcTilingV2(hcclCcTilingOffset); ``` | 假设 | 证据收集 | 自信值增量 | |------|---------|-----------| | H1: 存在风险 | 红线#5 变量初始化：hcclInitTilingV2、hcclCcTilingOffset 声明即初始化 ✓ | 0% | | | TOPK-3 局部变量指针生命周期：hcclInitTilingV2 指向 tilingData_ 成员（GM 指针），函数内有效 ✓ | 0% | | | 3.4 sizeof 指针：offsetof 作用于类型名，非指针，用法正确 ✓ | 0% | | | PRINTF 删除：纯清理，无安全影响 ✓ | 0% | **自信值：0%** → **✅ PASS** --- ### 代码段 6: 4 个 Kernel Header — .template 关键字（Kernel 侧） **代码**（以 matmul_reduce_scatter_fp16_bf16.h:398-400 为例）： ```cpp handles_[i + shift] = hccl_.template ReduceScatter<true>( cWork, recvBuffer, recvCount, dataType_, HcclReduceOp::HCCL_REDUCE_SUM, stride, repeat); ``` | 假设 | 证据收集 | 自信值增量 | |------|---------|-----------| | H1: 存在风险 | C++ 标准要求：hccl_ 类型为 typename HcclTypeSelector<TPL_COMM_MODE>::type（依赖类型），模板成员函数必须加 template 关键字，不加则编译失败 ✓ | 0% | | | 2.1 运算溢出：i + shift 为循环索引 + 常量偏移，来自模板参数范围 ✓ | 0% | | | 3.3 数组索引：handles_[i + shift] 在循环内（for i = 0; i < MAX_HANDLE），边界由循环保证 ✓ | 0% | **自信值：0%** → **✅ PASS** --- ### 代码段 7: Tiling + Graph 层 — LOG API 安全（Host 侧） **OP_LOGD 调用**（以 matmul_reduce_scatter_v2_tiling.cpp:150-152 为例）： ```cpp OP_LOGD(opName_, "[COMM_MODE] Set CommEngine to %d (AICPU=%d, CCU=default) for matmul_reduce_scatter_v2.", commMode, Mc2Comm::ENGINE_AICPU); ``` | 假设 | 证据收集 | 自信值增量 | |------|---------|-----------| | H1: 存在风险 | 11.2 参数数量：2个 %d + 2个参数 ✓ | 0% | | | 11.3 类型匹配：%d 匹配 uint8_t（变参提升为 int，值范围 0~2 安全） ✓ | 0% | | | 11.1 空指针：opName_ 为类成员，无 %s 解引用风险 ✓ | 0% | **自信值：0%** → **✅ PASS** > 同类 LOG 调用共 8 处，全部通过 11.1/11.2/11.3 检查。 --- ## 三、风险点清单 **本次检视未发现安全风险。** | 代码段 | 检查项 | 判定 | |--------|--------|------| | GetCommModeFromEnv() | 红线#4/#5、4.1、TOPK-1 | ✅ PASS | | SetNnopbaseHcclServerTypeByArch() | 红线#4/#5、3.5、11.1/11.2/11.3 | ✅ PASS | | 调用侧判空 | 红线#4 双重指针 | ✅ PASS | | HcclTypeSelector | API-2/API-7 | ✅ PASS | | HCCL V2 升级 + PRINTF 清理 | 红线#5、TOPK-3、3.4 | ✅ PASS | | .template 关键字 | C++ 标准合规、3.3 数组边界 | ✅ PASS | | LOG API（8处） | 11.1/11.2/11.3 | ✅ PASS | --- ## 四、红线问题 CheckList | 红线 | 检查结果 | |------|---------| | 1. 除法/求余除零保护 | 无除法/求余操作 | | 2. 数组越界保护 | handles_[] 访问在循环边界内 | | 3. 溢出/下溢保护 | 参与运算的量均为小范围 uint8_t/uint32_t | | 4. 指针保护 | 所有指针使用前已判空 ✓ | | 5. 变量初始化 | 全部声明即初始化 ✓ | | 6. 资源申请/释放匹配 | 无动态资源申请 | --- ## 五、存疑项（非安全问题，供参考） ### 存疑 1：HcclTypeSelector 默认模板静默兜底 - **位置**：op_kernel/common_def.h:597-600 - **描述**：非 AICPU_COMM_MODE(=1) 的任何值都静默回退 CCU，不触发编译错误 - **规范参考**：TOPK-7（输入校验精神，编译期场景无运行时影响） - **建议**： ```cpp // 仅允许已知值，非法值触发编译错误 template <int commMode> struct HcclTypeSelector; // 不定义 template <> struct HcclTypeSelector<0> { using type = ...CCU; }; template <> struct HcclTypeSelector<1> { using type = ...AICPU; }; ``` ### 存疑 2：SetNnopbaseHcclServerTypeByArch 缺少未知架构处理 - **位置**：aclnn_matmul_reduce_scatter_v2.cpp:87-93 - **描述**：仅处理 DAV\_2201 / DAV\_3510，其他架构静默跳过（函数直接 return） - **建议**：添加 else 分支记录 WARNING 日志或断言 --- ## 六、总结 | 维度 | 评价 | |------|------| | 安全性 | ✅ 无安全红线违反 | | API 合规 | ✅ HCCL V2 用法正确，.template 关键字符合 C++ 标准 | | LOG 安全 | ✅ 8 处 OP_LOGD 参数数量/类型匹配 | | 可维护性 | 📝 2 项存疑（非阻塞，不影响合入） | **最终判定：PASS** ✅ See merge request: cann/ops-transformer!500215 天前
copy_gm_to_l1.h
Change Template Linear Algebra Co-authored-by: urbub<linhailong3@huawei.com> # message auto-generated for no-merge-commit merge: !3260 merge catlass_3rd into master Change Template Linear Algebra Created-by: urbub Commit-by: urbub Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> 修改mc2/3rd/template_linear_algebra路径下的文件夹名，include到op_kernel。 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> [#1529](https://gitcode.com/cann/ops-transformer/issues/1529) ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [x] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!32602 个月前
copy_l0c_to_gm.h
Change Template Linear Algebra Co-authored-by: urbub<linhailong3@huawei.com> # message auto-generated for no-merge-commit merge: !3260 merge catlass_3rd into master Change Template Linear Algebra Created-by: urbub Commit-by: urbub Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> 修改mc2/3rd/template_linear_algebra路径下的文件夹名，include到op_kernel。 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> [#1529](https://gitcode.com/cann/ops-transformer/issues/1529) ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [x] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!32602 个月前
matmul.hpp
【A2/A3】【MC2】MatmulReduceScatterV2支持bias输入 Co-authored-by: sangzhenguo<sangzhenguo@huawei.com> # message auto-generated for no-merge-commit merge: !2648 merge matmulreducescatter_bias into master 【A2/A3】【MC2】MatmulReduceScatterV2支持bias输入 Created-by: sangzhenguo Commit-by: sangzhenguo Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> MatmulReduceScatterV2可以支持非空Tensor输入 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!264810 天前
matmul_reduce_scatter_aiv_mode.h
【A2/A3】【MC2】MatmulReduceScatterV2支持bias输入 Co-authored-by: sangzhenguo<sangzhenguo@huawei.com> # message auto-generated for no-merge-commit merge: !2648 merge matmulreducescatter_bias into master 【A2/A3】【MC2】MatmulReduceScatterV2支持bias输入 Created-by: sangzhenguo Commit-by: sangzhenguo Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> MatmulReduceScatterV2可以支持非空Tensor输入 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!264810 天前
matmul_reduce_scatter_aiv_mode_block_epilogue_dequant.h
【A2/A3】【MC2】MatmulReduceScatterV2支持bias输入 Co-authored-by: sangzhenguo<sangzhenguo@huawei.com> # message auto-generated for no-merge-commit merge: !2648 merge matmulreducescatter_bias into master 【A2/A3】【MC2】MatmulReduceScatterV2支持bias输入 Created-by: sangzhenguo Commit-by: sangzhenguo Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> MatmulReduceScatterV2可以支持非空Tensor输入 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!264810 天前
matmul_reduce_scatter_aiv_mode_dequant.h
【A2/A3】【MC2】MatmulReduceScatterV2支持bias输入 Co-authored-by: sangzhenguo<sangzhenguo@huawei.com> # message auto-generated for no-merge-commit merge: !2648 merge matmulreducescatter_bias into master 【A2/A3】【MC2】MatmulReduceScatterV2支持bias输入 Created-by: sangzhenguo Commit-by: sangzhenguo Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> MatmulReduceScatterV2可以支持非空Tensor输入 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!264810 天前
matmul_reduce_scatter_aiv_mode_padding.h
Change Template Linear Algebra Co-authored-by: urbub<linhailong3@huawei.com> # message auto-generated for no-merge-commit merge: !3260 merge catlass_3rd into master Change Template Linear Algebra Created-by: urbub Commit-by: urbub Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> 修改mc2/3rd/template_linear_algebra路径下的文件夹名，include到op_kernel。 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> [#1529](https://gitcode.com/cann/ops-transformer/issues/1529) ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [x] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!32602 个月前
matmul_reduce_scatter_aiv_mode_smallM.h
【A2/A3】【MC2】MatmulReduceScatterV2支持bias输入 Co-authored-by: sangzhenguo<sangzhenguo@huawei.com> # message auto-generated for no-merge-commit merge: !2648 merge matmulreducescatter_bias into master 【A2/A3】【MC2】MatmulReduceScatterV2支持bias输入 Created-by: sangzhenguo Commit-by: sangzhenguo Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> MatmulReduceScatterV2可以支持非空Tensor输入 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!264810 天前
matmul_reduce_scatter_aiv_mode_util.h
修改setBuffFlag同步问题，以及setBuffFlag与checkBuffFlag同步问题 Co-authored-by: gcw_xe6QyzGC<gcw_xe6QyzGC@noreply.gitcode.com> # message auto-generated for no-merge-commit merge: !5556 merge yuhuangqi_AllGatherMM into master 修改setBuffFlag同步问题，以及setBuffFlag与checkBuffFlag同步问题 Created-by: gcw_xe6QyzGC Commit-by: gcw_xe6QyzGC Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> 1、all2allmatmul、matmulall2all、allgathermatmulv2、matmulreducescatterv2四个算子都使用了SetBuffFlag函数，经检视，发现同步问题，补充MTE3_S信号 2、修复allgathermatmulv2中setBuffFlag与checkBuffFlag之间的冲突问题 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> https://gitcode.com/cann/ops-transformer/issues/2127 https://gitcode.com/cann/ops-transformer/issues/2384 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!55569 天前
matmul_reduce_scatter_v2_aiv_mode_tiling.h
add new ops: all_gather_mamtul_v2、matmul_reduce_scatter_v2；update all_gather_mamtul、matmul_reduce_scatter、matmul_all_reduce、moe_update_expert to support Ascend950 Co-authored-by: cann-robot<songchangxia@cann.team> Co-authored-by: lidongsheng<lidongsheng43@huawei.com> # message auto-generated for no-merge-commit merge: !986 merge develop into master add new ops: all_gather_mamtul_v2、matmul_reduce_scatter_v2；update all_gather_mamtul、matmul_reduce_scatter、matmul_all_reduce、moe_update_expert to support Ascend950 Created-by: qq_46353993 Commit-by: cann-robot;lidongsheng Merged-by: cann-robot Description: ## 描述 支持新算子all_gather_mamtul_v2、matmul_reduce_scatter_v2；all_gather_mamtul、matmul_reduce_scatter、matmul_all_reduce、moe_update_expert支持Ascend950 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-transformer!9863 个月前
matmul_reduce_scatter_v2_apt_tiling_key.h
MatmulReduceScatterV2 切换AICPU通信引擎 Co-authored-by: andingfangt<miaominghao@huawei.com> # message auto-generated for no-merge-commit merge: !5002 merge master into master MatmulReduceScatterV2 切换AICPU通信引擎 Created-by: andingfangt Commit-by: andingfangt Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> https://gitcode.com/cann/ops-transformer/issues/2423 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： # Ascend C 代码检视报告 **检视对象**：ops-transformer_AICPU eb6bffd57 — MatmulReduceScatterV2 支持环境变量切换 CCU/AICPU 通信引擎 (V2版本) **检视时间**：2026-05-13 **检视技能**：ascendc-code-review（假设检验驱动） **适用规范**：cpp-secure.md、ascendc-topk.md、ascendc-api.md **变更规模**：12 files, +175 / -96 --- ## 一、检视范围 | 层级 | 文件 | 变更 | |------|------|------| | API | op_api/aclnn_matmul_reduce_scatter_v2.cpp | +37 | | Graph | op_graph/matmul_reduce_scatter_v2_gen_task.cpp | +27 | | Tiling | arch35/matmul_reduce_scatter_v2_tiling.cpp | +7 | | Tiling | arch35/quant_bmm_reduce_scatter_tiling.cpp | +10 | | Tiling | matmul_reduce_scatter_tiling_base.cpp | +4 | | Kernel | arch35/matmul_a2a_vec_reduce_fp16_bf16.h | +27 (含删 PRINTF) | | Kernel | arch35/matmul_reduce_scatter_fp16_bf16.h | +49 (含删 PRINTF) | | Kernel | arch35/quant_bmm_a2a_vec_reduce_fp8_hif8.h | +27 (含删 PRINTF) | | Kernel | arch35/quant_bmm_reduce_scatter_fp8_hif8.h | +25 (含删 PRINTF) | | 公共 | op_kernel/common_def.h | +14 | | 公共 | op_kernel/matmul_reduce_scatter_v2_apt.cpp | +32 | | 公共 | op_kernel/matmul_reduce_scatter_v2_apt_tiling_key.h | +12 | --- ## 二、假设检验过程 ### 代码段 1: mc2_comm_utils.h — GetCommModeFromEnv()（Host 侧） **代码**（:27-34）： ```cpp inline uint8_t GetCommModeFromEnv() { auto commModeEnv = getenv(COMM_MODE_ENV); if (commModeEnv != nullptr) { return COMM_MODE_AICPU; } return COMM_MODE_CCU; } ``` | 假设 | 证据收集 | 自信值增量 | |------|---------|-----------| | H1: 存在风险 | 红线#4 指针保护：getenv() 返回 char*，已判空 ✓ | 0% | | | 红线#5 变量初始化：commModeEnv 由 getenv() 赋值 ✓ | 0% | | | 4.1 外部输入校验：环境变量已判空 ✓ | 0% | | | TOPK-1 校验返回值：已校验 ✓ | 0% | | | 无除法、无数组访问、无动态内存 | — | **自信值：0%** → **✅ PASS** --- ### 代码段 2: aclnn_matmul_reduce_scatter_v2.cpp — SetNnopbaseHcclServerTypeByArch()（Host 侧） **代码**（:87-93）： ```cpp static void SetNnopbaseHcclServerTypeByArch(aclOpExecutor *executor) { if ((executor == nullptr) || (NnopbaseSetHcclServerType == nullptr)) { return; } if (GetCurrentPlatformInfo().GetCurNpuArch() == NpuArch::DAV_2201) { NnopbaseSetHcclServerType(executor, NnopbaseHcclServerType::NNOPBASE_HCCL_SERVER_TYPE_MTE); } else if (GetCurrentPlatformInfo().GetCurNpuArch() == NpuArch::DAV_3510) { uint8_t commMode = Mc2Comm::GetCommModeFromEnv(); NnopbaseHcclServerType serverType = (commMode == Mc2Comm::COMM_MODE_AICPU) ? NnopbaseHcclServerType::NNOPBASE_HCCL_SERVER_TYPE_AICPU : NnopbaseHcclServerType::NNOPBASE_HCCL_SERVER_TYPE_CCU; OP_LOGD("[COMM_MODE] Set HcclServerType to %s for DAV_3510.", (commMode == Mc2Comm::COMM_MODE_AICPU) ? "AICPU" : "CCU"); NnopbaseSetHcclServerType(executor, serverType); } } ``` | 假设 | 证据收集 | 自信值增量 | |------|---------|-----------| | H1: 存在风险 | 红线#4 指针保护：executor 判空 ✓、NnopbaseSetHcclServerType 函数指针判空 ✓ | 0% | | | 红线#5 变量初始化：commMode 函数调用赋值 ✓、serverType 三元表达式赋值 ✓ | 0% | | | 11.1 LOG API 空指针：%s 参数为字符串字面量（非 nullptr） ✓ | 0% | | | 11.2 参数数量：1个%s + 1个参数 ✓ | 0% | | | 11.3 类型匹配：%s 匹配 const char* ✓ | 0% | **自信值：0%（无安全风险）** → **✅ PASS** --- ### 代码段 3: aclnn_matmul_reduce_scatter_v2.cpp — 调用侧判空（Host 侧） **代码**（:357-359）： ```cpp if ((executor != nullptr) && (*executor != nullptr)) { SetNnopbaseHcclServerTypeByArch(*executor); } ``` | 假设 | 证据收集 | 自信值增量 | |------|---------|-----------| | H1: 存在风险 | 红线#4 指针保护：双重指针 executor 和 *executor 都判空 ✓ | 0% | **自信值：0%** → **✅ PASS** --- ### 代码段 4: common_def.h — HcclTypeSelector（Kernel 侧） **代码**（:597-604）： ```cpp constexpr int CCU_COMM_MODE = 0; constexpr int AICPU_COMM_MODE = 1; template <int commMode> struct HcclTypeSelector { using type = AscendC::Hccl<AscendC::HcclServerType::HCCL_SERVER_TYPE_CCU>; }; template <> struct HcclTypeSelector<AICPU_COMM_MODE> { using type = AscendC::Hccl<AscendC::HcclServerType::HCCL_SERVER_TYPE_AICPU>; }; ``` | 假设 | 证据收集 | 自信值增量 | |------|---------|-----------| | H1: 存在风险 | 编译期纯类型计算，无运行时行为 ✓ | 0% | | | API-7（动态内存）：无动态分配 ✓ | 0% | | | API-2（std::）：未使用 std:: ✓ | 0% | **自信值：0%（无安全风险）** → **✅ PASS** --- ### 代码段 5: 4 个 Kernel Header — HCCL V1→V2 升级 + PRINTF 清理（Kernel 侧） **代码**（以 matmul_reduce_scatter_fp16_bf16.h:334-337 为例）： ```cpp const void* hcclInitTilingV2 = &(tilingData_->mc2InitTiling); uint64_t hcclCcTilingOffset = offsetof(Mc2Tiling::MatmulReduceScatterV2TilingData, mc2CcTiling); hccl_.InitV2(contextGM, hcclInitTilingV2); hccl_.SetCcTilingV2(hcclCcTilingOffset); ``` | 假设 | 证据收集 | 自信值增量 | |------|---------|-----------| | H1: 存在风险 | 红线#5 变量初始化：hcclInitTilingV2、hcclCcTilingOffset 声明即初始化 ✓ | 0% | | | TOPK-3 局部变量指针生命周期：hcclInitTilingV2 指向 tilingData_ 成员（GM 指针），函数内有效 ✓ | 0% | | | 3.4 sizeof 指针：offsetof 作用于类型名，非指针，用法正确 ✓ | 0% | | | PRINTF 删除：纯清理，无安全影响 ✓ | 0% | **自信值：0%** → **✅ PASS** --- ### 代码段 6: 4 个 Kernel Header — .template 关键字（Kernel 侧） **代码**（以 matmul_reduce_scatter_fp16_bf16.h:398-400 为例）： ```cpp handles_[i + shift] = hccl_.template ReduceScatter<true>( cWork, recvBuffer, recvCount, dataType_, HcclReduceOp::HCCL_REDUCE_SUM, stride, repeat); ``` | 假设 | 证据收集 | 自信值增量 | |------|---------|-----------| | H1: 存在风险 | C++ 标准要求：hccl_ 类型为 typename HcclTypeSelector<TPL_COMM_MODE>::type（依赖类型），模板成员函数必须加 template 关键字，不加则编译失败 ✓ | 0% | | | 2.1 运算溢出：i + shift 为循环索引 + 常量偏移，来自模板参数范围 ✓ | 0% | | | 3.3 数组索引：handles_[i + shift] 在循环内（for i = 0; i < MAX_HANDLE），边界由循环保证 ✓ | 0% | **自信值：0%** → **✅ PASS** --- ### 代码段 7: Tiling + Graph 层 — LOG API 安全（Host 侧） **OP_LOGD 调用**（以 matmul_reduce_scatter_v2_tiling.cpp:150-152 为例）： ```cpp OP_LOGD(opName_, "[COMM_MODE] Set CommEngine to %d (AICPU=%d, CCU=default) for matmul_reduce_scatter_v2.", commMode, Mc2Comm::ENGINE_AICPU); ``` | 假设 | 证据收集 | 自信值增量 | |------|---------|-----------| | H1: 存在风险 | 11.2 参数数量：2个 %d + 2个参数 ✓ | 0% | | | 11.3 类型匹配：%d 匹配 uint8_t（变参提升为 int，值范围 0~2 安全） ✓ | 0% | | | 11.1 空指针：opName_ 为类成员，无 %s 解引用风险 ✓ | 0% | **自信值：0%** → **✅ PASS** > 同类 LOG 调用共 8 处，全部通过 11.1/11.2/11.3 检查。 --- ## 三、风险点清单 **本次检视未发现安全风险。** | 代码段 | 检查项 | 判定 | |--------|--------|------| | GetCommModeFromEnv() | 红线#4/#5、4.1、TOPK-1 | ✅ PASS | | SetNnopbaseHcclServerTypeByArch() | 红线#4/#5、3.5、11.1/11.2/11.3 | ✅ PASS | | 调用侧判空 | 红线#4 双重指针 | ✅ PASS | | HcclTypeSelector | API-2/API-7 | ✅ PASS | | HCCL V2 升级 + PRINTF 清理 | 红线#5、TOPK-3、3.4 | ✅ PASS | | .template 关键字 | C++ 标准合规、3.3 数组边界 | ✅ PASS | | LOG API（8处） | 11.1/11.2/11.3 | ✅ PASS | --- ## 四、红线问题 CheckList | 红线 | 检查结果 | |------|---------| | 1. 除法/求余除零保护 | 无除法/求余操作 | | 2. 数组越界保护 | handles_[] 访问在循环边界内 | | 3. 溢出/下溢保护 | 参与运算的量均为小范围 uint8_t/uint32_t | | 4. 指针保护 | 所有指针使用前已判空 ✓ | | 5. 变量初始化 | 全部声明即初始化 ✓ | | 6. 资源申请/释放匹配 | 无动态资源申请 | --- ## 五、存疑项（非安全问题，供参考） ### 存疑 1：HcclTypeSelector 默认模板静默兜底 - **位置**：op_kernel/common_def.h:597-600 - **描述**：非 AICPU_COMM_MODE(=1) 的任何值都静默回退 CCU，不触发编译错误 - **规范参考**：TOPK-7（输入校验精神，编译期场景无运行时影响） - **建议**： ```cpp // 仅允许已知值，非法值触发编译错误 template <int commMode> struct HcclTypeSelector; // 不定义 template <> struct HcclTypeSelector<0> { using type = ...CCU; }; template <> struct HcclTypeSelector<1> { using type = ...AICPU; }; ``` ### 存疑 2：SetNnopbaseHcclServerTypeByArch 缺少未知架构处理 - **位置**：aclnn_matmul_reduce_scatter_v2.cpp:87-93 - **描述**：仅处理 DAV\_2201 / DAV\_3510，其他架构静默跳过（函数直接 return） - **建议**：添加 else 分支记录 WARNING 日志或断言 --- ## 六、总结 | 维度 | 评价 | |------|------| | 安全性 | ✅ 无安全红线违反 | | API 合规 | ✅ HCCL V2 用法正确，.template 关键字符合 C++ 标准 | | LOG 安全 | ✅ 8 处 OP_LOGD 参数数量/类型匹配 | | 可维护性 | 📝 2 项存疑（非阻塞，不影响合入） | **最终判定：PASS** ✅ See merge request: cann/ops-transformer!500215 天前
matmul_reduce_scatter_v2_tiling_key.h
【A2/A3】【MC2】MatmulReduceScatterV2支持bias输入 Co-authored-by: sangzhenguo<sangzhenguo@huawei.com> # message auto-generated for no-merge-commit merge: !2648 merge matmulreducescatter_bias into master 【A2/A3】【MC2】MatmulReduceScatterV2支持bias输入 Created-by: sangzhenguo Commit-by: sangzhenguo Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> MatmulReduceScatterV2可以支持非空Tensor输入 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!264810 天前
matmul_smallM.hpp
fix: cleancode Co-authored-by: libohao6<libohao3@huawei.com> # message auto-generated for no-merge-commit merge: !5489 merge fix-cleancode-issues into master fix: cleancode Created-by: libohao6 Commit-by: libohao6 Merged-by: cann-robot Description: ## 描述 修复基础算子主线cleancode问题。 ## 关联的Issue [Bug-Report|缺陷反馈]: MC2基础算子主线存在cleancode问题 #2566 ## 测试 二级冒烟。 ## 文档更新 不涉及。 ## 类型标签 <!-- [x] 表示选中 --> - [x] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： # PR #5489 代码检视报告 **PR**: https://gitcode.com/cann/ops-transformer/pull/5489 **标题**: merge 'fix-cleancode-issues' into 'master' **作者**: libohao6 | **范围**: 19 files, +453 -531 | **日期**: 2026-05-21 --- ## 改动分类 | 类型 | 改动数 | 说明 | |------|-------|------| | 删除未使用函数 | 5个 | matmul_util.cpp删3个static函数 + aclnn_matmul_reduce_scatter_v2.cpp删CreateWinTensor | | 删除冗余变量 | 2个 | x1Dtype/biasDtype 在 aclnn_weight_quant_matmul_all_reduce.cpp | | const修饰添加 | 5处 | 多个Tiling函数context参数加const | | 位操作类型修复 | 1处 | int32_t code → uint32_t bitCode | | 魔法数字→常量 | 1处 | alignas(8) → alignas(L2_CACHE_ALIGNMENT) | | 变量重命名 | 多处 | Kernel侧blockLocCoord→blockLocCoord_等; Host侧index→idx/i | | 参数名拼写修正 | 1处 | rcfCfg→rcsCfg | | include guard | 1处 | matmul_reduce_scatter_v2_tiling_common.h | | 头文件清理 | 3处 | 移除冗余#include | | 未使用参数标记 | 5处 | (void)opType等 | | 注释风格更新 | 大量 | 决策树层级注释从======改为差异化标记 | | 代码格式修复 | 多处 | 空格/缩进/换行 | --- ## 假设检验检视结果 ### 1. 删除未使用函数 ✅ PASS **涉及文件**: matmul_util.cpp, aclnn_matmul_reduce_scatter_v2.cpp 删除5个static函数（CheckShapeValidWithTrans、ProcessEmptyTensor、ProcessEmptyTensorWithTrans、GetMatmulOpInfoWithTrans、CreateWinTensor）。均为static，无跨文件引用风险。 | 规范条款 | 检查项 | 结果 | |---------|--------|------| | cpp-secure 5.2 资源泄露防护 | 删除是否遗漏调用点 | ✅ static函数仅文件内可见 | ### 2. const修饰添加 ✅ PASS **涉及文件**: all_gather_matmul_tiling_base.cpp/.h, quant_all_reduce_tiling.cpp, quant_reduce_scatter_tiling.cpp gert::TilingContext *context → const gert::TilingContext *context。GetNodeName()、GetAttrPointer()等方法为const兼容，头文件声明同步更新。 | 规范条款 | 检查项 | 结果 | |---------|--------|------| | cpp-secure 1.1 静态类型安全 | const修饰是否导致编译错误 | ✅ 方法兼容const | ### 3. 位操作类型修复 ✅ PASS **涉及文件**: matmul_reduce_scatter_v2_aiv_mode_tiling.cpp int32_t code → uint32_t bitCode。对有符号数做右移(>>=)存在符号扩展问题，改为uint32_t消除风险。 | 规范条款 | 检查项 | 结果 | |---------|--------|------| | cpp-secure 2.1 有符号整数不溢出 | int32_t右移符号扩展 | ✅ 修复正确 | | cpp-secure 2.2 无符号整数不回绕 | uint32_t位操作安全 | ✅ | ### 4. 魔法数字→常量 ✅ PASS **涉及文件**: unquant_matmul_all_reduce_tiling_data.h static constexpr size_t L2_CACHE_ALIGNMENT = 8; 替换 alignas(8)。值等价，不影响ABI。 | 规范条款 | 检查项 | 结果 | |---------|--------|------| | cpp-secure 10.4 结构体字段末尾添加 | 常量定义位置 | ✅ 不影响ABI | ### 5. 冗余变量删除 ✅ PASS **涉及文件**: aclnn_weight_quant_matmul_all_reduce.cpp 删除 const auto x1Dtype = x1->GetDataType(); const auto biasDtype = bias->GetDataType();。两变量后续无引用，OP_CHECK_DTYPE_NOT_SAME宏自行获取dtype。 | 规范条款 | 检查项 | 结果 | |---------|--------|------| | cpp-secure 3.1 禁止未初始化变量 | 删除后功能等价 | ✅ | ### 6. 参数名拼写修正 ✅ PASS **涉及文件**: matmul_reduce_scatter_tiling_base.cpp rcfCfg → rcsCfg（RCS=ReduceScatter拼写修正）。所有引用已同步替换。 | 规范条款 | 检查项 | 结果 | |---------|--------|------| | MC2 MC2-13 CCU通信数据量限制 | AdjustHCCLLimit逻辑是否保留 | ✅ 256MB限制保留 | ### 7. include guard添加 ✅ PASS **涉及文件**: matmul_reduce_scatter_v2_tiling_common.h 新增 #ifndef MATMUL_REDUCE_SCATTER_V2_TILING_COMMON_H 保护。命名与文件路径一致。 ### 8. 未使用参数标记 ✅ PASS **涉及文件**: quant_reduce_scatter_util_tiling.cpp 添加 (void)opType; (void)context; (void)xDimNum; (void)outputDim; 消除编译器警告。注释声明"Reserved for future extension"。 | 规范条款 | 检查项 | 结果 | |---------|--------|------| | TOPK TOPK-7 外部输入校验 | 校验逻辑是否削弱 | ✅ 未削弱 | ### 9. LOG参数安全 ✅ PASS **涉及文件**: allto_allv_quant_grouped_mat_mul_tiling_base.cpp 循环变量 index → idx/i。sendCounts[%lu] should be in [0, %lu], but get %lu — 3个%lu对应3个uint64_t(idx, bsk_, sendCounts[idx])，类型和数量匹配。 | 规范条款 | 检查项 | 结果 | |---------|--------|------| | cpp-secure 11.2 参数数量与占位符匹配 | 3个%lu=3个uint64_t | ✅ | | cpp-secure 11.3 参数类型与格式化说明符匹配 | uint64_t用%lu | ✅ | | MC2 MC2-07 专家参数合法性校验 | sendCounts/recvCounts校验完整 | ✅ | ### 10. 头文件清理 ✅ PASS **涉及文件**: matmul_reduce_scatter_tiling_base.h, quant_reduce_scatter_util_tiling.h, all_gather_matmul_tiling_base.h 移除冗余 #include "ops_utils.h" 和 #include "mc2_log.h"。 --- ## 规范条款引用汇总 | 规范来源 | 条款编号 | 条款名称 | 检查结果 | |---------|---------|---------|---------| | cpp-secure.md | 1.1 | 保证静态类型安全 | ✅ | | cpp-secure.md | 2.1 | 有符号整数运算不溢出 | ✅ | | cpp-secure.md | 2.2 | 无符号整数运算不回绕 | ✅ | | cpp-secure.md | 3.1 | 禁止使用未初始化的变量 | ✅ | | cpp-secure.md | 5.2 | 资源泄露防护 | ✅ | | cpp-secure.md | 10.4 | 结构体字段末尾添加 | ✅ | | cpp-secure.md | 11.2 | LOG参数数量与占位符必须匹配 | ✅ | | cpp-secure.md | 11.3 | LOG参数类型与格式化说明符必须匹配 | ✅ | | mc2-specific.md | MC2-07 | 专家参数合法性校验 | ✅ | | mc2-specific.md | MC2-13 | CCU通信数据量限制 | ✅ | | ascendc-topk.md | TOPK-1 | 必须校验函数返回值 | ✅ | | ascendc-topk.md | TOPK-7 | 融合规则/InferShape/Tiling外部输入校验 | ✅ | --- ## 总体结论 **✅ 无问题** — PR #5489 为代码清理类改动，所有10项检视均PASS。改动不引入安全风险，不削弱校验逻辑，规范合规性100%。 See merge request: cann/ops-transformer!54898 天前
tile_broadcast_add.hpp
【A2/A3】【MC2】MatmulReduceScatterV2支持bias输入 Co-authored-by: sangzhenguo<sangzhenguo@huawei.com> # message auto-generated for no-merge-commit merge: !2648 merge matmulreducescatter_bias into master 【A2/A3】【MC2】MatmulReduceScatterV2支持bias输入 Created-by: sangzhenguo Commit-by: sangzhenguo Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> MatmulReduceScatterV2可以支持非空Tensor输入 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!264810 天前