master

cann-robotfix(kernel-template): 修复 TQue 模板第二参数 depth 被误用为 BUFFER_NUM 的问题

621923cf创建于 7月10日历史提交

文件	最后提交记录	最后更新时间
templates	fix(kernel-template): 修复 TQue 模板第二参数 depth 被误用为 BUFFER_NUM 的问题 Co-authored-by: jiangyixuan2<124090275@link.cuhk.edu.cn> # message auto-generated for no-merge-commit merge: !579 merge fix/tque-depth-parameter into master fix(kernel-template): 修复 TQue 模板第二参数 depth 被误用为 BUFFER_NUM 的问题 Created-by: jiangyixuan2 Commit-by: jiangyixuan2 Merged-by: cann-robot Description: ## 描述 修复仓库内 12 个文件中 TQue<TPosition, depth> 第二模板参数被误用为 BUFFER_NUM / 字面量 2 的问题。TQue 的第二参数是队列深度 depth，与 Double Buffer 无关；Double Buffer 应由 InitBuffer(que, num, size) 的 num 参数控制。 将所有 TQue<..., 2> / TQue<..., BUFFER_NUM> 统一改为 TQue<..., 1>，InitBuffer 的 num 参数保持不变。 ## 涉及文件（12 个） **算子模板 / 示例代码（7 个）** - ops/ascendc-registry-invoke-template/references/add_example/op_kernel/arch35/add_example.h — TQue<..., BUFFER_NUM> → TQue<..., 1> - ops/ascendc-registry-invoke-template/references/add_example/op_kernel/arch22/add_example.h — 同上 - plugins-community/tilelang2ascendc-ops-generator/skills/.../helloworld/op_kernel/kernel_helloworld.cpp — 同上 - plugins-community/tilelang2ascendc-ops-generator/workflows/templates/archive_tasks/rms_norm/kernel/op_kernel/rms_norm.cpp — 3 个 kernel 类共 10 处 TQue<..., BUFFER_NUM> → TQue<..., 1> - plugins-community/tilelang2ascendc-ops-generator/workflows/templates/archive_tasks/flash_attention/kernel/op_kernel/flash_attention_cube.h — TQue<A2/B2, 2> → TQue<A2/B2, 1> - plugins-community/tilelang2ascendc-ops-generator/workflows/templates/archive_tasks/flash_attention/kernel/op_kernel/flash_attention_vec.h — TQue<VECIN, 2> → TQue<VECIN, 1> **文档 / Agent 指令（5 个）** - ops/ascendc-performance-best-practices/reference/sort/radix_sort.md — depth=2→1，prolog 模式简化为标准三段流水线，修正表格"double buffer 唯一正确方式"表述，"Steady 阶段"引用修正 - ops/ascendc-regbase-best-practice/references/regbase_development_guide.md — TQue<..., 2> → TQue<..., 1> - ops/ops-simulator/references/performance-issues-aiv.md — TQue<..., 2> → TQue<..., 1>，修正注释从 "bufNum = 2" 改为 "depth=1, double buffer via InitBuffer num=2" - plugins-official/ops-direct-invoke/workflows/development-guide.md — 双缓冲 | BUFFER_NUM = 2 → 双缓冲 | TQue<VECIN/VECOUT, 1>（depth=1），Double Buffer 由 InitBuffer(que, 2, size) 的 num 参数控制 - plugins-official/ops-direct-invoke/agents/ascendc-kernel-reviewer.md — 审查标准修正：使用 TQue + BUFFER_NUM=2 → 使用 TQue<..., 1> + InitBuffer(que, 2, size) - ops/ascendc-direct-invoke-template/references/kirin_add_template/README.md — 通过队列深度为2实现流水线 → 通过 InitBuffer(que, 2, size) 开启双缓冲实现流水线 ## 修改理由 TQue<TPosition, depth> 第二参数是队列深度（允许连续 EnQue 的次数），与 Double Buffer 无关。Double Buffer 由 InitBuffer(que, num, size) 的 num 参数控制。 原代码将 BUFFER_NUM(2) 填入 depth 位置，导致： 1. Agent 生成代码时普遍产出 TQue<..., 2>，误以为这是开启双缓冲的写法 2. 与 ops/ascendc-api-best-practices/references/api-buffer.md 中正确的 API 文档矛盾（该文档已明确 depth 与 num 独立），但 Agent 优先模仿示例，示例与文档矛盾导致错误扩散 ## 参考官方文档 https://www.hiascend.com/document/detail/zh/CANNCommunityEdition/80RC3alpha003/apiref/opdevgapi/atlasascendc_api_07_0131.html See merge request: cann/cannbot-skills!579	1 个月前
development-guide.md	refactor: tilelang2ascendc-ops-generator插件重构（含 tilelangtoascendc bug 修复） Co-authored-by: chenshushu2020<chenshushu@huawei.com> # message auto-generated for no-merge-commit merge: !546 merge br_tilelang_rename into master refactor: tilelang2ascendc-ops-generator插件重构（含 tilelangtoascendc bug 修复） Created-by: chenshushu2020 Commit-by: chenshushu2020 Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述您的改动，包括改动的原因和所采取的方法。 ### 举例： #### 新特性 New Features - 【仿真 ops-simulator】新增Ascend 950 仿真Skill：支持 Ascend 950 仿真，并且输出性能分析报告和流水线图。 - 【PyPTO】【pypto-op-design】新增 Skill：pypto-op-design，PyPTO算子方案设计，含快速参考和设计模板。 #### 问题修复 Bug Fix 【算子直调 ops-direct-invoke】修改verify_environment.sh脚本，返回environment.json 固定为1的bug，应该按实际设备的npu count返回。 --> ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单的链接。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🌟 新特性 - [ ] ✨ 特性增强 - [ ] 🐛 Bug 修复 - [ ] ⚡ 性能优化 - [ ] 🧪 测试用例 - [ ] 📦 构建 / CI - [ ] 📝 文档更新 - [ ] 🔧 配置变更 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] ♻️ 代码重构 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： ## 流水线故障排查 <!-- 如果下方的流水线检查未通过，可参考以下指引进行自助修复 --> > **UT_Test 未通过？** > 请在下方流水线表格的 **UT_Test** 行，点击右侧的 >>>>> 下载 **CANNBot-skills UT 测试报告**（HTML 格式）。 > 打开报告后，展开页面中的「UT Test 失败修复指南」，复制其中的提示词并粘贴给 Agent，Agent 将根据日志自动定位并修复问题。 > > 示例： >  See merge request: cann/cannbot-skills!546	1 个月前
task-prompts.md	refactor: tilelang2ascendc-ops-generator插件重构（含 tilelangtoascendc bug 修复） Co-authored-by: chenshushu2020<chenshushu@huawei.com> # message auto-generated for no-merge-commit merge: !546 merge br_tilelang_rename into master refactor: tilelang2ascendc-ops-generator插件重构（含 tilelangtoascendc bug 修复） Created-by: chenshushu2020 Commit-by: chenshushu2020 Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述您的改动，包括改动的原因和所采取的方法。 ### 举例： #### 新特性 New Features - 【仿真 ops-simulator】新增Ascend 950 仿真Skill：支持 Ascend 950 仿真，并且输出性能分析报告和流水线图。 - 【PyPTO】【pypto-op-design】新增 Skill：pypto-op-design，PyPTO算子方案设计，含快速参考和设计模板。 #### 问题修复 Bug Fix 【算子直调 ops-direct-invoke】修改verify_environment.sh脚本，返回environment.json 固定为1的bug，应该按实际设备的npu count返回。 --> ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单的链接。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🌟 新特性 - [ ] ✨ 特性增强 - [ ] 🐛 Bug 修复 - [ ] ⚡ 性能优化 - [ ] 🧪 测试用例 - [ ] 📦 构建 / CI - [ ] 📝 文档更新 - [ ] 🔧 配置变更 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] ♻️ 代码重构 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： ## 流水线故障排查 <!-- 如果下方的流水线检查未通过，可参考以下指引进行自助修复 --> > **UT_Test 未通过？** > 请在下方流水线表格的 **UT_Test** 行，点击右侧的 >>>>> 下载 **CANNBot-skills UT 测试报告**（HTML 格式）。 > 打开报告后，展开页面中的「UT Test 失败修复指南」，复制其中的提示词并粘贴给 Agent，Agent 将根据日志自动定位并修复问题。 > > 示例： >  See merge request: cann/cannbot-skills!546	1 个月前