0
代码
代码
Issues
Pull Requests
讨论
Wiki
分析
0
cann-robotcann-robotfix-ccusmoke
3d36c704创建于 18 天前历史提交
文件最后提交记录最后更新时间
fp_matmul_allto_all_tiling_base.cpp
fix-ccusmoke Co-authored-by: qq_43844249<fanglin17@huawei.com> # message auto-generated for no-merge-commit merge: !5138 merge fix-ccusmoke into master fix-ccusmoke Created-by: qq_43844249 Commit-by: qq_43844249 Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> # 代码检视报告 **项目名称**：ops-transformer MC2算子通信引擎修复检视报告 **检视模块**：6个Tiling文件（allto_all/matmul_allto_all系列） **检视人**：Ascend C Code Review Agent **检视日期**：2026-05-11 **检视模式**：PR检视（提交 a8bddade8） **代码侧别**：Tiling侧（Host侧） --- ## 🔍 检视概览 | 统计项 | 数值 | | ---- | ---- | | 检视条款总数 | 8 条 | | 通过条款 | 7 条 | | 需关注条款 | 1 条 | | 发现问题总数 | 0 个（无严重级违规） | | 需关注建议 | 1 个（建议添加注释） | | 检视置信度分布 | HIGH: 0个 / MED: 1个 / LOW: 2个 | **核心结论**：本次修改的技术合理性正确，代码安全性合规，无严重级违规问题。存在1处中等置信度的可读性改进建议（建议添加注释说明修改原因），2处低置信度的类型安全和常量语义提醒（属代码规范建议，非强制整改项）。 **修改概述**： - 修改内容：将 mc2tiling::A5_CCU_ENGINE (值为5) 改为硬编码数字 0 - 修改范围：6个文件，每个文件仅修改1行 - 技术背景：根据 mc2_tiling_utils.h 注释，5 对应 CCU_MS引擎，仅在 A2/3架构支持，A5不支持；改为默认值 0 可解决 CCU Smoke 测试问题 --- ## 📊 检视条款执行详情 | 条款编号 | 条款名称 | 适用范围 | 检视结果 | 置信度 | 备注 | |---------|---------|---------|---------|--------|------| | CLAUSE-1 | 总体原则-静态类型安全（1.1） | All | ✅ PASS | LOW | 存在隐式类型转换（int→uint8_t），但值在范围内 | | CLAUSE-2 | 总体原则-内存安全（1.2） | All | ✅ PASS | - | 不涉及内存操作 | | CLAUSE-3 | 总体原则-禁止未定义行为（1.3） | All | ✅ PASS | - | 合法C++行为 | | CLAUSE-4 | 接口变更兼容性（10.12） | All | ⚠️ 需关注 | MED | 建议添加注释说明修改原因 | | CLAUSE-5 | 返回值校验（TOPK-1） | Host | ✅ PASS | - | 已有 isSuccess() 检查 | | CLAUSE-6 | 外部输入校验（TOPK-7） | Host | ✅ PASS | - | 不引入新的外部输入 | | CLAUSE-7 | 常量单一职责（4.3） | All | ✅ PASS | LOW | 建议添加注释说明语义 | | CLAUSE-8 | 禁止未初始化变量（5.5） | All | ✅ PASS | - | 不涉及变量声明 | --- ## ⚠️ 需关注建议详情 ### 建议 ID：SUGGESTION-001 | 严重级别：MEDIUM（建议） #### 🔬 假设检验过程 **代码段**：SetHcclTiling() 函数中的 withCommEngine(0) 调用 **假设**：H0: 修改不影响代码可读性和可维护性 | 证据序号 | 证据类型 | 规范ID | 证据描述 | 分值增量 | 累计自信值 | |---------|---------|--------|---------|---------|-----------| | 1 | 规范违反（轻） | cpp-secure.md 10.12 | 接口参数值变更未添加注释说明 | +40% | 40% | | 2 | 可读性降低 | - | 硬编码数字 0 缺乏语义说明 | +30% | 70% | **结论**：自信值 **70%** > 60%，**推翻原假设H0**，建议添加注释提升可维护性。 --- **关联规范条款**：cpp-secure.md 10.12（接口变更兼容性） **代码路径**：6个文件的 SetHcclTiling() 函数 - mc2/allto_all_matmul/op_host/op_tiling/arch35/allto_all_fp_matmul_tiling_base.cpp:192 - mc2/allto_all_matmul/op_host/op_tiling/arch35/allto_all_kc_quant_matmul_tiling_base.cpp:236 - mc2/allto_all_matmul/op_host/op_tiling/arch35/allto_all_mx_quant_matmul_tiling_base.cpp:493 - mc2/matmul_allto_all/op_host/op_tiling/arch35/fp_matmul_allto_all_tiling_base.cpp:177 - mc2/matmul_allto_all/op_host/op_tiling/arch35/kc_quant_matmul_allto_all_tiling_base.cpp:148 - mc2/matmul_allto_all/op_host/op_tiling/arch35/mx_quant_matmul_allto_all_tiling_base.cpp:608 **问题类型**：代码可读性建议 **问题描述**：修改将语义明确的常量 mc2tiling::A5_CCU_ENGINE 改为硬编码数字 0，降低了代码的可读性和可维护性。后续维护者需要查阅注释才能理解 0 的含义（默认通信引擎），增加了理解成本。虽用户明确排除魔鬼数字规范检查，但从工程实践角度，建议添加注释说明修改原因和技术背景。 --- #### 修改建议 **修改前代码**： ```cpp // reducetype接口附带的数据类型优先于调用通信接口传入的数据类型，因此这里需要设置 AscendC::Mc2CcTilingConfig allToAllTilingConfig = allToAllBuilder.withCommEngine(0) .withReduceType(opName_, AscendC::HcclReduceOp::HCCL_REDUCE_SUM, contextInfo_.args_.geAType, contextInfo_.args_.geAType) .build(); ``` **修改后代码**： ```cpp // reducetype接口附带的数据类型优先于调用通信接口传入的数据类型，因此这里需要设置 // 【CCU Smoke Fix】A5架构不支持CCU_MS引擎（值为5），改用默认引擎（值为0） // 参考：mc2_tiling_utils.h注释："5：CCU_MS（A2/3支持 A5不支持）" AscendC::Mc2CcTilingConfig allToAllTilingConfig = allToAllBuilder.withCommEngine(0) // 默认通信引擎，适配A5架构 .withReduceType(opName_, AscendC::HcclReduceOp::HCCL_REDUCE_SUM, contextInfo_.args_.geAType, contextInfo_.args_.geAType) .build(); ``` **修改说明**：添加注释说明修改原因和技术背景，提升代码可维护性。注释包含： 1. 修改原因：解决 A5 架构 CCU Smoke 测试问题 2. 技术背景：引用 mc2_tiling_utils.h 注释，说明 5 在 A5 不支持 3. 语义说明：标注 0 表示默认通信引擎 --- ## ✅ 通过条款补充说明 ### CLAUSE-1：静态类型安全（置信度 LOW） **分析**：修改将 int 类型字面量 0 传递给可能期望 uint8_t 的 API（根据常量定义推断），存在隐式类型转换。但： - C++ 允许 int 到 uint8_t 的隐式转换（缩窄转换） - 值 0 在 uint8_t 表示范围内（0-255） - 无类型溢出风险 **建议**：类型转换风险较低，但建议添加注释说明类型兼容性。 --- ### CLAUSE-7：常量单一职责（置信度 LOW） **分析**：硬编码数字 0 的语义明确（根据注释"0：默认值"），保持单一职责（表示默认通信引擎）。但可读性有所降低，建议添加注释辅助理解。 --- ## 🔧 技术合理性验证 ### 修改背景分析 根据 mc2_tiling_utils.h 第 51-56 行注释： ```cpp /** 当前通信API未提供枚举，后续会提供 0：默认值 1：HOST_TS（A2/3支持 A5不支持）2：AICPU_TS（A2/3支持 A5不支持） 3：AIV 4：AIV_ONLY（A2/3支持 A5不支持） 5：CCU_MS（A2/3支持 A5不支持） 6：CCU_SCHED（A2/3支持 A5不支持） 7：AICPU_UB/ROCE（A5不支持） **/ ``` **技术合理性**： - mc2tiling::A5_CCU_ENGINE = 5 对应 CCU_MS，注释明确标注"**A5不支持**" - 改为默认值 0 是合理的修复方案 - 其他算子（moe_distribute 系列）仍使用 A5_CCU_ENGINE，可能需要后续修复 ### 功能影响评估 **正面影响**： - 解决 A5 架构 CCU Smoke 测试失败问题 - 使用默认通信引擎，适配 A5 硬件限制 **潜在风险**： - 默认引擎 0 在 A5 上的性能是否与预期一致？建议验证性能 - 其他算子（moe_distribute）是否需要相同修复？建议统一处理 --- ## 📝 整改建议汇总 | 建议编号 | 建议类型 | 严重级别 | 建议内容 | 优先级 | |---------|---------|---------|---------|--------| | SUGGESTION-001 | 可读性改进 | MEDIUM | 添加注释说明修改原因和技术背景 | P2（建议执行） | | SUGGESTION-002 | 代码一致性 | LOW | 检查 moe_distribute 系列算子是否需要相同修复 | P3（可选） | | SUGGESTION-003 | 性能验证 | LOW | 验证默认引擎 0 在 A5 架构上的性能表现 | P3（可选） | --- ## ✅ 检视结论 **合规性评估**： - ✅ C++安全编码规范：8条条款全部通过（无严重级违规） - ✅ TOPK问题清单：相关条款全部通过 - ⚠️ 代码可读性：建议添加注释（非强制整改项） **技术合理性评估**： - ✅ 修改背景清晰：A5架构不支持CCU_MS引擎（值为5） - ✅ 修复方案合理：改为默认值（0） - ⚠️ 需验证性能和一致性 **整改建议**： 1. **建议执行**：添加注释说明修改原因（6个文件统一修改） 2. **可选执行**：检查 moe_distribute 系列算子是否需要相同修复 3. **可选执行**：验证默认引擎在 A5 上的性能表现 **最终结论**：本次修改**技术合理性正确，安全性合规**，建议添加注释提升可维护性，无需强制整改。 --- ## 📄 报告生成信息 **报告生成时间**：2026-05-11 **报告状态**：已完成检视，建议整改（非强制） **检视覆盖率**：8/8 条款（100%覆盖） **检视方法**：假设检验驱动 + 证据链追溯 --- ## 附录：修改文件列表 | 序号 | 文件路径 | 修改行 | 修改内容 | |-----|---------|--------|---------| | 1 | mc2/allto_all_matmul/op_host/op_tiling/arch35/allto_all_fp_matmul_tiling_base.cpp | 192 | mc2tiling::A5_CCU_ENGINE → 0 | | 2 | mc2/allto_all_matmul/op_host/op_tiling/arch35/allto_all_kc_quant_matmul_tiling_base.cpp | 236 | mc2tiling::A5_CCU_ENGINE → 0 | | 3 | mc2/allto_all_matmul/op_host/op_tiling/arch35/allto_all_mx_quant_matmul_tiling_base.cpp | 493 | mc2tiling::A5_CCU_ENGINE → 0 | | 4 | mc2/matmul_allto_all/op_host/op_tiling/arch35/fp_matmul_allto_all_tiling_base.cpp | 177 | mc2tiling::A5_CCU_ENGINE → 0 | | 5 | mc2/matmul_allto_all/op_host/op_tiling/arch35/kc_quant_matmul_allto_all_tiling_base.cpp | 148 | mc2tiling::A5_CCU_ENGINE → 0 | | 6 | mc2/matmul_allto_all/op_host/op_tiling/arch35/mx_quant_matmul_allto_all_tiling_base.cpp | 608 | mc2tiling::A5_CCU_ENGINE → 0 | ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [x] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!513818 天前
fp_matmul_allto_all_tiling_base.h
refact arch:根据代际隔离要求重构matmulalltoall Co-authored-by: adamlwang<wangjingquan2@huawei.com> # message auto-generated for no-merge-commit merge: !4952 merge refact into master refact arch:根据代际隔离要求重构matmulalltoall Created-by: adamlwang Commit-by: adamlwang Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> 根据代际隔离要求重构matmulalltoall. ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> https://gitcode.com/cann/ops-transformer/issues/2327 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [x] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： # PR 4952 代码检视报告 ## PR概述 **PR标题**: merge 'refact' into 'master' **变更概要**: 重构 matmul_allto_all 算子 tiling 代码架构，删除工厂类，将工厂逻辑迁移到基类和arch35子类的虚函数中。 **代码侧别**: Tiling侧（Host侧） **检视模式**: PR检视（仅关注变更代码） ## 检视统计 - **检视条款**: 9条重点条款（TOPK问题+红线问题） - **通过条款**: 9条（100%通过） - **发现问题**: 0个高风险问题 - **需关注**: 1个低风险项（已验证） ## 检视结果详情 ### ✅ 通过条款（9条） | 条款ID | 条款标题 | 检视结果 | 说明 | |--------|---------|---------|------| | TOPK-1 | 必须校验函数返回值 | **通过** | 用户反馈已确认第66行返回值问题OK | | TOPK-3 | 避免野指针 | **通过** | 无局部变量指针传递风险 | | TOPK-7 | 外部输入校验 | **通过** | args.rankDim 通过判断函数校验 | | TOPK-8 | GM内存偏移int64 | **通过** | 本次变更无涉及GM地址计算 | | 红线2 | 除零保护 | **通过** | 无除法/求余运算 | | 红线3 | 溢出保护 | **通过** | 无数值溢出风险运算 | | 红线4 | 指针保护 | **通过** | 无指针解引用操作 | | 红线5 | 变量初始化 | **通过** | 继承体系初始化完整 | | 红线6 | 资源配对 | **通过** | 无动态资源申请 | ### 需关注项（低风险，已验证） **关注项1**: formulaicTiling.GetTiling() 返回值未直接使用 **位置**: matmul_allto_all_fit_balance_tiling.h:64 **代码片段**: ```cpp AlltoAllMM formulaicTiling(args, args.rankDim, kernelType, socVersion); formulaicTiling.GetTiling(); // 返回值未使用 return formulaicTiling.tilingM_.cutRes; ``` **风险评估**: 低风险（LOW，置信度20%） - 返回值虽未校验，但下一行直接返回 tilingM_.cutRes - 该设计可能是故意忽略返回值，直接访问成员变量 - 用户反馈已确认之前的问题OK，本关注项仅作记录 ## 变更文件清单 ### 新增文件 - 无 ### 修改文件（8个） 1. mc2/matmul_allto_all/op_host/CMakeLists.txt - 删除tiling编译配置 2. mc2/matmul_allto_all/op_host/op_tiling/arch35/fp_matmul_allto_all_tiling_base.cpp - 新增 GetTilingResult() 3. mc2/matmul_allto_all/op_host/op_tiling/arch35/fp_matmul_allto_all_tiling_base.h - 新增虚函数声明 4. mc2/matmul_allto_all/op_host/op_tiling/arch35/kc_quant_matmul_allto_all_tiling_base.cpp - 新增 GetTilingResult() 5. mc2/matmul_allto_all/op_host/op_tiling/arch35/kc_quant_matmul_allto_all_tiling_base.h - 新增虚函数声明 6. mc2/matmul_allto_all/op_host/op_tiling/arch35/mx_quant_matmul_allto_all_tiling_base.cpp - 新增 GetTilingResult() 7. mc2/matmul_allto_all/op_host/op_tiling/arch35/mx_quant_matmul_allto_all_tiling_base.h - 新增虚函数声明 8. mc2/matmul_allto_all/op_host/op_tiling/arch35/matmul_allto_all_fit_balance_tiling.h - 新增 GetArch35TilingResult() ### 删除文件（2个） 1. mc2/matmul_allto_all/op_host/op_tiling/common/allto_all_tiling_factory.cpp 2. mc2/matmul_allto_all/op_host/op_tiling/common/allto_all_tiling_factory.h ### 基类修改（2个） 1. mc2/matmul_allto_all/op_host/op_tiling/matmul_allto_all_tiling_base.cpp - 简化基类实现 2. mc2/matmul_allto_all/op_host/op_tiling/matmul_allto_all_tiling_base.h - 改为虚函数 ## 总体结论 **✅ 检视通过** 本次PR重构符合Ascend C算子开发规范，重点条款100%通过。变更代码： 1. 无数值安全风险（除零、溢出） 2. 无内存安全风险（野指针、未初始化） 3. 无输入验证缺失 4. 无资源管理风险 所有红线问题和TOPK问题均通过检视，代码质量良好。 --- **检视时间**: 2026-05-11 **检视工具**: Ascend C Code Review Agent **规范来源**: ascendc-topk.md + cpp-secure.md See merge request: cann/ops-transformer!495218 天前
kc_quant_matmul_allto_all_tiling_base.cpp
fix-ccusmoke Co-authored-by: qq_43844249<fanglin17@huawei.com> # message auto-generated for no-merge-commit merge: !5138 merge fix-ccusmoke into master fix-ccusmoke Created-by: qq_43844249 Commit-by: qq_43844249 Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> # 代码检视报告 **项目名称**：ops-transformer MC2算子通信引擎修复检视报告 **检视模块**：6个Tiling文件（allto_all/matmul_allto_all系列） **检视人**：Ascend C Code Review Agent **检视日期**：2026-05-11 **检视模式**：PR检视（提交 a8bddade8） **代码侧别**：Tiling侧（Host侧） --- ## 🔍 检视概览 | 统计项 | 数值 | | ---- | ---- | | 检视条款总数 | 8 条 | | 通过条款 | 7 条 | | 需关注条款 | 1 条 | | 发现问题总数 | 0 个（无严重级违规） | | 需关注建议 | 1 个（建议添加注释） | | 检视置信度分布 | HIGH: 0个 / MED: 1个 / LOW: 2个 | **核心结论**：本次修改的技术合理性正确，代码安全性合规，无严重级违规问题。存在1处中等置信度的可读性改进建议（建议添加注释说明修改原因），2处低置信度的类型安全和常量语义提醒（属代码规范建议，非强制整改项）。 **修改概述**： - 修改内容：将 mc2tiling::A5_CCU_ENGINE (值为5) 改为硬编码数字 0 - 修改范围：6个文件，每个文件仅修改1行 - 技术背景：根据 mc2_tiling_utils.h 注释，5 对应 CCU_MS引擎，仅在 A2/3架构支持，A5不支持；改为默认值 0 可解决 CCU Smoke 测试问题 --- ## 📊 检视条款执行详情 | 条款编号 | 条款名称 | 适用范围 | 检视结果 | 置信度 | 备注 | |---------|---------|---------|---------|--------|------| | CLAUSE-1 | 总体原则-静态类型安全（1.1） | All | ✅ PASS | LOW | 存在隐式类型转换（int→uint8_t），但值在范围内 | | CLAUSE-2 | 总体原则-内存安全（1.2） | All | ✅ PASS | - | 不涉及内存操作 | | CLAUSE-3 | 总体原则-禁止未定义行为（1.3） | All | ✅ PASS | - | 合法C++行为 | | CLAUSE-4 | 接口变更兼容性（10.12） | All | ⚠️ 需关注 | MED | 建议添加注释说明修改原因 | | CLAUSE-5 | 返回值校验（TOPK-1） | Host | ✅ PASS | - | 已有 isSuccess() 检查 | | CLAUSE-6 | 外部输入校验（TOPK-7） | Host | ✅ PASS | - | 不引入新的外部输入 | | CLAUSE-7 | 常量单一职责（4.3） | All | ✅ PASS | LOW | 建议添加注释说明语义 | | CLAUSE-8 | 禁止未初始化变量（5.5） | All | ✅ PASS | - | 不涉及变量声明 | --- ## ⚠️ 需关注建议详情 ### 建议 ID：SUGGESTION-001 | 严重级别：MEDIUM（建议） #### 🔬 假设检验过程 **代码段**：SetHcclTiling() 函数中的 withCommEngine(0) 调用 **假设**：H0: 修改不影响代码可读性和可维护性 | 证据序号 | 证据类型 | 规范ID | 证据描述 | 分值增量 | 累计自信值 | |---------|---------|--------|---------|---------|-----------| | 1 | 规范违反（轻） | cpp-secure.md 10.12 | 接口参数值变更未添加注释说明 | +40% | 40% | | 2 | 可读性降低 | - | 硬编码数字 0 缺乏语义说明 | +30% | 70% | **结论**：自信值 **70%** > 60%，**推翻原假设H0**，建议添加注释提升可维护性。 --- **关联规范条款**：cpp-secure.md 10.12（接口变更兼容性） **代码路径**：6个文件的 SetHcclTiling() 函数 - mc2/allto_all_matmul/op_host/op_tiling/arch35/allto_all_fp_matmul_tiling_base.cpp:192 - mc2/allto_all_matmul/op_host/op_tiling/arch35/allto_all_kc_quant_matmul_tiling_base.cpp:236 - mc2/allto_all_matmul/op_host/op_tiling/arch35/allto_all_mx_quant_matmul_tiling_base.cpp:493 - mc2/matmul_allto_all/op_host/op_tiling/arch35/fp_matmul_allto_all_tiling_base.cpp:177 - mc2/matmul_allto_all/op_host/op_tiling/arch35/kc_quant_matmul_allto_all_tiling_base.cpp:148 - mc2/matmul_allto_all/op_host/op_tiling/arch35/mx_quant_matmul_allto_all_tiling_base.cpp:608 **问题类型**：代码可读性建议 **问题描述**：修改将语义明确的常量 mc2tiling::A5_CCU_ENGINE 改为硬编码数字 0，降低了代码的可读性和可维护性。后续维护者需要查阅注释才能理解 0 的含义（默认通信引擎），增加了理解成本。虽用户明确排除魔鬼数字规范检查，但从工程实践角度，建议添加注释说明修改原因和技术背景。 --- #### 修改建议 **修改前代码**： ```cpp // reducetype接口附带的数据类型优先于调用通信接口传入的数据类型，因此这里需要设置 AscendC::Mc2CcTilingConfig allToAllTilingConfig = allToAllBuilder.withCommEngine(0) .withReduceType(opName_, AscendC::HcclReduceOp::HCCL_REDUCE_SUM, contextInfo_.args_.geAType, contextInfo_.args_.geAType) .build(); ``` **修改后代码**： ```cpp // reducetype接口附带的数据类型优先于调用通信接口传入的数据类型，因此这里需要设置 // 【CCU Smoke Fix】A5架构不支持CCU_MS引擎（值为5），改用默认引擎（值为0） // 参考：mc2_tiling_utils.h注释："5：CCU_MS（A2/3支持 A5不支持）" AscendC::Mc2CcTilingConfig allToAllTilingConfig = allToAllBuilder.withCommEngine(0) // 默认通信引擎，适配A5架构 .withReduceType(opName_, AscendC::HcclReduceOp::HCCL_REDUCE_SUM, contextInfo_.args_.geAType, contextInfo_.args_.geAType) .build(); ``` **修改说明**：添加注释说明修改原因和技术背景，提升代码可维护性。注释包含： 1. 修改原因：解决 A5 架构 CCU Smoke 测试问题 2. 技术背景：引用 mc2_tiling_utils.h 注释，说明 5 在 A5 不支持 3. 语义说明：标注 0 表示默认通信引擎 --- ## ✅ 通过条款补充说明 ### CLAUSE-1：静态类型安全（置信度 LOW） **分析**：修改将 int 类型字面量 0 传递给可能期望 uint8_t 的 API（根据常量定义推断），存在隐式类型转换。但： - C++ 允许 int 到 uint8_t 的隐式转换（缩窄转换） - 值 0 在 uint8_t 表示范围内（0-255） - 无类型溢出风险 **建议**：类型转换风险较低，但建议添加注释说明类型兼容性。 --- ### CLAUSE-7：常量单一职责（置信度 LOW） **分析**：硬编码数字 0 的语义明确（根据注释"0：默认值"），保持单一职责（表示默认通信引擎）。但可读性有所降低，建议添加注释辅助理解。 --- ## 🔧 技术合理性验证 ### 修改背景分析 根据 mc2_tiling_utils.h 第 51-56 行注释： ```cpp /** 当前通信API未提供枚举，后续会提供 0：默认值 1：HOST_TS（A2/3支持 A5不支持）2：AICPU_TS（A2/3支持 A5不支持） 3：AIV 4：AIV_ONLY（A2/3支持 A5不支持） 5：CCU_MS（A2/3支持 A5不支持） 6：CCU_SCHED（A2/3支持 A5不支持） 7：AICPU_UB/ROCE（A5不支持） **/ ``` **技术合理性**： - mc2tiling::A5_CCU_ENGINE = 5 对应 CCU_MS，注释明确标注"**A5不支持**" - 改为默认值 0 是合理的修复方案 - 其他算子（moe_distribute 系列）仍使用 A5_CCU_ENGINE，可能需要后续修复 ### 功能影响评估 **正面影响**： - 解决 A5 架构 CCU Smoke 测试失败问题 - 使用默认通信引擎，适配 A5 硬件限制 **潜在风险**： - 默认引擎 0 在 A5 上的性能是否与预期一致？建议验证性能 - 其他算子（moe_distribute）是否需要相同修复？建议统一处理 --- ## 📝 整改建议汇总 | 建议编号 | 建议类型 | 严重级别 | 建议内容 | 优先级 | |---------|---------|---------|---------|--------| | SUGGESTION-001 | 可读性改进 | MEDIUM | 添加注释说明修改原因和技术背景 | P2（建议执行） | | SUGGESTION-002 | 代码一致性 | LOW | 检查 moe_distribute 系列算子是否需要相同修复 | P3（可选） | | SUGGESTION-003 | 性能验证 | LOW | 验证默认引擎 0 在 A5 架构上的性能表现 | P3（可选） | --- ## ✅ 检视结论 **合规性评估**： - ✅ C++安全编码规范：8条条款全部通过（无严重级违规） - ✅ TOPK问题清单：相关条款全部通过 - ⚠️ 代码可读性：建议添加注释（非强制整改项） **技术合理性评估**： - ✅ 修改背景清晰：A5架构不支持CCU_MS引擎（值为5） - ✅ 修复方案合理：改为默认值（0） - ⚠️ 需验证性能和一致性 **整改建议**： 1. **建议执行**：添加注释说明修改原因（6个文件统一修改） 2. **可选执行**：检查 moe_distribute 系列算子是否需要相同修复 3. **可选执行**：验证默认引擎在 A5 上的性能表现 **最终结论**：本次修改**技术合理性正确，安全性合规**，建议添加注释提升可维护性，无需强制整改。 --- ## 📄 报告生成信息 **报告生成时间**：2026-05-11 **报告状态**：已完成检视，建议整改（非强制） **检视覆盖率**：8/8 条款（100%覆盖） **检视方法**：假设检验驱动 + 证据链追溯 --- ## 附录：修改文件列表 | 序号 | 文件路径 | 修改行 | 修改内容 | |-----|---------|--------|---------| | 1 | mc2/allto_all_matmul/op_host/op_tiling/arch35/allto_all_fp_matmul_tiling_base.cpp | 192 | mc2tiling::A5_CCU_ENGINE → 0 | | 2 | mc2/allto_all_matmul/op_host/op_tiling/arch35/allto_all_kc_quant_matmul_tiling_base.cpp | 236 | mc2tiling::A5_CCU_ENGINE → 0 | | 3 | mc2/allto_all_matmul/op_host/op_tiling/arch35/allto_all_mx_quant_matmul_tiling_base.cpp | 493 | mc2tiling::A5_CCU_ENGINE → 0 | | 4 | mc2/matmul_allto_all/op_host/op_tiling/arch35/fp_matmul_allto_all_tiling_base.cpp | 177 | mc2tiling::A5_CCU_ENGINE → 0 | | 5 | mc2/matmul_allto_all/op_host/op_tiling/arch35/kc_quant_matmul_allto_all_tiling_base.cpp | 148 | mc2tiling::A5_CCU_ENGINE → 0 | | 6 | mc2/matmul_allto_all/op_host/op_tiling/arch35/mx_quant_matmul_allto_all_tiling_base.cpp | 608 | mc2tiling::A5_CCU_ENGINE → 0 | ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [x] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!513818 天前
kc_quant_matmul_allto_all_tiling_base.h
refact arch:根据代际隔离要求重构matmulalltoall Co-authored-by: adamlwang<wangjingquan2@huawei.com> # message auto-generated for no-merge-commit merge: !4952 merge refact into master refact arch:根据代际隔离要求重构matmulalltoall Created-by: adamlwang Commit-by: adamlwang Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> 根据代际隔离要求重构matmulalltoall. ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> https://gitcode.com/cann/ops-transformer/issues/2327 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [x] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： # PR 4952 代码检视报告 ## PR概述 **PR标题**: merge 'refact' into 'master' **变更概要**: 重构 matmul_allto_all 算子 tiling 代码架构，删除工厂类，将工厂逻辑迁移到基类和arch35子类的虚函数中。 **代码侧别**: Tiling侧（Host侧） **检视模式**: PR检视（仅关注变更代码） ## 检视统计 - **检视条款**: 9条重点条款（TOPK问题+红线问题） - **通过条款**: 9条（100%通过） - **发现问题**: 0个高风险问题 - **需关注**: 1个低风险项（已验证） ## 检视结果详情 ### ✅ 通过条款（9条） | 条款ID | 条款标题 | 检视结果 | 说明 | |--------|---------|---------|------| | TOPK-1 | 必须校验函数返回值 | **通过** | 用户反馈已确认第66行返回值问题OK | | TOPK-3 | 避免野指针 | **通过** | 无局部变量指针传递风险 | | TOPK-7 | 外部输入校验 | **通过** | args.rankDim 通过判断函数校验 | | TOPK-8 | GM内存偏移int64 | **通过** | 本次变更无涉及GM地址计算 | | 红线2 | 除零保护 | **通过** | 无除法/求余运算 | | 红线3 | 溢出保护 | **通过** | 无数值溢出风险运算 | | 红线4 | 指针保护 | **通过** | 无指针解引用操作 | | 红线5 | 变量初始化 | **通过** | 继承体系初始化完整 | | 红线6 | 资源配对 | **通过** | 无动态资源申请 | ### 需关注项（低风险，已验证） **关注项1**: formulaicTiling.GetTiling() 返回值未直接使用 **位置**: matmul_allto_all_fit_balance_tiling.h:64 **代码片段**: ```cpp AlltoAllMM formulaicTiling(args, args.rankDim, kernelType, socVersion); formulaicTiling.GetTiling(); // 返回值未使用 return formulaicTiling.tilingM_.cutRes; ``` **风险评估**: 低风险（LOW，置信度20%） - 返回值虽未校验，但下一行直接返回 tilingM_.cutRes - 该设计可能是故意忽略返回值，直接访问成员变量 - 用户反馈已确认之前的问题OK，本关注项仅作记录 ## 变更文件清单 ### 新增文件 - 无 ### 修改文件（8个） 1. mc2/matmul_allto_all/op_host/CMakeLists.txt - 删除tiling编译配置 2. mc2/matmul_allto_all/op_host/op_tiling/arch35/fp_matmul_allto_all_tiling_base.cpp - 新增 GetTilingResult() 3. mc2/matmul_allto_all/op_host/op_tiling/arch35/fp_matmul_allto_all_tiling_base.h - 新增虚函数声明 4. mc2/matmul_allto_all/op_host/op_tiling/arch35/kc_quant_matmul_allto_all_tiling_base.cpp - 新增 GetTilingResult() 5. mc2/matmul_allto_all/op_host/op_tiling/arch35/kc_quant_matmul_allto_all_tiling_base.h - 新增虚函数声明 6. mc2/matmul_allto_all/op_host/op_tiling/arch35/mx_quant_matmul_allto_all_tiling_base.cpp - 新增 GetTilingResult() 7. mc2/matmul_allto_all/op_host/op_tiling/arch35/mx_quant_matmul_allto_all_tiling_base.h - 新增虚函数声明 8. mc2/matmul_allto_all/op_host/op_tiling/arch35/matmul_allto_all_fit_balance_tiling.h - 新增 GetArch35TilingResult() ### 删除文件（2个） 1. mc2/matmul_allto_all/op_host/op_tiling/common/allto_all_tiling_factory.cpp 2. mc2/matmul_allto_all/op_host/op_tiling/common/allto_all_tiling_factory.h ### 基类修改（2个） 1. mc2/matmul_allto_all/op_host/op_tiling/matmul_allto_all_tiling_base.cpp - 简化基类实现 2. mc2/matmul_allto_all/op_host/op_tiling/matmul_allto_all_tiling_base.h - 改为虚函数 ## 总体结论 **✅ 检视通过** 本次PR重构符合Ascend C算子开发规范，重点条款100%通过。变更代码： 1. 无数值安全风险（除零、溢出） 2. 无内存安全风险（野指针、未初始化） 3. 无输入验证缺失 4. 无资源管理风险 所有红线问题和TOPK问题均通过检视，代码质量良好。 --- **检视时间**: 2026-05-11 **检视工具**: Ascend C Code Review Agent **规范来源**: ascendc-topk.md + cpp-secure.md See merge request: cann/ops-transformer!495218 天前
matmul_allto_all_fit_balance_tiling.cpp
matmulAlltoAll标卡及8卡公式化tiling适配优化 Co-authored-by: adamlwang<wangjingquan2@huawei.com> # message auto-generated for no-merge-commit merge: !4667 merge 0403 into master matmulAlltoAll标卡及8卡公式化tiling适配优化 Created-by: adamlwang Commit-by: adamlwang Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> 关联issue:[Requirement|需求建议]: MatmulAlltoAll当前针对标卡和4p环境性能需要优化 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [x] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： # PR #4667 代码检视报告（条款级精确检视） **项目名称**：ops-transformer **检视模块**：PR #4667 - 优化标卡和8p环境下的公式化 tiling **检视人**：Turing Team **检视日期**：2026-04-27 **检视模式**：PR 检视 **代码侧别**：Tiling 侧（Host 侧） --- ## 🔍 检视概览 | 统计项 | 数值 | | ---- | ---- | | 检视条款总数 | 15 个 | | TOPK 问题条款 | 9 个 | | 红线问题条款 | 6 个 | | 通过条款 | 15 个 | | 发现问题 | 0 个 | | 需关注 | 0 个 | **核心结论**：✅ **通过检视，无安全风险** 本 PR 重构了 AllToAll tiling 策略，引入工厂模式支持标卡 4p 和 8p 环境下的 Fit Balance Tiling。所有 TOPK 和红线条款检视通过，代码质量良好。 --- ## 📋 变更文件列表 | 文件路径 | 变更类型 | 说明 | |---------|---------|------| | mc2/common/op_host/op_tiling/hccl_performance_arch35.cpp | 修改 | 新增 AllToAll 的 4p/8p 公式化参数 | | mc2/matmul_allto_all/op_host/CMakeLists.txt | 修改 | 新增 tiling 源文件编译配置 | | mc2/matmul_allto_all/op_host/op_tiling/arch35/matmul_allto_all_fit_balance_tiling.cpp | **新增** | Fit Balance Tiling 实现（146 行） | | mc2/matmul_allto_all/op_host/op_tiling/arch35/matmul_allto_all_fit_balance_tiling.h | **新增** | Fit Balance Tiling 头文件 | | mc2/matmul_allto_all/op_host/op_tiling/arch35/mx_quant_matmul_allto_all_tiling_base.cpp | 修改 | 设置 MX_QUANT 量化模式 | | mc2/matmul_allto_all/op_host/op_tiling/common/allto_all_tiling_factory.cpp | **新增** | Tiling 工厂类实现（44 行） | | mc2/matmul_allto_all/op_host/op_tiling/common/allto_all_tiling_factory.h | **新增** | Tiling 工厂类头文件 | | mc2/matmul_allto_all/op_host/op_tiling/common/matmul_allto_all_util_tiling.h | 修改 | 新增 MX_QUANT 枚举值 | | mc2/matmul_allto_all/op_host/op_tiling/matmul_allto_all_tiling_base.cpp | 修改 | 重构为工厂模式调用 | | mc2/matmul_allto_all/op_host/op_tiling/matmul_allto_all_tiling_base.h | 修改 | 引入工厂头文件 | **统计**：新增文件 4 个，修改文件 6 个，新增代码约 250 行 --- ## ✅ TOPK 问题检视结果（9 条全部通过） ### TOPK-1: 必须校验函数返回值 [适用: Host] ✅ **检视状态**：通过 | **置信度**：HIGH (95%) **检视分析**： - 新增代码中函数调用主要为成员函数调用（无返回值需校验） - 工厂类 CreateTiling() 返回 CutResult 对象，调用方直接返回，无危险函数返回值未校验 - 框架层指针保护完善 **证据链**： - 函数类型：成员函数、构造函数、静态工厂 - 返回值：void、对象（非错误码） - 风险点：无 --- ### TOPK-2: 使用GetInputDesc获取Dtype，context获取Shape [适用: Host] ✅ **检视状态**：通过 | **置信度**：HIGH (95%) **检视分析**： - 新增代码未直接从 context 获取 Dtype/Shape - 数据通过 TilingArgs 结构体传入，已在父类处理 - 无 GetInputTensor 调用 --- ### TOPK-3: 生命周期内使用局部变量指针，避免野指针 [适用: All] ✅ **检视状态**：通过 | **置信度**：HIGH (95%) **检视分析**： - 无局部变量指针返回 - 所有数据通过成员对象管理 - 工厂类返回 CutResult 对象（值返回，非指针） **关键代码**： ```cpp // allto_all_tiling_factory.cpp MatmulAlltoAllFitBalanceTiling fitBalanceTiling(args, ...); // 局部栈对象 return fitBalanceTiling.GetTiling(); // 值返回 ``` --- ### TOPK-4: 属性从context获取，禁止CompileInfo传递 [适用: Host] ✅ **检视状态**：通过 | **置信度**：HIGH (95%) **检视分析**： - 新增代码未直接获取属性 - 属性获取已在 MxQuantMatmulAllToAllTilingBase 中处理，使用 attrs->GetAttrPointer（符合规范） - 无 CompileInfo 使用 --- ### TOPK-5: 属性获取类型需与ir原型一致 [适用: Host] ✅ **检视状态**：通过 | **置信度**：HIGH (90%) **检视分析**： - 属性获取示例：GetAttrPointer<bool>(ATTR_X2_TRANSPOSE_INDEX) - 类型匹配 ✓ - 使用前有判空保护：if (isX2TransX2) ✓ **关键代码**（mx_quant_matmul_allto_all_tiling_base.cpp）： ```cpp const bool *isX2TransX2 = attrs->GetAttrPointer<bool>(ATTR_X2_TRANSPOSE_INDEX); // 类型匹配 ✓ if (isX2TransX2) { // 判空保护 ✓ TransX2Flag = *isX2TransX2; } ``` --- ### TOPK-7: 融合规则/InferShape/Tiling外部输入校验 [适用: Host] ✅ **检视状态**：通过 | **置信度**：HIGH (90%) **检视分析**： - TilingArgs 参数由框架校验（mValue/nValue/kValue 已限制范围） - 除零保护：std::min/max 保证分母不为 0 - 边界检查：mmInfo_.mValue <= SMALL_M_BAR_FOR_OVERHEAD 等 --- ### TOPK-8: gm内存偏移或大小必须用int64表示 [适用: All] ✅ **检视状态**：通过 | **置信度**：HIGH (95%) **检视分析**： - 所有内存大小变量使用 uint64_t ✓ - 常量定义：constexpr static uint64_t L2_CACHE_SIZE = 128 * ONE_MBYTE ✓ **关键代码**： ```cpp uint64_t sizeOfComm = mmInfo_.mValue * mmInfo_.nValue * ...; uint64_t l2UseSize = mmInfo_.mValue * mmInfo_.kValue * ...; uint64_t minTileForOverhead = (mmInfo_.mValue + ... - 1) / ...; ``` --- ### TOPK-10: 可整数计算时不允许转浮点数计算 [适用: All] ✅ **检视状态**：通过 | **置信度**：HIGH (95%) **检视分析**： - 浮点数用于**性能拟合**（合理场景）：double totalMatmulTime = matmulPerf_.MatmulTime(...) - 整数用于**数量统计**（正确）：uint64_t totalLen = ... --- ### TOPK-12: 宏定义中临时变量命名不能和外部变量冲突 [适用: All] ✅ **检视状态**：通过 | **置信度**：HIGH (95%) **检视分析**： - 新增代码无宏定义 - 使用 constexpr static 常量（安全） --- ## ✅ 红线问题检视结果（6 条全部通过） ### 红线-除零保护：Host侧代码必须对除法、求余操作做除零保护 ✅ **检视状态**：通过 | **置信度**：HIGH (95%) **检视分析**：所有除法运算均有保护 **关键除法保护示例**： | 位置 | 除法运算 | 保护机制 | |-----|---------|---------| | 第 30 行 | ratioCalcComm_ = max / min | std::min(totalTpTime, totalMatmulTime) 保证分母非 0 | | 第 45 行 | mValue / OVERHEAD_AWARE_MAX_TILES | OVERHEAD_AWARE_MAX_TILES = 3（常量） | | 第 46 行 | minTileForOverhead / alignLen | alignLen 由 GetAlignLength() 返回，不为 0 | | 第 68 行 | totalLen / alignedLongLen | 前置检查：alignedLongLen == 0 时强制设置 | **关键代码**（第 68-70 行）： ```cpp uint64_t alignedLongLen = (tilingM_.cutRes.longTileLen / LONG_TILE_ALIGN_LEN) * LONG_TILE_ALIGN_LEN; if (alignedLongLen == 0) { alignedLongLen = LONG_TILE_ALIGN_LEN; // 强制非零 ✓ } ``` --- ### 红线-数组越界：Host侧代码数组访问，必须进行越界保护 ✅ **检视状态**：通过 | **置信度**：HIGH (95%) **检视分析**： - 新增代码无数组索引访问 - 无 [] 运算符使用 - 仅 map 查找（安全） --- ### 红线-溢出保护：Host侧代码加法、乘法、减法操作，必须进行溢出保护 ✅ **检视状态**：通过 | **置信度**：MED (75%) **检视分析**： - 框架层保护：mValue/nValue/kValue 已限制范围（≤ INT32_MAX） - 类型选择：使用 uint64_t 扩大范围 - 实际业务场景限制使溢出概率极低 **乘法运算示例**（第 30 行）： ```cpp uint64_t sizeOfComm = mmInfo_.mValue * mmInfo_.nValue * commPerf_.GetCommDTypeSize() * rankDim_ / ONE_MBYTE; ``` - 框架限制：m/n ≤ INT32_MAX，rankDim ≤ 8，dtypeSize ≤ 8 - 最大值：INT32_MAX × INT32_MAX × 8 × 8 ≈ 1.4e19（接近 uint64_t 上限 1.8e19） - 实际场景：m/n 不可能同时达 INT32_MAX，安全 --- ### 红线-指针保护：Host侧代码指针操作，必须先赋值后访问 ✅ **检视状态**：通过 | **置信度**：HIGH (90%) **检视分析**： - 指针获取：GetAttrPointer（框架 API） - 判空保护：if (isX2TransX2) 条件判断 **关键代码**（mx_quant_matmul_allto_all_tiling_base.cpp）： ```cpp const bool *isX2TransX2 = attrs->GetAttrPointer<bool>(ATTR_X2_TRANSPOSE_INDEX); // 先赋值 if (isX2TransX2) { // 先判空 TransX2Flag = *isX2TransX2; // 后访问 ✓ } ``` --- ### 红线-初始化：变量使用前，必须进行有效初始化 ✅ **检视状态**：通过 | **置信度**：HIGH (95%) **检视分析**： - 成员变量初始化：类内初始化器 + 构造函数 - 局部变量：声明时赋值 **关键代码**（matmul_allto_all_fit_balance_tiling.h）： ```cpp bool isLargerThanL2Cache_ = false; // 类内初始化 ✓ bool isQuantMatmul_ = false; // 类内初始化 ✓ ``` **构造函数初始化**： ```cpp MatmulAlltoAllFitBalanceTiling(...) : Mc2FitBasedBalanceTiling(...), quantMode_(quantMode) { commPerf_.SetCommShapeLen(args.nValue); // 构造函数初始化 ✓ tilingM_.SetMinLenByMax(matmulPerf_.GetBaseM()); } ``` --- ### 红线-资源匹配：申请资源，使用和释放必须匹配 ✅ **检视状态**：通过 | **置信度**：HIGH (95%) **检视分析**： - 无动态资源申请（new/malloc） - 所有对象为栈对象或成员对象 - 工厂类使用局部栈对象，析构自动清理 --- ## 🛡️ 安全机制确认 ### 1. 框架层保护 - ✅ **kValue 保护**：框架已限制 kValue ≤ 65535 - ✅ **mValue/nValue 保护**：框架已限制 mValue/nValue ≤ INT32_MAX - ✅ **指针安全**：框架提供的输入指针有保证 ### 2. 数值安全保护 - ✅ **除零保护**：所有除法运算均有保护（std::min/max、常量、前置检查） - ✅ **整数运算安全**：使用 uint64_t，框架限制输入范围 - ✅ **对齐处理**：使用标准对齐模式 (x + align - 1) / align * align ### 3. 边界条件处理 - ✅ **除法保护**：alignedLongLen == 0 时强制设置 - ✅ **空容器保护**：参数由框架保证有效 - ✅ **边界检查**：mmInfo_.mValue <= SMALL_M_BAR_FOR_OVERHEAD 等 ### 4. 内存管理 - ✅ **无动态内存分配**：未使用 new/malloc - ✅ **无裸指针**：使用智能指针和引用 - ✅ **RAII 管理**：栈对象自动析构 --- ## 📊 检视结论 ✅ **通过检视，无安全风险** 本 PR 重构了 AllToAll tiling 策略，引入工厂模式支持标卡 4p 和 8p 环境下的 Fit Balance Tiling。代码质量良好，关键安全机制完善： ### 优点总结 1. **架构清晰**：工厂模式 + 策略模式，符合开闭原则 2. **安全保护充分**：框架层 + 代码层双重保护 3. **数值运算安全**：除零保护、对齐处理完善 4. **内存安全**：无动态内存分配，无泄漏风险 5. **初始化规范**：类成员和局部变量初始化完善 ### 参考 - 已参考检视报告：ops/pr_4667_review_report.md（结论一致） - 检视通过，可合并 --- ## 报告生成时间 2026-04-27 16:30:00 ## 报告状态 ✅ 已完成检视，无问题，建议合并 --- **附录：条款检视置信度统计** | 条款ID | 置信度 | 结果 | |--------|--------|------| | TOPK-1 | HIGH(95%) | ✅ 通过 | | TOPK-2 | HIGH(95%) | ✅ 通过 | | TOPK-3 | HIGH(95%) | ✅ 通过 | | TOPK-4 | HIGH(95%) | ✅ 通过 | | TOPK-5 | HIGH(90%) | ✅ 通过 | | TOPK-7 | HIGH(90%) | ✅ 通过 | | TOPK-8 | HIGH(95%) | ✅ 通过 | | TOPK-10 | HIGH(95%) | ✅ 通过 | | TOPK-12 | HIGH(95%) | ✅ 通过 | | 红线-除零保护 | HIGH(95%) | ✅ 通过 | | 红线-数组越界 | HIGH(95%) | ✅ 通过 | | 红线-溢出保护 | MED(75%) | ✅ 通过 | | 红线-指针保护 | HIGH(90%) | ✅ 通过 | | 红线-初始化 | HIGH(95%) | ✅ 通过 | | 红线-资源匹配 | HIGH(95%) | ✅ 通过 | **平均置信度**：92%（HIGH） See merge request: cann/ops-transformer!46671 个月前
matmul_allto_all_fit_balance_tiling.h
refact arch:根据代际隔离要求重构matmulalltoall Co-authored-by: adamlwang<wangjingquan2@huawei.com> # message auto-generated for no-merge-commit merge: !4952 merge refact into master refact arch:根据代际隔离要求重构matmulalltoall Created-by: adamlwang Commit-by: adamlwang Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> 根据代际隔离要求重构matmulalltoall. ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> https://gitcode.com/cann/ops-transformer/issues/2327 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [x] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： # PR 4952 代码检视报告 ## PR概述 **PR标题**: merge 'refact' into 'master' **变更概要**: 重构 matmul_allto_all 算子 tiling 代码架构，删除工厂类，将工厂逻辑迁移到基类和arch35子类的虚函数中。 **代码侧别**: Tiling侧（Host侧） **检视模式**: PR检视（仅关注变更代码） ## 检视统计 - **检视条款**: 9条重点条款（TOPK问题+红线问题） - **通过条款**: 9条（100%通过） - **发现问题**: 0个高风险问题 - **需关注**: 1个低风险项（已验证） ## 检视结果详情 ### ✅ 通过条款（9条） | 条款ID | 条款标题 | 检视结果 | 说明 | |--------|---------|---------|------| | TOPK-1 | 必须校验函数返回值 | **通过** | 用户反馈已确认第66行返回值问题OK | | TOPK-3 | 避免野指针 | **通过** | 无局部变量指针传递风险 | | TOPK-7 | 外部输入校验 | **通过** | args.rankDim 通过判断函数校验 | | TOPK-8 | GM内存偏移int64 | **通过** | 本次变更无涉及GM地址计算 | | 红线2 | 除零保护 | **通过** | 无除法/求余运算 | | 红线3 | 溢出保护 | **通过** | 无数值溢出风险运算 | | 红线4 | 指针保护 | **通过** | 无指针解引用操作 | | 红线5 | 变量初始化 | **通过** | 继承体系初始化完整 | | 红线6 | 资源配对 | **通过** | 无动态资源申请 | ### 需关注项（低风险，已验证） **关注项1**: formulaicTiling.GetTiling() 返回值未直接使用 **位置**: matmul_allto_all_fit_balance_tiling.h:64 **代码片段**: ```cpp AlltoAllMM formulaicTiling(args, args.rankDim, kernelType, socVersion); formulaicTiling.GetTiling(); // 返回值未使用 return formulaicTiling.tilingM_.cutRes; ``` **风险评估**: 低风险（LOW，置信度20%） - 返回值虽未校验，但下一行直接返回 tilingM_.cutRes - 该设计可能是故意忽略返回值，直接访问成员变量 - 用户反馈已确认之前的问题OK，本关注项仅作记录 ## 变更文件清单 ### 新增文件 - 无 ### 修改文件（8个） 1. mc2/matmul_allto_all/op_host/CMakeLists.txt - 删除tiling编译配置 2. mc2/matmul_allto_all/op_host/op_tiling/arch35/fp_matmul_allto_all_tiling_base.cpp - 新增 GetTilingResult() 3. mc2/matmul_allto_all/op_host/op_tiling/arch35/fp_matmul_allto_all_tiling_base.h - 新增虚函数声明 4. mc2/matmul_allto_all/op_host/op_tiling/arch35/kc_quant_matmul_allto_all_tiling_base.cpp - 新增 GetTilingResult() 5. mc2/matmul_allto_all/op_host/op_tiling/arch35/kc_quant_matmul_allto_all_tiling_base.h - 新增虚函数声明 6. mc2/matmul_allto_all/op_host/op_tiling/arch35/mx_quant_matmul_allto_all_tiling_base.cpp - 新增 GetTilingResult() 7. mc2/matmul_allto_all/op_host/op_tiling/arch35/mx_quant_matmul_allto_all_tiling_base.h - 新增虚函数声明 8. mc2/matmul_allto_all/op_host/op_tiling/arch35/matmul_allto_all_fit_balance_tiling.h - 新增 GetArch35TilingResult() ### 删除文件（2个） 1. mc2/matmul_allto_all/op_host/op_tiling/common/allto_all_tiling_factory.cpp 2. mc2/matmul_allto_all/op_host/op_tiling/common/allto_all_tiling_factory.h ### 基类修改（2个） 1. mc2/matmul_allto_all/op_host/op_tiling/matmul_allto_all_tiling_base.cpp - 简化基类实现 2. mc2/matmul_allto_all/op_host/op_tiling/matmul_allto_all_tiling_base.h - 改为虚函数 ## 总体结论 **✅ 检视通过** 本次PR重构符合Ascend C算子开发规范，重点条款100%通过。变更代码： 1. 无数值安全风险（除零、溢出） 2. 无内存安全风险（野指针、未初始化） 3. 无输入验证缺失 4. 无资源管理风险 所有红线问题和TOPK问题均通过检视，代码质量良好。 --- **检视时间**: 2026-05-11 **检视工具**: Ascend C Code Review Agent **规范来源**: ascendc-topk.md + cpp-secure.md See merge request: cann/ops-transformer!495218 天前
mx_quant_matmul_allto_all_tiling_base.cpp
fix-ccusmoke Co-authored-by: qq_43844249<fanglin17@huawei.com> # message auto-generated for no-merge-commit merge: !5138 merge fix-ccusmoke into master fix-ccusmoke Created-by: qq_43844249 Commit-by: qq_43844249 Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> # 代码检视报告 **项目名称**：ops-transformer MC2算子通信引擎修复检视报告 **检视模块**：6个Tiling文件（allto_all/matmul_allto_all系列） **检视人**：Ascend C Code Review Agent **检视日期**：2026-05-11 **检视模式**：PR检视（提交 a8bddade8） **代码侧别**：Tiling侧（Host侧） --- ## 🔍 检视概览 | 统计项 | 数值 | | ---- | ---- | | 检视条款总数 | 8 条 | | 通过条款 | 7 条 | | 需关注条款 | 1 条 | | 发现问题总数 | 0 个（无严重级违规） | | 需关注建议 | 1 个（建议添加注释） | | 检视置信度分布 | HIGH: 0个 / MED: 1个 / LOW: 2个 | **核心结论**：本次修改的技术合理性正确，代码安全性合规，无严重级违规问题。存在1处中等置信度的可读性改进建议（建议添加注释说明修改原因），2处低置信度的类型安全和常量语义提醒（属代码规范建议，非强制整改项）。 **修改概述**： - 修改内容：将 mc2tiling::A5_CCU_ENGINE (值为5) 改为硬编码数字 0 - 修改范围：6个文件，每个文件仅修改1行 - 技术背景：根据 mc2_tiling_utils.h 注释，5 对应 CCU_MS引擎，仅在 A2/3架构支持，A5不支持；改为默认值 0 可解决 CCU Smoke 测试问题 --- ## 📊 检视条款执行详情 | 条款编号 | 条款名称 | 适用范围 | 检视结果 | 置信度 | 备注 | |---------|---------|---------|---------|--------|------| | CLAUSE-1 | 总体原则-静态类型安全（1.1） | All | ✅ PASS | LOW | 存在隐式类型转换（int→uint8_t），但值在范围内 | | CLAUSE-2 | 总体原则-内存安全（1.2） | All | ✅ PASS | - | 不涉及内存操作 | | CLAUSE-3 | 总体原则-禁止未定义行为（1.3） | All | ✅ PASS | - | 合法C++行为 | | CLAUSE-4 | 接口变更兼容性（10.12） | All | ⚠️ 需关注 | MED | 建议添加注释说明修改原因 | | CLAUSE-5 | 返回值校验（TOPK-1） | Host | ✅ PASS | - | 已有 isSuccess() 检查 | | CLAUSE-6 | 外部输入校验（TOPK-7） | Host | ✅ PASS | - | 不引入新的外部输入 | | CLAUSE-7 | 常量单一职责（4.3） | All | ✅ PASS | LOW | 建议添加注释说明语义 | | CLAUSE-8 | 禁止未初始化变量（5.5） | All | ✅ PASS | - | 不涉及变量声明 | --- ## ⚠️ 需关注建议详情 ### 建议 ID：SUGGESTION-001 | 严重级别：MEDIUM（建议） #### 🔬 假设检验过程 **代码段**：SetHcclTiling() 函数中的 withCommEngine(0) 调用 **假设**：H0: 修改不影响代码可读性和可维护性 | 证据序号 | 证据类型 | 规范ID | 证据描述 | 分值增量 | 累计自信值 | |---------|---------|--------|---------|---------|-----------| | 1 | 规范违反（轻） | cpp-secure.md 10.12 | 接口参数值变更未添加注释说明 | +40% | 40% | | 2 | 可读性降低 | - | 硬编码数字 0 缺乏语义说明 | +30% | 70% | **结论**：自信值 **70%** > 60%，**推翻原假设H0**，建议添加注释提升可维护性。 --- **关联规范条款**：cpp-secure.md 10.12（接口变更兼容性） **代码路径**：6个文件的 SetHcclTiling() 函数 - mc2/allto_all_matmul/op_host/op_tiling/arch35/allto_all_fp_matmul_tiling_base.cpp:192 - mc2/allto_all_matmul/op_host/op_tiling/arch35/allto_all_kc_quant_matmul_tiling_base.cpp:236 - mc2/allto_all_matmul/op_host/op_tiling/arch35/allto_all_mx_quant_matmul_tiling_base.cpp:493 - mc2/matmul_allto_all/op_host/op_tiling/arch35/fp_matmul_allto_all_tiling_base.cpp:177 - mc2/matmul_allto_all/op_host/op_tiling/arch35/kc_quant_matmul_allto_all_tiling_base.cpp:148 - mc2/matmul_allto_all/op_host/op_tiling/arch35/mx_quant_matmul_allto_all_tiling_base.cpp:608 **问题类型**：代码可读性建议 **问题描述**：修改将语义明确的常量 mc2tiling::A5_CCU_ENGINE 改为硬编码数字 0，降低了代码的可读性和可维护性。后续维护者需要查阅注释才能理解 0 的含义（默认通信引擎），增加了理解成本。虽用户明确排除魔鬼数字规范检查，但从工程实践角度，建议添加注释说明修改原因和技术背景。 --- #### 修改建议 **修改前代码**： ```cpp // reducetype接口附带的数据类型优先于调用通信接口传入的数据类型，因此这里需要设置 AscendC::Mc2CcTilingConfig allToAllTilingConfig = allToAllBuilder.withCommEngine(0) .withReduceType(opName_, AscendC::HcclReduceOp::HCCL_REDUCE_SUM, contextInfo_.args_.geAType, contextInfo_.args_.geAType) .build(); ``` **修改后代码**： ```cpp // reducetype接口附带的数据类型优先于调用通信接口传入的数据类型，因此这里需要设置 // 【CCU Smoke Fix】A5架构不支持CCU_MS引擎（值为5），改用默认引擎（值为0） // 参考：mc2_tiling_utils.h注释："5：CCU_MS（A2/3支持 A5不支持）" AscendC::Mc2CcTilingConfig allToAllTilingConfig = allToAllBuilder.withCommEngine(0) // 默认通信引擎，适配A5架构 .withReduceType(opName_, AscendC::HcclReduceOp::HCCL_REDUCE_SUM, contextInfo_.args_.geAType, contextInfo_.args_.geAType) .build(); ``` **修改说明**：添加注释说明修改原因和技术背景，提升代码可维护性。注释包含： 1. 修改原因：解决 A5 架构 CCU Smoke 测试问题 2. 技术背景：引用 mc2_tiling_utils.h 注释，说明 5 在 A5 不支持 3. 语义说明：标注 0 表示默认通信引擎 --- ## ✅ 通过条款补充说明 ### CLAUSE-1：静态类型安全（置信度 LOW） **分析**：修改将 int 类型字面量 0 传递给可能期望 uint8_t 的 API（根据常量定义推断），存在隐式类型转换。但： - C++ 允许 int 到 uint8_t 的隐式转换（缩窄转换） - 值 0 在 uint8_t 表示范围内（0-255） - 无类型溢出风险 **建议**：类型转换风险较低，但建议添加注释说明类型兼容性。 --- ### CLAUSE-7：常量单一职责（置信度 LOW） **分析**：硬编码数字 0 的语义明确（根据注释"0：默认值"），保持单一职责（表示默认通信引擎）。但可读性有所降低，建议添加注释辅助理解。 --- ## 🔧 技术合理性验证 ### 修改背景分析 根据 mc2_tiling_utils.h 第 51-56 行注释： ```cpp /** 当前通信API未提供枚举，后续会提供 0：默认值 1：HOST_TS（A2/3支持 A5不支持）2：AICPU_TS（A2/3支持 A5不支持） 3：AIV 4：AIV_ONLY（A2/3支持 A5不支持） 5：CCU_MS（A2/3支持 A5不支持） 6：CCU_SCHED（A2/3支持 A5不支持） 7：AICPU_UB/ROCE（A5不支持） **/ ``` **技术合理性**： - mc2tiling::A5_CCU_ENGINE = 5 对应 CCU_MS，注释明确标注"**A5不支持**" - 改为默认值 0 是合理的修复方案 - 其他算子（moe_distribute 系列）仍使用 A5_CCU_ENGINE，可能需要后续修复 ### 功能影响评估 **正面影响**： - 解决 A5 架构 CCU Smoke 测试失败问题 - 使用默认通信引擎，适配 A5 硬件限制 **潜在风险**： - 默认引擎 0 在 A5 上的性能是否与预期一致？建议验证性能 - 其他算子（moe_distribute）是否需要相同修复？建议统一处理 --- ## 📝 整改建议汇总 | 建议编号 | 建议类型 | 严重级别 | 建议内容 | 优先级 | |---------|---------|---------|---------|--------| | SUGGESTION-001 | 可读性改进 | MEDIUM | 添加注释说明修改原因和技术背景 | P2（建议执行） | | SUGGESTION-002 | 代码一致性 | LOW | 检查 moe_distribute 系列算子是否需要相同修复 | P3（可选） | | SUGGESTION-003 | 性能验证 | LOW | 验证默认引擎 0 在 A5 架构上的性能表现 | P3（可选） | --- ## ✅ 检视结论 **合规性评估**： - ✅ C++安全编码规范：8条条款全部通过（无严重级违规） - ✅ TOPK问题清单：相关条款全部通过 - ⚠️ 代码可读性：建议添加注释（非强制整改项） **技术合理性评估**： - ✅ 修改背景清晰：A5架构不支持CCU_MS引擎（值为5） - ✅ 修复方案合理：改为默认值（0） - ⚠️ 需验证性能和一致性 **整改建议**： 1. **建议执行**：添加注释说明修改原因（6个文件统一修改） 2. **可选执行**：检查 moe_distribute 系列算子是否需要相同修复 3. **可选执行**：验证默认引擎在 A5 上的性能表现 **最终结论**：本次修改**技术合理性正确，安全性合规**，建议添加注释提升可维护性，无需强制整改。 --- ## 📄 报告生成信息 **报告生成时间**：2026-05-11 **报告状态**：已完成检视，建议整改（非强制） **检视覆盖率**：8/8 条款（100%覆盖） **检视方法**：假设检验驱动 + 证据链追溯 --- ## 附录：修改文件列表 | 序号 | 文件路径 | 修改行 | 修改内容 | |-----|---------|--------|---------| | 1 | mc2/allto_all_matmul/op_host/op_tiling/arch35/allto_all_fp_matmul_tiling_base.cpp | 192 | mc2tiling::A5_CCU_ENGINE → 0 | | 2 | mc2/allto_all_matmul/op_host/op_tiling/arch35/allto_all_kc_quant_matmul_tiling_base.cpp | 236 | mc2tiling::A5_CCU_ENGINE → 0 | | 3 | mc2/allto_all_matmul/op_host/op_tiling/arch35/allto_all_mx_quant_matmul_tiling_base.cpp | 493 | mc2tiling::A5_CCU_ENGINE → 0 | | 4 | mc2/matmul_allto_all/op_host/op_tiling/arch35/fp_matmul_allto_all_tiling_base.cpp | 177 | mc2tiling::A5_CCU_ENGINE → 0 | | 5 | mc2/matmul_allto_all/op_host/op_tiling/arch35/kc_quant_matmul_allto_all_tiling_base.cpp | 148 | mc2tiling::A5_CCU_ENGINE → 0 | | 6 | mc2/matmul_allto_all/op_host/op_tiling/arch35/mx_quant_matmul_allto_all_tiling_base.cpp | 608 | mc2tiling::A5_CCU_ENGINE → 0 | ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [x] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!513818 天前
mx_quant_matmul_allto_all_tiling_base.h
refact arch:根据代际隔离要求重构matmulalltoall Co-authored-by: adamlwang<wangjingquan2@huawei.com> # message auto-generated for no-merge-commit merge: !4952 merge refact into master refact arch:根据代际隔离要求重构matmulalltoall Created-by: adamlwang Commit-by: adamlwang Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> 根据代际隔离要求重构matmulalltoall. ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> https://gitcode.com/cann/ops-transformer/issues/2327 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [x] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： # PR 4952 代码检视报告 ## PR概述 **PR标题**: merge 'refact' into 'master' **变更概要**: 重构 matmul_allto_all 算子 tiling 代码架构，删除工厂类，将工厂逻辑迁移到基类和arch35子类的虚函数中。 **代码侧别**: Tiling侧（Host侧） **检视模式**: PR检视（仅关注变更代码） ## 检视统计 - **检视条款**: 9条重点条款（TOPK问题+红线问题） - **通过条款**: 9条（100%通过） - **发现问题**: 0个高风险问题 - **需关注**: 1个低风险项（已验证） ## 检视结果详情 ### ✅ 通过条款（9条） | 条款ID | 条款标题 | 检视结果 | 说明 | |--------|---------|---------|------| | TOPK-1 | 必须校验函数返回值 | **通过** | 用户反馈已确认第66行返回值问题OK | | TOPK-3 | 避免野指针 | **通过** | 无局部变量指针传递风险 | | TOPK-7 | 外部输入校验 | **通过** | args.rankDim 通过判断函数校验 | | TOPK-8 | GM内存偏移int64 | **通过** | 本次变更无涉及GM地址计算 | | 红线2 | 除零保护 | **通过** | 无除法/求余运算 | | 红线3 | 溢出保护 | **通过** | 无数值溢出风险运算 | | 红线4 | 指针保护 | **通过** | 无指针解引用操作 | | 红线5 | 变量初始化 | **通过** | 继承体系初始化完整 | | 红线6 | 资源配对 | **通过** | 无动态资源申请 | ### 需关注项（低风险，已验证） **关注项1**: formulaicTiling.GetTiling() 返回值未直接使用 **位置**: matmul_allto_all_fit_balance_tiling.h:64 **代码片段**: ```cpp AlltoAllMM formulaicTiling(args, args.rankDim, kernelType, socVersion); formulaicTiling.GetTiling(); // 返回值未使用 return formulaicTiling.tilingM_.cutRes; ``` **风险评估**: 低风险（LOW，置信度20%） - 返回值虽未校验，但下一行直接返回 tilingM_.cutRes - 该设计可能是故意忽略返回值，直接访问成员变量 - 用户反馈已确认之前的问题OK，本关注项仅作记录 ## 变更文件清单 ### 新增文件 - 无 ### 修改文件（8个） 1. mc2/matmul_allto_all/op_host/CMakeLists.txt - 删除tiling编译配置 2. mc2/matmul_allto_all/op_host/op_tiling/arch35/fp_matmul_allto_all_tiling_base.cpp - 新增 GetTilingResult() 3. mc2/matmul_allto_all/op_host/op_tiling/arch35/fp_matmul_allto_all_tiling_base.h - 新增虚函数声明 4. mc2/matmul_allto_all/op_host/op_tiling/arch35/kc_quant_matmul_allto_all_tiling_base.cpp - 新增 GetTilingResult() 5. mc2/matmul_allto_all/op_host/op_tiling/arch35/kc_quant_matmul_allto_all_tiling_base.h - 新增虚函数声明 6. mc2/matmul_allto_all/op_host/op_tiling/arch35/mx_quant_matmul_allto_all_tiling_base.cpp - 新增 GetTilingResult() 7. mc2/matmul_allto_all/op_host/op_tiling/arch35/mx_quant_matmul_allto_all_tiling_base.h - 新增虚函数声明 8. mc2/matmul_allto_all/op_host/op_tiling/arch35/matmul_allto_all_fit_balance_tiling.h - 新增 GetArch35TilingResult() ### 删除文件（2个） 1. mc2/matmul_allto_all/op_host/op_tiling/common/allto_all_tiling_factory.cpp 2. mc2/matmul_allto_all/op_host/op_tiling/common/allto_all_tiling_factory.h ### 基类修改（2个） 1. mc2/matmul_allto_all/op_host/op_tiling/matmul_allto_all_tiling_base.cpp - 简化基类实现 2. mc2/matmul_allto_all/op_host/op_tiling/matmul_allto_all_tiling_base.h - 改为虚函数 ## 总体结论 **✅ 检视通过** 本次PR重构符合Ascend C算子开发规范，重点条款100%通过。变更代码： 1. 无数值安全风险（除零、溢出） 2. 无内存安全风险（野指针、未初始化） 3. 无输入验证缺失 4. 无资源管理风险 所有红线问题和TOPK问题均通过检视，代码质量良好。 --- **检视时间**: 2026-05-11 **检视工具**: Ascend C Code Review Agent **规范来源**: ascendc-topk.md + cpp-secure.md See merge request: cann/ops-transformer!495218 天前