已合并
fix(om2): 删除L0 exception dump不支持的过时警告 #4491
fix(om2): 删除L0 exception dump不支持的过时警告 #4491
已合并
lidaoming1创建于 16 天前
lidaoming1成员
16 天前

描述

OM2 已支持 L0 exception dump(aic_err_brief_dumplite_exception),CheckDumpSceneSwitch 中的 validScenes 已包含这两个场景,IsL0ExceptionDumpEnabled 也有对应实现。删除 CheckUnsupportedConfigs 中对该场景输出 "not supported in OM2" 的过时警告代码。

变更类型

关联的Issue

如何测试

  1. 配置 dump_scene 为 aic_err_brief_dumplite_exception,确认不再输出 "not supported in OM2" 警告
  2. 确认 L0 exception dump 功能正常工作

核对清单

其他信息

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 lidaoming1 的贡献)
Llidaoming1成员
16 天前 创建了 pull request,commit 3a868237
atomgit-bot
atomgit-bot
16 天前 评论:

变更摘要

该 PR 属于 Bug 修复,针对 OM2 运行时 dump 配置校验逻辑:由于 OM2 已支持 L0 exception dump(aic_err_brief_dumplite_exceptionCheckDumpSceneSwitchvalidScenes 已包含这两个场景,且 IsL0ExceptionDumpEnabled 已有对应实现),因此在 runtime/om2/dump/dump_config.ccDumpConfig::CheckUnsupportedConfigs 中删除了此前对这些场景输出 "not supported in OM2" 的过时警告代码,避免用户配置 L0 exception dump 场景时收到误导性警告。

主要改动

  • 删除过时警告逻辑: 移除 CheckUnsupportedConfigs 中针对 GE_DUMP_SCENE 配置项存在性判断及其对应的警告分支,该分支会读取 dump_scene 值并输出 L0 exception dump 不支持的告警。
  • 移除 aic_err_brief_dump 场景警告: 不再对 GE_DUMP_EXCEPTION_AIC_ERR_BRIEF 场景输出 "not supported in OM2" 的 GELOGW 警告日志。
  • 移除 lite_exception 场景警告: 不再对 GE_DUMP_LITE_EXCEPTION 场景输出 "not supported in OM2" 的 GELOGW 警告日志。
  • 保留其余校验逻辑: 函数体仅删除上述 9 行过时警告代码,CheckUnsupportedConfigs 中其他配置校验及 CheckDumpSceneSwitch 等现有逻辑保持不变。
likedislike
atomgit-bot
atomgit-bot
16 天前 评论:

代码审查

✅ 未发现问题

