已合并
去除代码中显式的CheckLogLevel调用,改为使用OP_LOGD正式日志接口 #1002
liu-wei创建于 6月16日
去除代码中显式的CheckLogLevel调用,改为使用OP_LOGD正式日志接口 #1002
已合并
liu-wei创建于 6月16日
liu-wei
liu-wei成员
6月16日

描述

去除 common/inc/op_host/tiling_base.h 中 TilingBaseClass::DumpTilingInfo() 里显式的 CheckLogLevel 调用,改为直接使用 OP_LOGD 正式日志接口。

变更内容(2 个文件,+8/-6):

1. common/inc/op_host/tiling_base.h(+7/-5)—— 主题改动

  • DumpTilingInfo() 内部去掉"先 CheckLogLevel 判级别、再 OP_LOGD"的两步式写法
  • 拆出 int64_t DefaultTilingInfoDump() 成员函数,把"获取 raw tiling data + 构造 oss + 调 OP_LOGD"逻辑搬过去
  • DumpTilingInfo() 简化为:OP_LOGD(context_, "%ld", DefaultTilingInfoDump());——把日志级别判断交给 OP_LOGD 内部
  • DefaultTilingInfoDump() 返回 int64_t(占位 return 0),便于后续可测 / 可 mock
  • 顺手补文件末尾换行

2. cmake/func.cmake(+1/-1)—— 顺手补 include path

-    target_include_directories(opbase_util_objs PRIVATE ${OP_TILING_INCLUDE})
+    target_include_directories(opbase_util_objs PRIVATE ${OP_TILING_INCLUDE} ${ASCEND_DIR}/pkg_inc)

add_opbase_modules() 中给 opbase_util_objs target 补 ${ASCEND_DIR}/pkg_inc include path——让 opbase_util_objs 能找到 CANN 包自带的额外头文件,避免部分构建场景下 include 路径缺失。

关于变更范围说明:本 PR 主题是"去 CheckLogLevel",cmake/func.cmake 这一行改动不属于该主题——属于顺手补 include path 的小修(同一特性切片下发现的构建缺失)。考虑到改动量极小(+1/-1)且跟其他 opbase_util_objs 的 include 配置类似,合并到本 PR 比单独开 PR 更经济。如果 reviewer 倾向于严格原子提交,可以拆出另开 PR。

功能等价性(针对主题改动 tiling_base.h):

行为 改前 改后
日志级别不达 debug CheckLogLevel 返回 0 → 早退(不打 log) OP_LOGD 内部判级别,不达 debug 不输出
日志级别达 debug 打 "Start to dump tiling info..." 先打 "0"(%ld 的占位值),再打 "Start to dump..."
ABI 兼容性 virtual void DumpTilingInfo() 签名不变 DefaultTilingInfoDump() 是 protected 非 virtual,子类可继承但不能 override

已知小问题(不影响功能):改后每次 dump 会多打一条 "0" 的冗余 log——这是为了保留 DefaultTilingInfoDump 作为可独立调用 / 可测试的接口而做的妥协。后续如需清理这条冗余 log,可单独提交 PR 处理。

关联的Issue

关闭 Issue: #537(去除 TilingBaseClass::DumpTilingInfo 中显式的 CheckLogLevel 调用,改为使用 OP_LOGD 日志接口)

测试

文档更新

  • 不涉及 README.md / CONTRIBUTING.md / docs/ 改动
  • tiling_base.h 头注释未变

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 liu-wei 的贡献)
liu-weiliu-wei成员
6月16日 创建了 pull request,commit 476b6796
CANN-robotCANN-robot成员
6月16日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
6月16日 评论:

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


PR Approval Progress

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

Module Approval Details

