已合并
fix rdma signal flag #5397
hu-yiliang11创建于 9月16日
fix rdma signal flag #5397
已合并
hu-yiliang11创建于 9月16日
hu-yiliang11
9月16日

描述

设置集合通信RDMA wr属性,datanotify和dataacknotify才生成cqe

变更类型

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

关联的Issue

https://gitcode.com/cann/hcomm/issues/921

测试

已完成的测试用例和场景:
1.
2.

补充的UT用例:

文档更新

合入检查

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 hu-yiliang11 的贡献)
Hhu-yiliang11
9月16日 创建了 pull request,commit 051ec7c1
atomgit-bot
atomgit-bot
9月16日 评论:

变更摘要

该 PR 为一次 Bug 修复,主要修复 RDMA 传输路径中发送请求(WQE)的 signal 标志设置逻辑,涉及文件 src/legacy/ascend910/platform/resource/transport/device/transport_device_ibverbs.cc。核心改动是:在 TxSendWqe 中不再无条件地为写请求设置 RA_SEND_SIGNALED,而是根据 wqeType 是否为数据通知类型来决定是否携带 signal 标志;同时在 TxPayLoad 中调用 ConstructPayLoadWqe 时显式传入 false,明确该路径无需 signal。

主要改动

  • 按 WQE 类型决定 signal 标志:在 TxSendWqe 中新增 isNeedSignaled 判断,仅当 wqeType 为 WQE_TYPE_DATA_NOTIFY 或 WQE_TYPE_DATA_ACK_NOTIFY 时才设置 RA_SEND_SIGNALED,否则该位为 0;fence_ 为真时仍保留 RA_SEND_SIGNALED | RA_SEND_FENCE,修复了原先对所有发送均强制设置 RA_SEND_SIGNALED 的问题。

  • TxPayLoad 显式关闭 signal:在 TxPayLoad 中调用 ConstructPayLoadWqe 时追加参数 false,明确该场景不产生 signal,保证与 TxSendWqe 的新逻辑一致。

likedislike
不准确?
CANN-robotCANN-robot成员
9月16日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
9月16日 评论:

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.
You can self-configure the PR merge rules for this repository. For more details, please refer to Here.
For more, you also can visit HICANN.


PR Approval Progress

✅ Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
src/legacy/ascend910/platform ✅ 陈浩, 陈量 (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

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

likedislike
atomgit-bot
atomgit-bot
9月16日 评论:

代码审查

✅ 未发现问题

likedislike
不准确?
CANN-robotCANN-robot成员
9月16日 将leabclove,Flashfuture,yanglianxiang2,linyf950,yanzhenghang,chenliang267,yanyefeng,chenhao_,dingweiqin_57,songmingyang,wenxuemin,LQ_OY,yangxiaoyuan,laodazhao1,yangshaohua,chenke2026,zhongqin2020,bz_lee,Innoecho,chenyia,temper7620,zhangxp1030,lilin_137,hulk1997,ouyangxizi,one_random设为评审人
CANN-robotCANN-robot成员
9月16日 将leabclove,yanyefeng,chenhao_,wenxuemin,Innoecho,temper7620设为审查人
hu-yiliang11
9月17日 评论:

compile

likedislike
Hhu-yiliang11
9月17日 预合并成功(commit_id: 88aa5a6d54dd38316d319f182224eac56bef5e1a)
CANN-robot
CANN-robot成员
9月17日 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
9月17日 添加了label:ci-pipeline-running
CANN-robotCANN-robot成员
9月17日 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
9月17日 添加了label:ci-pipeline-passed
Hhu-yiliang11
9月17日 修改了pull request 的描述
陈浩
陈浩成员
9月17日 评论:

/approve

likedislike
CANN-robotCANN-robot成员
9月17日 添加了label:approved
chenliang267
chenliang267成员
9月17日 评论:

/lgtm

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