已合并
feat(ccu): synchronize dataplane optimizations and fixes #160
DragonBornHD84创建于 29 天前
feat(ccu): synchronize dataplane optimizations and fixes #160
已合并
DragonBornHD84创建于 29 天前
DragonBornHD84成员
29 天前

当前PR是否有AI参与:

[x] 否

[] 是

PR功能描述 / 为什么需要这个合入**:

将 CCU 数据面从 hcomm 仓迁移到 asc-comm 仓,实现通信数据面在 asc-comm 的独立承载与交付:

  • 数据面迁入:CCU 数据面(DSL/Kernel/REP/microcode/Launch)整体迁入 asc-comm(新增 src/ccu 165 个文件、include/ccu 17 个头文件),跨 SO 边界只传 POD、handle 和 C 回调
  • 独立构建交付:独立构建交付 libasccomm_ccu_dataplane.so 与 pkg_inc 头;本仓以构建挂接方式成为 asc-devkit 的构建子工程(源码仓独立、同级放置),编译期不依赖 hcomm 源码树或已安装 hcomm 头文件;跨 SO ABI 头原样保存在 include/hcomm
  • 样例迁移:样例目录迁至 examples/ccu 并重命名(all_to_all / all_to_all_vc / all_to_allv 等,共 112 个文件),build.sh 新增 CCU UT 的 CTest 挂接与覆盖率收集
  • 同步主线修复:同步主线数据面优化与修复,补齐 UT(新增 tests/ccu 97 个文件),并修复桩库 CXX11 ABI 不一致导致的段错误

该PR关联的issue

(格式为fixes #<issue号>, 或者resolves #<issue号>): 无

希望检视人员了解:

  • 改动量大(416 个文件,+53176/-772;406 新增/7 修改/3 删除),主体为数据面整体迁移,建议按目录分片检视:src/ccu(数据面实现)、include/ccu(对外头)、examples/ccu(样例)、tests/ccu(UT)
  • 对外 README 已补充"交付与构建方式"说明
  • pre-commit 全量检查(clang-format / ruff / codespell / OAT / markdown-link)已通过

改动类型 / Change Type

测试信息 / Testing

检查清单 / Checklist

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 DragonBornHD84 的贡献)
DDragonBornHD84成员
29 天前 创建了 pull request,commit fa540e0a
atomgit-bot
atomgit-bot
29 天前 评论:

🤖 正在生成合并请求摘要,请稍候…

likedislike
不准确?
atomgit-bot
atomgit-bot
29 天前 评论:

🤖 AI 代码检视正在进行中,请稍候…

likedislike
不准确?
CANN-robotCANN-robot成员
29 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
29 天前 评论:

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
examples ✅ 闫庆尚, bluesky901 (2/2) ✅ 闫庆尚 (1/1)
include ✅ 李东锋, bluesky901 (2/2) ✅ 李东锋 (1/1)
repo-cann/asc-comm ✅ 李东锋, 闫庆尚, bluesky901 (3/2) ✅ 闫庆尚, 李东锋 (2/1)
src ✅ 闫庆尚, bluesky901 (2/2) ✅ 闫庆尚 (1/1)
tests ✅ 闫庆尚, bluesky901 (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

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

likedislike
此处折叠了129条消息 查看更多
bluesky901成员
26 天前 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
26 天前 添加了label:lgtm
CANN-robot
CANN-robot成员
26 天前 评论:

The MR can not be merged, because of CodeReview discussion not resolved

If you want to solve this problem, you can click here to do it in the FAQs.

likedislike
DDragonBornHD84成员
26 天前 解决了最后一个问题
CANN-robotCANN-robot成员
26 天前 合入了pull request