Closed
[Requirement|需求建议]:针对mc2近期出现的数据地址越界的情况,希望进行批量排查修改 #7284
adamlwangAdd on  6 days agoClosed at  5 days ago
adamlwangCollaborators
6 days ago Create

Backgroud(背景信息)

针对mc2近期出现的数据地址越界的情况,希望进行批量排查修改,消除潜在隐患

Benefit / Necessity (价值/作用)

Design(设计方案)

likedislike
AadamlwangCollaborators
6 days ago Added label: requirement
AadamlwangCollaborators
6 days ago Modify description of issue
AadamlwangCollaborators
6 days ago Modify description of issue
周云飞
周云飞Collaborators
6 days ago Comments:

问题说明

目前还不能确定近期 MC2 地址越界是否属于同一类问题,也不足以确定批量修改范围。现有片段主要涉及图融合匹配及输入数量等检查,不能据此判断运行时数据访问是否越界。

处理建议

建议先以已发生的案例为依据,排查同类地址偏移、缓冲区大小计算与实际读写范围,再对确认存在共性的实现批量修改,重点验证边界尺寸,并保持合法输入行为不变。

请补充一个代表性案例的越界日志及触发配置(算子、输入形状与类型、CANN 版本和芯片型号),或提供已有问题链接,以便确定排查入口和适用范围。

likedislike
AadamlwangCollaborators
6 days ago Associated pull request
周云飞
周云飞Collaborators
6 days ago Comments:

/assign @adamlwang

likedislike
CANN-robotCANN-robotCollaborators
6 days ago Assign adamlwang as responsible person
CANN-robotCANN-robotCollaborators
5 days ago Close issue
CANN-robotCANN-robotCollaborators
5 days ago Added label: resolved
周云飞
周云飞Collaborators
5 days ago Comments:

已针对 MC2 部分潜在地址偏移越界风险进行修改:在 AllGatherMatmul 的存储空间计算、AlltoAllMatmul 的量化数据偏移计算,以及 MatmulAllReduce 的工作空间大小计算中,将相关乘法运算提升至 64 位,避免中间结果溢出。

PR #15564 描述中记录了内部二级冒烟测试通过。

关联 PR:PR #15563、PR #15564

likedislike