已合并
Revert "feat: support A5 UB-to-L1 ND-to-NZ with A2A3 path guards" #1741
zhywang创建于 28 天前
Revert "feat: support A5 UB-to-L1 ND-to-NZ with A2A3 path guards" #1741
已合并
zhywang创建于 28 天前
zhywang
28 天前

This reverts commit 7e58ace4b2d44f792486199f64118e5096bf03ba.

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 zhywang 的贡献)
Zzhywang
28 天前 创建了 pull request,commit 2cc76ac1
atomgit-bot
atomgit-bot
28 天前 评论:

变更摘要

本 PR 回退提交 7e58ace4b2d44f792486199f64118e5096bf03ba,撤销 A5 上「UB 到 L1 的 ND-to-NZ」搬运能力以及配套的 A2A3 路径守卫。核心是删除 include/pto/npu/a5/TExtractCommon.hpp 中的 TCopyNdToNzUbToMat 模板实现,并让 TEXTRACT、TMOV、TINSERT 在 A5 上重新统一走 TExtractVecToMat 路径;同时在 A2A3 侧恢复原有的 static_assert 校验分支,并移除随之引入的一整套测试用例与辅助头文件。

主要改动

  • 移除 UB-to-L1 ND-to-NZ 核心实现: 从 include/pto/npu/a5/TExtractCommon.hpp 删除 #if defined(PTO_NPU_ARCH_A5) 包裹的 TCopyNdToNzUbToMat 函数,该函数此前负责 ND Vec 源到非紧凑 NZ512 Mat 目标的 copy_ubuf_to_cbuf 分块搬运。
  • A5 三个算子回退到原路径: TEXTRACT_TILE_IMPL(include/pto/npu/a5/TExtract.hpp)、TMOV_TILE_IMPL(include/pto/npu/a5/TMov.hpp)和 TInsertVecToMatImpl(include/pto/npu/a5/TInsert.hpp)均删除 ND-to-NZ 分支及相关 PTO_ASSERT 形状检查,统一改回调用 TExtractVecToMat;TInsert.hpp 同时移除对 TExtractCommon.hpp 的条件包含。
  • 恢复 A2A3 路径守卫: include/pto/npu/a2a3/TExtract.hpp 重新加入 static_assert(sizeof(DstTileData) == 0, ...) 的不支持组合报错,并将 Vec->Vec 的 ND 分派条件恢复为同时要求 SFractal == SLayout::NoneBox,其余情况改用 isRowMajor 与 SFractal 匹配的 static_assert 提示。
  • 删除 UB2L1 ND2NZ 测试支撑文件: 移除 tests/npu/a5/src/st/testcase/tmov_ub2l1/ 下的 ub2l1_nd2nz_kernel.h、ub2l1_nd2nz_test.h、ub2l1_nd2nz_types.h 及 nd2nz_cases.h,并删除各测试目录下同名的 nd2nz_cases.h 与 kernel 中的 LAUNCH_ND2NZ/TEST_ND2NZ 宏展开。
  • 清理测试用例与设备参数处理: tests/npu/a5/src/st/testcase/textract/main.cpp、tinsert/main.cpp、tmov_ub2l1/main.cpp 移除对应的 ND-to-NZ 测试与引入头文件,tmov_ub2l1 恢复为固定 aclrtSetDevice(0);tests/costmodel/st/testcase/textract/main.cpp 重写为基于 EXPECT_CYCLE_NEAR 的 Mat 类型用例,并删除 tests/cpu/st/testcase/tmov/main.cpp 中的 testUbToL1Nd2Nz 系列测试。
likedislike
不准确?
atomgit-bot
atomgit-bot
28 天前 评论:

🤖 AI 代码检视正在进行中,请稍候…

likedislike
不准确?
CANN-robotCANN-robot成员
28 天前 添加了label:cann-cla/yes
zhywang
28 天前 评论:

/compile

likedislike
Zzhywang
28 天前 预合并成功(commit_id: 14335608a98c8ab68b37603156b35162b359e3d4)
CANN-robot
CANN-robot成员
28 天前 评论:

Thanks for your pull-request.
The full list of commands accepted by me can be found at here.
You can get sig-info at here.
You can self-configure the PR merge rules for this repository. For more details, please refer to Here.
For more, you also can visit HICANN.


PR Approval Progress

✅ Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
docs ✅ zhangjian_hz11, Zhanghaijian (2/2) ✅ zhangjian_hz11 (1/1)
repo-cann/pto-isa ✅ zhangjian_hz11, Zhanghaijian (2/2) ✅ zhangjian_hz11, Zhanghaijian (2/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

zhhywang, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
CANN-robotCANN-robot成员
28 天前 将zhangjian_hz11,csjlchen,zhezhou,bluesky901,changxianyu,Zhanghaijian,syfeng,shengxinchen,yinjie27_,zhoubotcam,HuangXingYuan_777,linjiashu,WenboCodes,qukelin,ChanKaLok,kerwinair,echo_stone设为评审人
CANN-robotCANN-robot成员
28 天前 将zhangjian_hz11,csjlchen,zhezhou,bluesky901,changxianyu,Zhanghaijian,syfeng,shengxinchen,yinjie27_,zhoubotcam,HuangXingYuan_777,linjiashu,WenboCodes,qukelin,ChanKaLok,kerwinair,echo_stone设为审查人
CANN-robot
CANN-robot成员
28 天前 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
28 天前 添加了label:ci-pipeline-running
zhangjian_hz11成员
28 天前 评论:

/approve

likedislike
Zhanghaijian成员
28 天前 评论:

/approve

likedislike
CANN-robotCANN-robot成员
28 天前 添加了label:approved
Zhanghaijian成员
28 天前 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
28 天前 添加了label:lgtm
ChanKaLok成员
28 天前 评论:

/approve

likedislike
CANN-robotCANN-robot成员
28 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
28 天前 添加了label:ci-pipeline-passed
CANN-robotCANN-robot成员
28 天前 合入了pull request