已合并
modify quant_reduce_scatter hccl_context #6357
modify quant_reduce_scatter hccl_context #6357
已合并
yifuxiong创建于 6月3日
yifuxiong
yifuxiong成员
6月3日

描述

切换修改新的hccl context开源方案

关联的Issue

https://gitcode.com/cann/ops-transformer/issues/2844

测试

image.png

文档更新

不涉及

类型标签

代码检视报告

检视概览

  • PR 链接:https://gitcode.com/cann/ops-transformer/pull/6357
  • PR 描述:merge 'pr_qrs_context' into 'master' — 为 quant_reduce_scatter 算子引入 Mc2Context(v2)路径,支持通过 GM 参数传入 HCCL 通信上下文
  • 变更文件数:39
  • 检视侧别:Kernel + Tiling + 混合(op_api/op_graph/build)
  • 检视文档:cpp-secure.md, ascendc-topk.md, ascendc-api.md, ascendc-perf.md, mc2-specific.md, cpp-general.md, compile-secure.md
  • 初次检视时间:2026-06-06
  • 复查时间:2026-06-06(两次更新复查)

修复跟踪

上轮问题 修复轮次 当前状态
SEC-1.3 mc2ContextStruct 未零初始化 第1次更新 ✅ 已修复 → = {}; (mc2_context.cpp:739)
MC2-09 worldSize %d 格式化 第1次更新 ✅ 已修复 → %ld (mc2_context.cpp:724)
MC2-14 x/scales 缺 AutoContiguous 第1次更新引入语法错误(分号断链),第2次更新修复 ✅ 已修复 → 链式调用正确 (v2_def.cpp:39-40, 53-54)

检视统计(当前状态)

状态 条例数 占比
PASS 25 96%
已修复(原 FAIL/SUSPICIOUS) 3

已修复问题(原 FAIL/SUSPICIOUS,现已 PASS)

[SEC-1.3] 未初始化变量 — Mc2QuantReduceScatterContext ✅ 已修复

  • 原问题Mc2QuantReduceScatterContext mc2ContextStruct; 声明时未零初始化,HcclEngineCtxCopy 拷贝含未初始化字节的结构体到 device,构成 ISO C++ 未定义行为。
  • 修复确认:已改为 Mc2QuantReduceScatterContext mc2ContextStruct = {}; (mc2_context.cpp:739)
  • 原假设检验证据:正向规范违反(+40%)+上下文防御缺失(+30%)+PR归属(+20%)+数据流风险(+15%);负向防御存在(-20%)+上游校验(-15%);自信值=85%
  • 原修复建议= {} 零初始化 → 已采纳

[MC2-09] 量化参数类型一致性 — worldSize 格式化 ✅ 已修复

  • 原问题worldSize(int64_t)使用 %d 格式化打印,类型链路不一致。
  • 修复确认:已改为 %ld (mc2_context.cpp:724)
  • 原假设检验证据:自信值=70% → MED
  • 原修复建议%d%ld → 已采纳

[MC2-14] Tiling校验与结构规范 — v2 OpDef AutoContiguous ✅ 已修复

  • 原问题:v2 OpDef 中 x 和 scales 输入缺少 .AutoContiguous() 声明。
  • 修复过程:第1次更新修复时引入语法错误(.FormatList({ge::FORMAT_ND}); 末尾分号断链,.AutoContiguous() 成独立语句未生效),第2次更新移除分号,链式调用正确。
  • 修复确认
    // v2_def.cpp:39-40 (x)
    .FormatList({ge::FORMAT_ND})
    .AutoContiguous();
    // v2_def.cpp:53-54 (scales)
    .FormatList({ge::FORMAT_ND})
    .AutoContiguous();
    
  • 原假设检验证据:正向规范违反(+40%)+上下文防御缺失(+30%)+PR归属(+20%)+领域关联(+10%);自信值=100%

通过条例

[quant_reduce_scatter_kernel_G1] SEC-1.1 PASS
[quant_reduce_scatter_kernel_G1] SEC-1.2 PASS
[quant_reduce_scatter_kernel_G1] SEC-1.3 PASS
[quant_reduce_scatter_kernel_G1] SEC-2.1 PASS
[quant_reduce_scatter_kernel_G1] SEC-2.2 PASS
[quant_reduce_scatter_kernel_G1] SEC-2.3 PASS
[quant_reduce_scatter_kernel_G1] TOPK-8 PASS
[quant_reduce_scatter_kernel_G1] MC2-01 PASS
[quant_reduce_scatter_kernel_G1] MC2-02 PASS
[quant_reduce_scatter_kernel_G1] MC2-04 PASS
[quant_reduce_scatter_kernel_G1] MC2-09 PASS
[quant_reduce_scatter_kernel_G1] MC2-11 PASS
[quant_reduce_scatter_kernel_G1] MC2-17 PASS
[quant_reduce_scatter_kernel_G1] MC2-18 PASS
[quant_reduce_scatter_kernel_G1] API-3 PASS
[quant_reduce_scatter_kernel_G1] API-6 PASS
[quant_reduce_scatter_kernel_G1] API-8 PASS
[quant_reduce_scatter_host_G1] SEC-1.1 PASS
[quant_reduce_scatter_host_G1] SEC-1.2 PASS
[quant_reduce_scatter_host_G1] SEC-1.3 PASS
[quant_reduce_scatter_host_G1] SEC-2.1 PASS
[quant_reduce_scatter_host_G1] SEC-2.2 PASS
[quant_reduce_scatter_host_G1] SEC-2.3 PASS
[shared] SEC-1.1 PASS
[shared] MC2-04 PASS
[shared] MC2-14 PASS
[shared] MC2-15 PASS
[quant_reduce_scatter_v2_host_G1] MC2-04 PASS
[quant_reduce_scatter_v2_host_G1] MC2-09 PASS
[quant_reduce_scatter_v2_host_G1] TOPK-1 PASS
[quant_reduce_scatter_v2_host_G1] TOPK-2 PASS

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

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
*/*/op_api/aclnn*.h 王永光, 俞郑中 (2/2) 王永光 (1/1)
*/*/op_graph/*_proto.h 王永光, 俞郑中 (2/2) 王永光 (1/1)
*/*/op_host/*_def.cpp 王永光, 俞郑中 (2/2) 王永光 (1/1)
build.sh wang-minbo, 俞郑中 (2/2) wang-minbo (1/1)
classify_rule.yaml wang-minbo, 俞郑中 (2/2) wang-minbo (1/1)
cmake wang-minbo, 俞郑中 (2/2) wang-minbo (1/1)
mc2 wang-minbo, libohao6, 俞郑中 (3/2) wang-minbo, libohao6 (2/1)
tests wang-minbo, 俞郑中 (2/2) wang-minbo (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
yifuxiongyifuxiong成员
6月3日 update merge request[project id: 7673863, iid: 6357, commit_id: bde05619b8c1e9d36891779c7cf35d58a063f795] virtual merging success
yifuxiongyifuxiong成员
6月3日 强制推送  1 个提交:c8d8fe6c-modify quant_reduce_scatter hccl_context
此处折叠了310条消息 查看更多
CANN-robotCANN-robot成员
6月6日 添加了label:ci-pipeline-passed
YuZhengzhong成员
6月6日 评论:

/lgtm

likedislike
wangyongguang成员
6月6日 评论:

/approve

likedislike
CANN-robotCANN-robot成员
6月6日 添加了label:lgtmapproved
CANN-robotCANN-robot成员
6月6日 合入了pull request