已合并
针对nn仓部分算子存在错误日志问题进行统一整改 #9275
gaomingyuan创建于 22 天前
针对nn仓部分算子存在错误日志问题进行统一整改 #9275
已合并
gaomingyuan创建于 22 天前
gaomingyuan
gaomingyuan
22 天前

描述

本 PR 针对 nn 仓 activation 目录下多个算子的 tiling 实现进行统一整改,核心目的是修正错误日志问题:一方面删除大量冗余的 OP_LOGD 调试日志(如 ... enter.、Enter new XxxTiling 等),另一方面在 elewiseBaseTiling/BroadcastBaseTiling 等关键失败分支中补充数据类型或 opKey 等上下文信息,使错误日志更具可诊断性。涉及 LeakyReluTiling、PreluTiling、ReluTiling、ReluV2Tiling、SigmoidTiling、SwiGluBaseTiling4RegBase、SwishTiling、SwishGradTiling 等 tiling 类。

关联的Issue

关联Issue#5183

测试

本地已完成新精度对比与性能测试,通过静态检查、蓝区冒烟、算子ST以及算子二级冒烟。

文档更新

类型标签

AI/Agent生成声明

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 gaomingyuan 的贡献)
gaomingyuangaomingyuan
22 天前 创建了 pull request,commit acf8f1e2
atomgit-bot
atomgit-bot
22 天前 评论:

变更摘要

本 PR 针对 nn 仓 activation 目录下多个算子的 tiling 实现进行统一整改,核心目的是修正错误日志问题:一方面删除大量冗余的 OP_LOGD 调试日志(如 ... enter.Enter new XxxTiling 等),另一方面在 elewiseBaseTiling/BroadcastBaseTiling 等关键失败分支中补充数据类型或 opKey 等上下文信息,使错误日志更具可诊断性。涉及 LeakyReluTilingPreluTilingReluTilingReluV2TilingSigmoidTilingSwiGluBaseTiling4RegBaseSwishTilingSwishGradTiling 等 tiling 类。

主要改动

  • 移除冗余调试日志: 在 leaky_relup_relurelurelu_v2swishswish_gradswi_glu_grad 等多个算子 tiling 文件中删除 OP_LOGD 形式的入口/过程调试日志(如 CalcInputDtype enter.Enter new ReluTiling 等),减少非必要日志输出。
  • 失败日志补充数据类型信息: 在 elewiseBaseTiling failedBroadcastBaseTiling do tiling failed 等错误日志中通过 ge::TypeUtils::DataTypeToSerialString() 追加输出 dtype(如 LeakyReluTilingReluTilingReluV2TilingSwishTilingSwishGradTiling)或输入 dtype(如 LeakyReluGradTilingPreluTilingReluGradTiling),提升排障信息量。
  • 失败日志补充 opKey: SigmoidTilingSigmoidGradTilingelewise tiling failed 以及 SiluGradTilingbroadcast tiling failed 错误日志中追加 opKey: %lu. 字段,便于定位具体算子实例。
  • 规范化文件收尾: leaky_relu_gradp_relureluswish_grad 等文件末尾补充缺失的换行符,统一文件格式。
likedislike
不准确?
atomgit-bot
atomgit-bot
22 天前 评论:

代码审查

✅ 未发现问题

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

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
activation 王星, 汤平川 (2/2) 汤平川, 王星 (2/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
22 天前 将zengjuan,chaotang233,kevin_huang1234,crystalhu,yangyang016,fanqirui,wangrui_,renruhai,su-yueming,zhajianqing123,chenqi317,wang-xing001,liubo75,tangweiwei2,liujie12345678,pingchuantang,Chen_HaoWen,qianzehong设为评审人
CANN-robotCANN-robot成员
22 天前 将kevin_huang1234,wangrui_,renruhai,su-yueming,zhajianqing123,wang-xing001,liujie12345678,pingchuantang,Chen_HaoWen,qianzehong设为审查人
gaomingyuan
gaomingyuan
22 天前 评论:

compile

likedislike
gaomingyuangaomingyuan
22 天前 预合并成功(commit_id: d85a7569d24a207f81d0679fa942f829ba072fcc)
gaomingyuangaomingyuan
22 天前 修改了pull request 的描述
CANN-robotCANN-robot成员
22 天前 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
22 天前 评论:

流水线任务触发成功
任务链接 [38c07d1aa3994c9e9ea05008ae2e4da4][流水线指导]

任务名称状态日志下载链接
Compile_Ascend_X86_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_mobile_station ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_single ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_experimental ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_950 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_950 ✅ SUCCESS >>>>> >>>>>
Compile_Pre ✅ SUCCESS >>>>>
pre_comment ✅ SUCCESS >>>>>
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_mobile_station_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_950_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_single_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_experimental_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_950_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_X86_monitor_910b ✅ SUCCESS >>>>> >>>>>
Compile_X86_monitor_910c ✅ SUCCESS >>>>> >>>>>
Compile_X86_monitor_950 ✅ SUCCESS >>>>> >>>>>
Compile_classify ✅ SUCCESS >>>>>
Compile_Ascend_X86_mobile_station_9030_ubuntu24 ✅ SUCCESS >>>>> >>>>>
UT_Test_ophost ✅ SUCCESS
UT_Test_opapi ✅ SUCCESS
UT_Test_kernel ✅ SUCCESS
UT_Test_opgraph ✅ SUCCESS
PreSmoke_A900 ✅ SUCCESS >>>>>
API_Check ✅ SUCCESS >>>>>
PreSmoke_ATK_Test_A2 ✅ SUCCESS >>>>>
UT_Test_report_lcov ⚠️ WARNING >>>>>

[2026-08-26 16:42:28]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
22 天前 评论:

流水线任务触发成功
任务链接 [d67783bbc7e148a39c53c69be2beac23][流水线指导]

任务名称状态日志下载链接
antipoison ✅ SUCCESS >>>>>
codecheck_checkpr ✅ SUCCESS
codecheck_precommit ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
StaticCheck_md ✅ SUCCESS

[2026-08-26 15:54:56]    CI执行结束

likedislike
gaomingyuangaomingyuan
22 天前 修改了pull request 的描述
CANN-robotCANN-robot成员
22 天前 添加了label:api-check-pass
CANN-robotCANN-robot成员
22 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
22 天前 添加了label:ci-pipeline-passed
wang-xing001成员
22 天前 评论:

/lgtm
/approve

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

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
22 天前 添加了label:lgtm
CANN-robotCANN-robot成员
22 天前 合入了pull request
yuning_chen
yuning_chen成员
22 天前 评论:

您好,pr合入已准备就绪,请尽快联系committer进行检视,谢谢!

likedislike