likedislike
CANN-robotCANN-robot成员
16 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
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.
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
repo-cann/ge 张德鹏, ClarkXie, stevenaw0 (3/2) 张德鹏 (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
16 天前 将yangyongqiang0606,kobemini,ClarkXie,tangqunzhang,xchu42,hugo111,lining23666,fu-jun2,depeng1994,wqtshg_wt,peiyang,shengnan666,zhangfan_hanq,zhanj,zhujingjing,Mexyy,wangxiaotian995,stevenaw0,yuht9设为评审人
CANN-robotCANN-robot成员
16 天前 将yangyongqiang0606,kobemini,ClarkXie,tangqunzhang,xchu42,hugo111,lining23666,fu-jun2,depeng1994,wqtshg_wt,peiyang,shengnan666,zhangfan_hanq,zhanj,zhujingjing,Mexyy,wangxiaotian995,stevenaw0,yuht9设为审查人
lidaoming1成员
16 天前 评论:

compile

likedislike
Llidaoming1成员
16 天前 update merge request[project id: 8824148, iid: 4491, commit_id: 7a3445387726aa97f9992976d4f25f7b8d4fd8a7] virtual merging success
CANN-robotCANN-robot成员
16 天前 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
16 天前 评论:

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

任务名称状态日志下载链接
pre_comment ✅ SUCCESS >>>>>
Compile_ARM_compiler ✅ SUCCESS >>>>> >>>>>
Compile_ARM_executor ✅ SUCCESS >>>>> >>>>>
Compile_ARM_dflow ✅ SUCCESS >>>>> >>>>>
API_Check ✅ SUCCESS >>>>>
PreSmoke_A900_npupool ✅ SUCCESS >>>>>

[2026-08-21 11:31:18]    CI执行结束

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

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

任务名称状态日志下载链接
pre_comment ✅ SUCCESS >>>>>
Compile_X86_compiler ✅ SUCCESS >>>>> >>>>>
Compile_X86_executor ✅ SUCCESS >>>>> >>>>>
Compile_X86_dflow ✅ SUCCESS >>>>> >>>>>
UT_Test_dflow ✅ SUCCESS
UT_Test_ge_common ✅ SUCCESS
UT_Test_parser ✅ SUCCESS
UT_Test_python ✅ SUCCESS >>>>>
UT_Test_rt ✅ SUCCESS
UT_Test_acl ✅ SUCCESS
UT_Test_autofuse ✅ SUCCESS
UT_Test_executor ✅ SUCCESS
UT_Test_autofuse_ascendc_api ✅ SUCCESS
ST_Test_dflow ✅ SUCCESS
ST_Test_ge_common ✅ SUCCESS
ST_Test_parser ✅ SUCCESS
ST_Test_python ✅ SUCCESS >>>>>
ST_Test_rt ✅ SUCCESS
ST_Test_autofuse ✅ SUCCESS
ST_Test_executor ✅ SUCCESS
ST_Test_hetero ✅ SUCCESS
ST_Test_autofuse_ascendc_api ✅ SUCCESS
ST_Test_autofuse_e2e ✅ SUCCESS
UT_Test_fe ✅ SUCCESS
UT_Test_tefusion ✅ SUCCESS
ST_Test_fe ✅ SUCCESS
ST_Test_tefusion ✅ SUCCESS
ST_Test_ge_common_atc ✅ SUCCESS
UT_Test_Report_graphengine ✅ SUCCESS >>>>>
UT_Test_Report_ge_executor_c ✅ SUCCESS >>>>>
UT_Test_Report_ge_autofuse ✅ SUCCESS >>>>>
ST_Test_Report_graphengine ✅ SUCCESS >>>>>
ST_Test_Report_ge_executor_c ✅ SUCCESS >>>>>
ST_Test_Report_ge_autofuse ✅ SUCCESS >>>>>
UT_Test_Report_fe ✅ SUCCESS >>>>>
ST_Test_Report_fe ✅ SUCCESS >>>>>

[2026-08-21 12:07:47]    CI执行结束

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

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

任务名称状态日志下载链接
codecheck_Pr ✅ SUCCESS
antipoison ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
codecheck_dt ✅ SUCCESS >>>>>
codecheck_precommit ✅ SUCCESS >>>>>
StaticCheck_codespell ✅ SUCCESS
StaticCheck_link_validity ✅ SUCCESS
StaticCheck_resource_existence ✅ SUCCESS
StaticCheck_tag_closed ✅ SUCCESS
StaticCheck_markdownlint ✅ SUCCESS

[2026-08-21 11:29:51]    CI执行结束

likedislike
CANN-robotCANN-robot成员
16 天前 添加了label:api-check-pass
CANN-robotCANN-robot成员
16 天前 删除了label:api-check-pass
CANN-robotCANN-robot成员
16 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
16 天前 添加了label:ci-pipeline-passed
ClarkXie成员
16 天前 评论:

/lgtm

likedislike
黄桂军
黄桂军成员
16 天前 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
16 天前 添加了label:lgtm
张德鹏成员
16 天前 评论:

/approve

likedislike
CANN-robotCANN-robot成员
16 天前 添加了label:approved
CANN-robotCANN-robot成员
16 天前 合入了pull request