已合并
[9.1.0同步] capture model update adapt & fix: A5 update save arg handle #4004
yanan创建于 25 天前
[9.1.0同步] capture model update adapt & fix: A5 update save arg handle #4004
已合并
yanan创建于 25 天前
yanan成员
25 天前

Pull Request

描述

1、super kernel场景:如果流上全部任务都disable后,下发一个NOP,不做streamDestroy。
2、aclgraph场景,kernel任务更新为其他类型时,arg资源不能释放,kernel分支已适配,本次适配write value和wait value分支。
3、资料添加约束

变更类型

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

关联的Issue

  • 同步自 PR #3995
  • 同步自 PR #3971

如何测试

  1. 测试sk场景流上任务全部disable
  2. 测试aclgraph场景kernel任务更新为write value和wait value类型时arg资源不释放

核对清单

其他信息

本 PR 将 master 分支的两个修改同步到 9.1.0 分支:

  • PR #3995: capture model update adapt
  • PR #3971: fix: A5 update save arg handle
likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 yanan 的贡献)
Yyanan成员
25 天前 创建了 pull request,commit 681550a4
atomgit-bot
atomgit-bot
25 天前 评论:

变更摘要

本次 PR 是 #3995(master 分支)向 9.1.0 分支的 cherry-pick,主要针对 super kernel 场景下 capture stream disable 的处理策略进行适配:原先在 disable 时会直接销毁 stream,现改为发送 NOP 任务来替代 StreamDestroy,以更安全的方式完成 capture model 更新。

主要改动

  • Stream::UpdateAllPersistentTask() 放宽 NOP 发送条件:移除 parentCaptureStream_ != nullptr 的前置判断,现在仅需 totalSendSqeNum == 0U 即可发送 NOP 任务,使得没有父捕获流时也能正常构造 NOP。
  • CaptureModel::Update() 移除流销毁逻辑:删除 GetDelayRecycleTaskSqeNum() == 0U 条件下调用 ModelDelStreamStreamDestroy 的代码块(共 5 行),disable 场景不再直接销毁流,改为依赖 NOP 机制处理。
likedislike
atomgit-bot
atomgit-bot
25 天前 评论:

代码审查

✅ 未发现问题

likedislike
CANN-robotCANN-robot成员
25 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
25 天前 评论:

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
9.1.0 Andy-lb, 侯延保 (2/2) Andy-lb (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
此处折叠了83条消息 查看更多
houyanbao成员
23 天前 评论:

/lgtm

likedislike
Yyanan成员
22 天前 修改了pull request 的描述
Andy-lb成员
22 天前 评论:

/approve

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