已合并
[fix] 替换 deprecated aclrtSetExceptionInfoCallback 为新接口 Register/Unregister #4542
zengls创建于 10 天前
[fix] 替换 deprecated aclrtSetExceptionInfoCallback 为新接口 Register/Unregister #4542
已合并
Pull Request已成功合入, 合并人@CANN-robot
（感谢 zengls 的贡献）
atomgit-bot
10 天前 评论：
10 天前 评论：
atomgit-bot
10 天前 评论：
10 天前 评论：
代码审查
所有 7 个文件已逐一审查完毕。以下是审查结论。
关闭摘要
| 优先级 | 数量 |
|---|---|
| P0 | 0 |
| P1 | 1 |
| P2 | 0 |
| P3 | 0 |
各文件审查结果：
| 文件 | 结论 |
|---|---|
src/coll_communicator_mgr/dfx/taskException/host/hcclCommTaskException.cc |
无问题 — Register/UnRegister/析构函数三处替换正确 |
src/legacy/ascend950/unified_platform/external_system/orion_adapter_rts.cc |
P1 问题 — 缺少 nullptr 分流到 Unregister |
test/hccl_vm/src/proxy/aclrt_exception_stub.cc |
无问题 |
test/legacy/st/fwk/rts_stub/rts_stub.cc |
无问题 |
test/legacy/ut/unified_platform/external_system/system/rts_stub_ut.cc |
无问题 |
test/stub/depends/include/acl/acl_rt.h |
无问题 — 新接口声明正确，保留旧声明兼容 |
test/ut/stub/llt_next_orion_stub.cc |
无问题 |
总体风险评估： 本次变更整体风险可控，但存在一项 P1 级逻辑缺陷——HrtRegTaskFailCallbackByModule 未按 PR 描述要求对 callback == nullptr 做分流，导致 TaskExceptionHandler::UnRegister() 调用时实际执行的是注册 nullptr 而非注销回调，task exception 注销逻辑完全失效。该问题需要在合入前修复。
| 类型 | 数量 |
|---|---|
| 🔴 阻塞 | 1 |
| 🟡 建议 | 0 |
⛔ 需要修改
10 天前 添加了label：cann-cla/yes
CANN-robot
10 天前 评论：
10 天前 评论：
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
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 |
|---|---|---|
| src/coll_communicator_mgr | ✅ 颜业峰, dingweiqin_57 (2/2) | ✅ dingweiqin_57 (1/1) |
| src/legacy/ascend950/framework | ✅ dingweiqin_57, 颜业峰 (2/2) | ✅ dingweiqin_57 (1/1) |
| src/legacy/ascend950/unified_platform | ✅ Innoecho, 颜业峰 (2/2) | ✅ Innoecho (1/1) |
| test | ✅ Innoecho, dingweiqin_57, 颜业峰 (3/2) | ✅ Innoecho, dingweiqin_57 (2/1) |
| test/hccl_vm | ✅ 颜业峰, whk19940509 (2/2) | ✅ whk19940509 (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
gcw_xljEvKEv, thanks for your pull request. All authors of the commits have signed the CLA. 👍
此处折叠了162条消息 查看更多
whk_24
7 天前 评论：
7 天前 评论：
/approve
57_Viking
6 天前 评论：
6 天前 评论：
/approve
6 天前 添加了label：lgtmapproved
6 天前 合入了pull request
描述
runtime 头将 aclrtSetExceptionInfoCallback 标记为 deprecated（-Werror=deprecated-declarations 导致编译失败），需替换为新接口 aclrtExceptionInfoCallbackRegister / aclrtExceptionInfoCallbackUnregister。
新接口语义（对照 runtime/src/acl/aclrt_impl/callback.cpp 实现）：
改动映射规则：
改动范围：
两函数各自传 callback，不再依赖 nullptr 区分意图。
变更类型
请选择本次引入的变更类型：
关联的Issue
NA
测试
已完成的测试用例和场景：
补充的UT用例：
本次为接口替换，原有 TaskException 相关 UT 用例通过 stub 覆盖 Register/Unregister 路径，未新增用例。
文档更新
更新了 taskException.md / taskException_en.md 中接口名与参数描述，与代码保持一致。
合入检查