已合并
【PR】: 简要描述 #3265
【PR】: 简要描述 #3265
已合并
wda1991创建于 6月30日
wda1991成员
6月30日

Pull Request

描述

请清晰准确地描述本次 Pull Request 的意图和变更内容。

变更类型

请选择本次引入的变更类型:

关联的Issue

如何测试

描述测试此变更的步骤和前提条件:
1.
2.

核对清单

其他信息

在此添加任何其他关于本次 PR 的说明。

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

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 ✅ 卢煜坤, zhangpengpeng8 (2/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

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

likedislike
wda1991成员
6月30日 评论:

/compile

likedislike
Wwda1991成员
6月30日 预合并成功(commit_id: 3f4005f7735cbd197439f326279416cff30f9b3d)
ykl999
ykl999成员
6月30日 评论:

/approve
/lgtm

likedislike
CANN-robotCANN-robot成员
6月30日 添加了label:approved
CANN-robotCANN-robot成员
6月30日 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
6月30日 评论:

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

任务名称状态日志下载链接
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
pre_comment ✅ SUCCESS >>>>>
UT_Test_acl ✅ SUCCESS >>>>>
UT_Test_rts_v201 ✅ SUCCESS >>>>>
UT_Test_rts_c ✅ SUCCESS >>>>>
UT_Test_rts_david ✅ SUCCESS >>>>>
UT_Test_rts_910b ✅ SUCCESS >>>>>
UT_Test_rts_common ✅ SUCCESS >>>>>
UT_Test_platform ✅ SUCCESS >>>>>
UT_Test_qs ✅ SUCCESS >>>>>
UT_Test_aicpusd ✅ SUCCESS >>>>>
UT_Test_tsd ✅ SUCCESS >>>>>
UT_Test_error_manager ✅ SUCCESS >>>>>
UT_Test_slog ✅ SUCCESS >>>>>
UT_Test_atrace ✅ SUCCESS >>>>>
UT_Test_msprof_part1 ✅ SUCCESS
UT_Test_msprof_part2 ✅ SUCCESS
UT_Test_adump ✅ SUCCESS >>>>>
UT_Test_mmpa ✅ SUCCESS >>>>>
PreSmoke_A900_npupool ✅ SUCCESS >>>>>
UT_Test_rts_report ✅ SUCCESS >>>>>
UT_Test_msprof_report ✅ SUCCESS

[2026-06-30 17:38:42]    CI执行结束

likedislike
atomgit-bot
atomgit-bot
6月30日 评论:

变更摘要

该 PR 是从 cherry-pick-mr-3216-1782811893783-auto 分支向 9.1.0 分支发起的自动合入请求,属于 MR #3216 的 cherry-pick 操作。本次变更涉及 1 个文件,净减少 165 行代码(新增 176 行,删除 341 行),属于一次代码重构或清理性质的改动。由于 PR 描述模板未填写具体内容,且未提供变更文件的实际 diff 补丁,无法进一步确定具体改动的文件路径和业务细节。

主要改动

  • Cherry-pick 合入:将 MR #3216 的变更通过 cherry-pick 方式合入 9.1.0 分支,保持版本线的一致性
  • 代码精简约 165 行:1 个文件经修改后,删除 341 行、新增 176 行,整体代码量显著减少,推测涉及函数/逻辑的合并、简化或移除冗余代码
likedislike
不准确?
CANN-robot
CANN-robot成员
6月30日 评论:

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

任务名称状态日志下载链接
codecheck ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
codecheck_Check_Pr ✅ SUCCESS
codecheck_style ✅ SUCCESS >>>>>
codecheck_precommit ✅ SUCCESS >>>>> >>>>>
StaticCheck_codespell_check ✅ SUCCESS
StaticCheck_link_validity_check ✅ SUCCESS
StaticCheck_resource_existence_check ✅ SUCCESS
StaticCheck_tag_closed_check ✅ SUCCESS
StaticCheck_markdownlint ✅ SUCCESS

[2026-06-30 17:37:43]    CI执行结束

likedislike
atomgit-bot
atomgit-bot
6月30日 评论:

代码审查

审查完成

已审查文件:

  • docs/design/modules/stream/stream.md — 发现 1 个 P3 问题

审查总结

优先级 问题数
P0 0
P1 0
P2 0
P3 1

整体风险评估:低风险。本次变更对 Stream 模块设计文档进行了大幅重构和精简,移除了大量内部实现细节(架构分层图、多类型流派生类图、自动切分 SQ、流捕获 API 等),新增了更清晰的 SQ/CQ 异步机制说明。唯一值得关注的是跨流事件同步 API(aclrtStreamWaitEvent / aclrtStreamWaitEventWithTimeout)及其使用场景被完全移除——若这些 API 在目标分支中仍然有效,可能导致开发者采用不正确的流同步方式。其余变更均为文档结构优化,无安全性或正确性问题。

⚠️ 已识别出整体风险,但无法提取行内评论,请参考整体评估。

likedislike
不准确?
CANN-robotCANN-robot成员
6月30日 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
6月30日 添加了label:ci-pipeline-passed
zhangpengpeng8成员6月30日进行代码检视2
docs/design/modules/stream/stream.md
已过期
@@ -69,4 +62,0 @@
69-| `aclrtStreamAbort()` | `include/external/acl/acl_rt.h` | 终止流执行 |
70-| `aclrtStreamGetId()` | `include/external/acl/acl_rt.h` | 获取流 ID |
71-| `aclrtGetStreamAvailableNum()` | `include/external/acl/acl_rt.h` | 获取可用流数量 |
72-| `aclrtStreamWaitEventWithTimeout()` | `include/external/acl/acl_rt.h` | 流等待事件带超时 |
zhangpengpeng86月30日评论:

aclrtStreamWaitEventWithTimeout这个接口不支持了?

likedislike
wda1991成员
7月1日 评论:

统一在event模块介绍

Wwda1991成员
7月1日 解决了最后一个问题
zhangpengpeng8成员
7月1日 评论:

/lgtm

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