已合并
[fix][ops]MatmulReduceScatter算子修复string_view悬空指针 #5383
chenjl创建于 7月6日
[fix][ops]MatmulReduceScatter算子修复string_view悬空指针 #5383
已合并
chenjl创建于 7月6日
chenjl
7月6日

Issue #248

【合入来源】

如有社区issue,请关联issue链接
请勿携带内部流程信息(需求链接、问题单、内部issue等)

【修改方案】

修复MatmulReduceScatter算子base/quant接口下c10::string_view 悬空指针导致非阻塞模式下 group 名读到脏数据的问题。由于 EXEC_NPU_CMD 可能以非阻塞方式执行,string_view 所引用的原始字符串可能在此之前析构,导致指针悬空、读取脏数据。
修改方案:将 string_view 的内容复制到局部 std::string 对象中,再从中获取指针,确保在函数调用期间指针始终有效

【资料变更】

不涉及

【接口变更】

不涉及

【功能验证】

测试用例来源:MC2自验证用例
结果:PASS
image.png

【CheckList】

PR提交人对以下CheckList自检项进行全量自检,自检通过或不涉及,均修改 [ ] 为 [x]

likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 chenjl 的贡献)
Cchenjl
7月6日 创建了 pull request,commit 026f9ff9
ascend-robotascend-robot成员
7月6日 添加了label:ascend-cla/yes
ascend-robotascend-robot成员
7月6日 添加了label:needs-issue
ascend-robot
ascend-robot成员
7月6日 评论:

CLA Signature Pass

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

likedislike
此处折叠了95条消息 查看更多
zichun_ye
zichun_ye成员
7月7日 评论:

/approve

likedislike
ascend-robotascend-robot成员
7月7日 添加了label:approved
songkai111
7月8日 评论:

/lgtm

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