module lgtm status approve status
image/grid_sample ✅ 徐鲁威, 张磊 (2/2) ✅ 徐鲁威 (1/1)
image/grid_sampler2_d_grad ✅ 张磊, 徐鲁威 (2/2) ✅ 徐鲁威 (1/1)
repo-cann/ops-cv ✅ 徐鲁威, 张磊 (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

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

likedislike
liu-wei
liu-wei成员
6月16日 评论:

compile

likedislike
liu-weiliu-wei成员
6月16日 预合并成功(commit_id: 789507f0fde1236ce76e26d501483902777033a3)
CANN-robotCANN-robot成员
6月16日 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
6月16日 评论:

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

任务名称状态日志下载链接
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_mobile_station ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_experimental ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_experimental ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_A5 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_A5 ✅ SUCCESS >>>>> >>>>>
pre_comment ✅ SUCCESS >>>>>
Compile_Pre ✅ SUCCESS >>>>>
Compile_X86_monitor_910b ✅ SUCCESS >>>>> >>>>>
Compile_X86_monitor_910c ✅ SUCCESS >>>>> >>>>>
Compile_X86_monitor_950 ✅ SUCCESS >>>>> >>>>>
UT_Test ❌ FAILED
PreSmoke_A900 ✅ SUCCESS >>>>>
PreSmoke_ATK_Test_A2 ✅ SUCCESS >>>>>
likedislike
CANN-robot
CANN-robot成员
6月16日 评论:

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

任务名称状态日志下载链接
codecheck ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
codecheck_checkpr ✅ SUCCESS
StaticCheck_codespell_check ✅ SUCCESS
StaticCheck_link_validity_check ✅ SUCCESS
StaticCheck_resource_existence_check ✅ SUCCESS
StaticCheck_tag_closed_check ✅ SUCCESS
StaticCheck_markdownlint ✅ SUCCESS
codecheck_codestyle ⚠️ WARNING >>>>>
codecheck_precommit ⚠️ WARNING >>>>> >>>>>

[2026-06-16 15:02:37]    CI执行结束

likedislike
CANN-robotCANN-robot成员
6月16日 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
6月16日 添加了label:ci-pipeline-failed
liu-weiliu-wei成员
6月16日 预合并成功(commit_id: 2ae1d605f4cc94f850be95ff7f875a75bde842cd)
liu-weiliu-wei成员
6月16日 强制推送  1 个提交:0dad203d-去除代码中显式的CheckLogLevel调用,改为使用OP_LOGD正式日志接口
liu-weiliu-wei成员
6月16日 预合并成功(commit_id: 361ffd72c702813953b7b50eec42d3cc94013fcd)
liu-wei
liu-wei成员
6月16日 评论:

compile

likedislike
liu-weiliu-wei成员
6月16日 预合并成功(commit_id: c2534a3bece81abd921e3ce540dfd812cf1d2084)
CANN-robotCANN-robot成员
6月16日 删除了label:ci-pipeline-failed
CANN-robotCANN-robot成员
6月16日 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
6月16日 评论:

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

任务名称状态日志下载链接
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_mobile_station ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_experimental ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_experimental ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_A5 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_A5 ✅ SUCCESS >>>>> >>>>>
pre_comment ✅ SUCCESS >>>>>
Compile_Pre ✅ SUCCESS >>>>>
Compile_X86_monitor_910b ✅ SUCCESS >>>>> >>>>>
Compile_X86_monitor_910c ✅ SUCCESS >>>>> >>>>>
Compile_X86_monitor_950 ✅ SUCCESS >>>>> >>>>>
UT_Test ✅ SUCCESS
PreSmoke_A900 ✅ SUCCESS >>>>>
PreSmoke_ATK_Test_A2 ✅ SUCCESS >>>>>

[2026-06-16 15:36:25]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
6月16日 评论:

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

任务名称状态日志下载链接
codecheck ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
codecheck_checkpr ✅ SUCCESS
StaticCheck_codespell_check ✅ SUCCESS
StaticCheck_link_validity_check ✅ SUCCESS
StaticCheck_resource_existence_check ✅ SUCCESS
StaticCheck_tag_closed_check ✅ SUCCESS
StaticCheck_markdownlint ✅ SUCCESS
codecheck_codestyle ⚠️ WARNING >>>>>
codecheck_precommit ⚠️ WARNING >>>>> >>>>>

[2026-06-16 15:15:27]    CI执行结束

likedislike
CANN-robotCANN-robot成员
6月16日 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
6月16日 添加了label:ci-pipeline-passed
liu-weiliu-wei成员
6月16日 修改了pull request 的描述
DevLeev成员
6月17日 评论:

/lgtm

likedislike
CANN-robot
CANN-robot成员
6月17日 评论:

The following users do not have permission to comment /lgtm or /approve on any module in this PR:
LiWei79

likedislike
songkai111成员
6月17日 评论:

/lgtm
/approve

likedislike
CANN-robot
CANN-robot成员
6月17日 评论:

The following users do not have permission to comment /lgtm or /approve on any module in this PR:
宋凯, LiWei79

likedislike
CANN-robot
CANN-robot成员
6月17日 评论:

The following users do not have permission to comment /lgtm or /approve on any module in this PR:
宋凯, LiWei79

likedislike
loov成员
6月17日 评论:

/lgtm

likedislike
CANN-robot
CANN-robot成员
6月17日 评论:

The following users do not have permission to comment /lgtm or /approve on any module in this PR:
宋凯, LiWei79

likedislike
zl_hw成员
6月17日 评论:

/lgtm
/approve

likedislike
CANN-robot
CANN-robot成员
6月17日 评论:

The following users do not have permission to comment /lgtm or /approve on any module in this PR:
LiWei79, 宋凯

likedislike
CANN-robotCANN-robot成员
6月17日 添加了label:lgtm
CANN-robot
CANN-robot成员
6月17日 评论:

The following users do not have permission to comment /lgtm or /approve on any module in this PR:
宋凯, LiWei79

likedislike
liu-wei
liu-wei成员
6月17日 评论:

/check-pr

likedislike
CANN-robot
CANN-robot成员
6月17日 评论:

The following label is not ready.

approved: Please wait for committers to review the code.

likedislike
loov成员
6月17日 评论:

/lgtm
/approve

likedislike
CANN-robot
CANN-robot成员
6月17日 评论:

The following users do not have permission to comment /lgtm or /approve on any module in this PR:
宋凯, LiWei79

likedislike
CANN-robotCANN-robot成员
6月17日 添加了label:approved
CANN-robotCANN-robot成员
6月17日 合入了pull request