master

cann-robot整改mc2部分仓内重名头文件现象

3331d84e创建于 8 天前历史提交

文件	最后提交记录	最后更新时间
all_gather_matmul_v2	整改mc2部分仓内重名头文件现象 Co-authored-by: chenyifan<chenyifan66@h-partners.com> # message auto-generated for no-merge-commit merge: !5637 merge clear_repeat_h into master 整改mc2部分仓内重名头文件现象 Created-by: mutex_lock Commit-by: chenyifan Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> - 整改mc2部分仓内重名头文件现象 --- | 文件名 | 源码路径|处理方式 | |-----|-----|-----| | all_gather_formulaic_tiling.h | mc2/all_gather_matmul/op_host/op_tiling/all_gather_formulaic_tiling.h,mc2/all_gather_matmul_v2/op_host/op_tiling/all_gather_formulaic_tiling.h | 都为内部头文件，但内容不同，已修改all_gather_matmul_v2算子内文件的命名 | | grouped_matmul_host_util.h | gmm/grouped_matmul/op_host/grouped_matmul_host_util.h,mc2/3rd/grouped_matmul/op_tiling/grouped_matmul_host_util.h | 都为内部头文件，但内容不完全相同，已修改mc2/3rd内文件的命名 | | grouped_matmul_tiling.h | gmm/grouped_matmul/op_host/op_tiling/grouped_matmul_tiling.h,mc2/3rd/grouped_matmul/op_tiling/grouped_matmul_tiling.h | 都为内部头文件，但内容不完全相同，已修改mc2/3rd内文件的命名 | | matmul_util.h | mc2/3rd/common/op_host/op_api/matmul_util.h,mc2/common/utils/matmul_util.h | 都为内部头文件，但内容不完全相同，已修改mc2/3rd内文件的命名，并消除宏冲突 | | runtime_kb_api.h | common/stub/op_tiling/runtime_kb_api.h,mc2/3rd/ops_legacy/op_tiling/runtime_kb_api.h | 声明完全相同，但实现侧不同，删除mc2/3rd/ops_legacy下的runtime_kb_api.h | | tiling_type.h | common/include/op_host/tiling_type.h,mc2/3rd/common/op_host/op_tiling/tiling_type.h |都为内部文件，但内容不完全相同，保留common/include/op_host/tiling_type.h统一使用 | | tuning_tiling_reflection_utils.h | common/stub/op_tiling/register/tuning_tiling_reflection_utils.h,mc2/3rd/ops_legacy/op_tiling/register/tuning_tiling_reflection_utils.h | 声明完全相同，删除mc2/3rd/ops_legacy/op_tiling/register/tuning_tiling_reflection_utils.h | | tuning_tiling_registry.h | common/stub/op_tiling/register/tuning_tiling_registry.h,mc2/3rd/ops_legacy/op_tiling/register/tuning_tiling_registry.h | 声明完全相同，删除mc2/3rd/ops_legacy/op_tiling/register/tuning_tiling_registry.h | --- ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> https://gitcode.com/cann/ops-transformer/issues/2550 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> rdv ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [x] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!5637	8 天前
common	D&C Fix Communication Engine Index Co-authored-by: gitcode_lijd<lijiandong20@huawei.com> # message auto-generated for no-merge-commit merge: !5261 merge fix_2 into master D&C Fix Communication Engine Index Created-by: gitcode_lijd Commit-by: gitcode_lijd Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> ## 关联的Issue https://gitcode.com/cann/ops-transformer/issues/2385 <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> **检视文件**：mc2_tiling_utils.h **修改内容**：修改 A5_CCU_ENGINE 常量值（5 → 0） **规范来源**：cpp-secure.md、cpp-general.md --- ## 修改内容说明 ### 本次修改的目的 将 A5 芯片的通信引擎配置从 CCU_MS（值5）改为默认值（值0）。 ### 原方案 **常量定义**： ```cpp constexpr uint8_t A5_CCU_ENGINE = 5; // CCU_MS引擎类型 ``` **注释说明**： ```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不支持） **/ ``` ### 新方案 **修改点**： ```cpp constexpr uint8_t A5_CCU_ENGINE = 0; // 改为COMM_ALG_DEFAULT ``` **使用场景**（arch35目录下两个算子）： - moe_distribute_dispatch_v2_tiling_arch35.cpp:845 - moe_distribute_combine_tiling_arch35.cpp:679 ```cpp mc2CcTilingConfig.SetCommEngine(mc2tiling::A5_CCU_ENGINE); ``` --- ## 问题列表 | 序号 | 级别 | 问题描述 | 行号 | 规范条款 | |-----|------|---------|-----|---------| | 1 | 中 | 常量命名与实际值语义不符 | 10,23 | 规范4.2 | | 2 | 低 | 缺少修改说明和原因 | 全文件 | 规范1.3 | --- ## 问题详情 ### 1. 常量命名与实际值语义不符 [中风险] **假设检验过程**： - **H0**：修改后常量语义清晰，不影响理解 - **H1**：常量名称与值不匹配，可能导致误解 - **自信值初始化**：0% **证据收集**： 1. **规范违反** (+40%)： - 规范 4.2（cpp-general.md）：禁止使用魔鬼数字/字符串，常量命名应体现语义 - 原定义：A5_CCU_ENGINE = 5，名称明确表示"CCU引擎"，值5对应注释中的"CCU_MS" - 新定义：A5_CCU_ENGINE = 0，名称仍为"CCU引擎"，但值0是"默认值"（COMM_ALG_DEFAULT） - 命名与值语义不符：A5_CCU_ENGINE 暗示CCU引擎，但值0是默认值 2. **上下文防御缺失** (+30%)： - 未修改注释说明，注释仍显示"5：CCU_MS" - 新值为0，但注释中0的含义是"默认值"，不是CCU引擎 3. **数据流追踪风险** (+25%)： - 使用场景在arch35目录（A5芯片相关），调用 SetCommEngine(A5_CCU_ENGINE) - 如果A5不支持CCU_MS（值5），改为默认值（值0）是合理的 - 但常量名称应改为 A5_DEFAULT_ENGINE 或类似命名 **自信值**：95%（超过60%，判定为中风险） **风险代码**： ```cpp // 5159.diff 第10、23行 -constexpr uint8_t A5_CCU_ENGINE = 5; +constexpr uint8_t A5_CCU_ENGINE = 0; ``` **建议修复方案**： **方案A**：修改常量名称以匹配语义 ```cpp // 如果A5不支持CCU，使用默认引擎 constexpr uint8_t A5_DEFAULT_ENGINE = 0; // 或 A5_COMM_DEFAULT ``` **方案B**：保留名称，但更新注释说明 ```cpp /** 当前通信API未提供枚举，后续会提供 0：默认值（A5使用） 1：HOST_TS（A2/3支持） 2：AICPU_TS（A2/3支持） 3：AIV 4：AIV_ONLY（A2/3支持） 5：CCU_MS（A2/3支持，A5不支持） 6：CCU_SCHED（A2/3支持） 7：AICPU_UB/ROCE（A5不支持） **/ constexpr uint8_t A5_CCU_ENGINE = 0; // A5使用默认引擎，不支持CCU ``` --- ### 2. 缺少修改说明和原因 [低风险] **假设检验过程**： - **H0**：修改有明确的设计意图 - **H1**：修改原因未说明，影响可维护性 - **自信值初始化**：0% **证据收集**： 1. **规范违反** (+40%)： - 规范 1.3（cpp-general.md）：删除无效、冗余或永不执行的代码需说明原因 - 修改常量值属于重要配置变更，应说明修改原因 2. **上下文防御缺失** (+30%)： - diff 中仅修改值，无commit message或注释说明原因 - 从注释"CCU_MS（A2/3支持 A5不支持）"推断：可能因A5不支持CCU而改为默认值 3. **数据流追踪风险** (+25%)： - 修改影响通信引擎配置，可能影响性能和功能 - 应说明：为何A5不能用CCU_MS、改为0后的影响 **自信值**：95%（超过60%，判定为低风险） **建议修复方案**： - 在commit message中说明："A5芯片不支持CCU_MS引擎，改为使用默认通信引擎配置" - 或在代码注释中补充说明 --- ## 总结 **修改合理性分析**： - 从注释"A5不支持CCU_MS"推断，修改可能合理（A5不支持值5，改用默认值0） - 但存在命名和语义不符问题 **建议**： 1. 修改常量名称为 A5_DEFAULT_ENGINE 或更新注释说明语义变更 2. 补充修改原因说明，提高可维护性 --- See merge request: cann/ops-transformer!5261	16 天前
quant_all_reduce	doc Tools工具扫描问题修改 Co-authored-by: gitee-yanglulu<yanglulul@h-partners.com> # message auto-generated for no-merge-commit merge: !3432 merge master into master doc Tools工具扫描问题修改 Created-by: gitee-yanglulu Commit-by: gitee-yanglulu Merged-by: cann-robot Description: doc Tools工具扫描问题修改 See merge request: cann/ops-transformer!3432	2 个月前
CMakeLists.txt	更新license Co-authored-by: PerrySkywalker<wangmingkang1@huawei.com> # message auto-generated for no-merge-commit merge: !568 merge lic into master 更新license Created-by: PerrySkywalker Commit-by: PerrySkywalker Merged-by: cann-robot Description: ## 描述 更新license <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-transformer!568	5 个月前
通算融合算子Experimental使用说明.md	修复mc2的一些文档中出现的断链问题 Co-authored-by: hblnb<huangbolin3@huawei.com> # message auto-generated for no-merge-commit merge: !3993 merge fix_md_link into master 修复mc2的一些文档中出现的断链问题 Created-by: hblnb Commit-by: hblnb Merged-by: cann-robot Description: ## 描述 解决mc2算子文档中的一些断链问题，更新了文档中不存在的链接 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [x] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!3993	1 个月前
通算融合自定义算子工程开发指南.md	修复mc2的一些文档中出现的断链问题 Co-authored-by: hblnb<huangbolin3@huawei.com> # message auto-generated for no-merge-commit merge: !3993 merge fix_md_link into master 修复mc2的一些文档中出现的断链问题 Created-by: hblnb Commit-by: hblnb Merged-by: cann-robot Description: ## 描述 解决mc2算子文档中的一些断链问题，更新了文档中不存在的链接 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [x] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!3993	1 个月前