已合并
feat: 新增acldumpSaveExceptionInfo/acldumpGetExceptionInfoPath接口 #3993
yring_8创建于 8月1日
feat: 新增acldumpSaveExceptionInfo/acldumpGetExceptionInfoPath接口 #3993
已合并
yring_8创建于 8月1日
已删除 :save_exception合入到cann/runtimemaster
yring_8成员
8月1日

Pull Request

描述

新增 acldumpSaveExceptionInfo / acldumpGetExceptionInfoPath 两个异常信息保存接口,支持在算子异常场景下将用户自定义的 tensor 数据主动落盘到 Exception Dump 路径,用于扩充维测信息。

主要变更:

  • 新增两个对外接口(include/external/acl/acl_dump.h),C 风格签名,保持 ABI 兼容
  • 提取 Path::BuildFullPathUnderRoot 公共函数:校验相对路径不含 .. 防路径穿越,递归建父目录 + RealPath + IsDirectory 校验
  • 落盘文件名追加custom+毫秒级时间戳后缀,避免重复运行覆盖已有文件
  • 落盘逻辑内聚到 ExceptionDumperSaveExceptionInfo / GetExceptionDumpPath),DumpManager 仅转发
  • 参数校验:fileName 空指针/空串、tensors 空指针、shapeNum/originShapeNum 超限,分别结构化上报(EP0006/0007/0008)
  • 补充接口参考文档(docs/zh/api_ref/18_dump_configuration.md)、头文件注释及 example 样例(3_save_exception_info

变更类型

关联的Issue

如何测试

  1. 通过 acl.json 配置 dump_scene=aic_err_brief_dump 使能 Exception Dump
  2. 调用 acldumpGetExceptionInfoPath 获取落盘根路径,构造 acldumpTensorInfo 后调用 acldumpSaveExceptionInfo 落盘
  3. 运行 adump UT:bash tests/build_ut.sh -u adump

测试结果:

  • adump_base_utest 637/637 通过
  • adump_tiny_utest 36/36 通过
  • 新增 UT 覆盖各错误分支、时间戳后缀、路径穿越防护

核对清单

其他信息

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 yring_8 的贡献)
Yyring_8成员
8月1日 添加了label:enhancement
Yyring_8成员
8月1日 创建了 pull request,commit 8d2e9563
CANN-robotCANN-robot成员
8月1日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
8月1日 评论:

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
docs 卢煜坤, 王涛 (2/2) 卢煜坤, 王涛 (2/1)
example/5_performance newstarzj, 卢煜坤, zhuliangying, 王涛 (4/2) zhuliangying (1/1)
include/external/acl/acl_dump.h zhuliangying, 卢煜坤, 王涛, newstarzj (4/2) zhuliangying (1/1)
src/dfx zhuliangying, newstarzj, 卢煜坤, 王涛 (4/2) zhuliangying (1/1)
tests/ut/adump zhuliangying, newstarzj, 卢煜坤, 王涛 (4/2) zhuliangying (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
8月1日 将newstarzj,chenhao_1209,zhuliangying,wangtao43,zhangpengpeng8,yanmingxiang,Reyn52166,ykl999,houyanbao,tingwood设为评审人
此处折叠了187条消息 查看更多
ykl999
ykl999成员
8月7日 评论:

/approve
/lgtm

likedislike
CANN-robotCANN-robot成员
8月7日 添加了label:approved
wangtao成员
8月7日 评论:

/approve
/lgtm

likedislike
CANN-robotCANN-robot成员
8月7日 添加了label:lgtm
CANN-robotCANN-robot成员
8月7日 合入了pull request